Patch to add a favicon to the web UI #1592

Closed
opened 2011-11-19 21:49:56 +00:00 by marcusw · 13 comments
marcusw commented 2011-11-19 21:49:56 +00:00
Owner

I came up with a small (almost trivial) git branch to add a favicon to the web UI. More information is available at the commit page on github.

I came up with a small (almost trivial) [git branch](https://github.com/marcuswanner/tahoe-lafs/tree/favicon) to add [a favicon](@@https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1185#[comment:86533](/tahoe-lafs/trac-2024-07-25/issues/1592#issuecomment-86533)@@) to the web UI. More information is available at the [commit page](https://github.com/marcuswanner/tahoe-lafs/commit/b3ee8882f0819cab0efdcf1b7620bc9c74b2ff2f) on github.
tahoe-lafs added the
code-frontend-web
minor
enhancement
1.9.0
labels 2011-11-19 21:49:56 +00:00
tahoe-lafs added this to the undecided milestone 2011-11-19 21:49:56 +00:00
davidsarah commented 2011-11-19 23:12:52 +00:00
Author
Owner

I reviewed the XHTML changes on the git branch and they look good. I confirmed that those are all the XHTML files that need to be changed. I also like the favicon (if we're staying with Kevin Reid's icon more generally).

CC BY 3.0 Unported is not compatible with GPL, primarily because of the attribution requirement. (I hate acting the part of license compatibility enforcer, but care about getting updates to Tahoe-LAFS into distributions like Debian that are strict about that.) This probably means that Kevin Reid's icon needs to be relicensed, so that the derived favicon can be.

I reviewed the XHTML changes on the [git branch](https://github.com/marcuswanner/tahoe-lafs/commit/b3ee8882f0819cab0efdcf1b7620bc9c74b2ff2f) and they look good. I confirmed that those are all the XHTML files that need to be changed. I also like the favicon (if we're staying with Kevin Reid's icon more generally). [CC BY 3.0 Unported](https://creativecommons.org/licenses/by/3.0/) is [not compatible with GPL](https://www.gnu.org/licenses/license-list.html#ccby), primarily because of the attribution requirement. (I hate acting the part of license compatibility enforcer, but care about getting updates to Tahoe-LAFS into distributions like Debian that are strict about that.) This probably means that Kevin Reid's icon needs to be relicensed, so that the derived favicon can be.
tahoe-lafs modified the milestone from undecided to 1.10.0 2011-11-19 23:13:26 +00:00
marcusw commented 2011-11-20 02:15:16 +00:00
Author
Owner

I was afraid of the licensing issues and totally understand (hence the long-winded git comment). Bad things happen to projects with sloppy copyright. Have you contacted kreid about relicensing?

If he agrees, you've got the go-ahead from me to relicense my derivative (the 16x16 icon) under whatever conditions he chooses.

I was afraid of the licensing issues and totally understand (hence the long-winded git comment). Bad things happen to projects with sloppy copyright. Have you contacted kreid about relicensing? If he agrees, you've got the go-ahead from me to relicense my derivative (the 16x16 icon) under whatever conditions he chooses.

Dear Kevin: may we please have permission to use your Tahoe-LAFS logo contributions under the same licences as Tahoe-LAFS itself? Actually, would you mind granting the Tahoe-LAFS Software Foundation permission to use it however that entity sees fit, with little or no restriction? Thank you!

(The reason we're asking is to include a favicon derived from your logo in the Tahoe-LAFS distribution to include in the WUI.)

Dear Kevin: may we please have permission to use your Tahoe-LAFS logo contributions under the same licences as Tahoe-LAFS itself? Actually, would you mind granting the Tahoe-LAFS Software Foundation permission to use it however that entity sees fit, with little or no restriction? Thank you! (The reason we're asking is to include a favicon derived from your logo in the Tahoe-LAFS distribution to include in the WUI.)

I hereby give permission for my Tahoe logo designs and SVG files (all material at http://switchb.org/kpreid/2009/tahoe/ ) to reused, modified, and distributed under the same license(s) as Tahoe-LAFS itself, including any future relicensing or other such change which the developer community has consensus upon.

I hereby give permission for my Tahoe logo designs and SVG files (all material at <http://switchb.org/kpreid/2009/tahoe/> ) to reused, modified, and distributed under the same license(s) as Tahoe-LAFS itself, including any future relicensing or other such change which the developer community has consensus upon.
marcusw commented 2011-12-16 16:56:44 +00:00
Author
Owner

Any idea when/whether this will be landed?

Any idea when/whether this will be landed?

I removed the licensing keyword. Thanks, Kevin! I don't see either a review-needed keyword or a reviewed keyword so I can't tell instantly what state this ticket is in. Someone please push this ticket further along! https://tahoe-lafs.org/trac/tahoe-lafs/wiki/PatchReviewProcess

I removed the `licensing` keyword. Thanks, Kevin! I don't see either a `review-needed` keyword or a `reviewed` keyword so I can't tell instantly what state this ticket is in. Someone please push this ticket further along! <https://tahoe-lafs.org/trac/tahoe-lafs/wiki/PatchReviewProcess>
davidsarah commented 2011-12-17 01:43:26 +00:00
Author
Owner

marcusw: your changes are still under the license at https://github.com/marcuswanner/tahoe-lafs/commit/b3ee8882f0819cab0efdcf1b7620bc9c74b2ff2f . You need to relax that license, as kpreid did.

marcusw: your changes are still under the license at <https://github.com/marcuswanner/tahoe-lafs/commit/b3ee8882f0819cab0efdcf1b7620bc9c74b2ff2f> . You need to relax that license, as kpreid did.
marcusw commented 2011-12-17 01:50:28 +00:00
Author
Owner

I hereby give permission for my Tahoe logo designs and SVG files previously referenced in this ticket and that git commit message to be reused, modified, and distributed under the same license(s) as Tahoe-LAFS itself, including any future relicensing or other such change which the developer community has consensus upon. This release is in addition to the one on the git branch message.

The same applies to the files linked to in (@@https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1185#comment:-1@@)

I hereby give permission for my Tahoe logo designs and SVG files previously referenced in this ticket and that git commit message to be reused, modified, and distributed under the same license(s) as Tahoe-LAFS itself, including any future relicensing or other such change which the developer community has consensus upon. This release is in addition to the one on the git branch message. The same applies to the files linked to in (@@https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1185#[comment:-1](/tahoe-lafs/trac-2024-07-25/issues/1592#issuecomment--1)@@)
davidsarah commented 2011-12-17 02:02:26 +00:00
Author
Owner

Thanks. Reviewing.

Thanks. Reviewing.
davidsarah commented 2011-12-17 02:05:20 +00:00
Author
Owner

Oh, I already reviewed it in comment:86519. OK, I'll land this now.

Oh, I already reviewed it in [comment:86519](/tahoe-lafs/trac-2024-07-25/issues/1592#issuecomment-86519). OK, I'll land this now.
david-sarah@jacaranda.org commented 2011-12-17 04:35:23 +00:00
Author
Owner

In changeset:054374400e28cb00:

Marcus Wanner's favicon patch. fixes #1592
In changeset:054374400e28cb00: ``` Marcus Wanner's favicon patch. fixes #1592 ```
tahoe-lafs added the
fixed
label 2011-12-17 04:35:23 +00:00
david-sarah@jacaranda.org closed this issue 2011-12-17 04:35:23 +00:00
davidsarah commented 2011-12-17 04:37:29 +00:00
Author
Owner

I added some tests in changeset:531f562fcfcb7b68. Review needed for those, and also for a cleanup patch changeset:4618571389c62d2f.

I added some tests in changeset:531f562fcfcb7b68. Review needed for those, and also for a cleanup patch changeset:4618571389c62d2f.
tahoe-lafs removed the
fixed
label 2011-12-17 04:37:29 +00:00
davidsarah reopened this issue 2011-12-17 04:37:29 +00:00
zooko self-assigned this 2011-12-17 15:59:44 +00:00

Reviewed changeset:4618571389c62d2f and changeset:531f562fcfcb7b68.

Reviewed changeset:4618571389c62d2f and changeset:531f562fcfcb7b68.
zooko added the
fixed
label 2011-12-17 16:31:09 +00:00
zooko closed this issue 2011-12-17 16:31:09 +00:00
tahoe-lafs modified the milestone from 1.10.0 to 1.9.2 2012-03-29 23:30:02 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Reference: tahoe-lafs/trac-2024-07-25#1592
No description provided.