Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=559117 --- Comment #9 from Jussi Lehtola <jussi.lehtola@xxxxxx> 2010-01-27 18:22:44 EST --- (In reply to comment #7) > OK. Regarding the icon, I'll move it into /usr/share/pixmaps as recommended, > but I thought I'd point out why I put it in /usr/share/icons/hicolor in the > first place. From the freedesktop.org Icon Theme Specification: Hmm, I'm no expert, but the beginning of the "Directory Layout" section states: "Icons and themes are looked for in a set of directories. By default, apps should look in $HOME/.icons (for backwards compatibility), in $XDG_DATA_DIRS/icons and in /usr/share/pixmaps (in that order)." > I can't see any way to interpret this that allows for the hicolor theme to be > optional; it seems that if Fedora intends to be compliant with freedesktop.org > standards, the hicolor theme *must* be provided. > > Anyhow, I'm not pushing for any change in Fedora policy, but since Fedora icon > policy differs from the freedesktop.org standard, it might be desirable to > update the desktop section of the packaging policy to explicitly state that the > basic fedora icon requirement is to use /usr/share/pixmaps. That might prevent > another novice packager from making the same mistake I did. OK. If you're interested in packaging standards, you can write up a proposal for the Fedora Packaging Committee, which can then propose the guideline to the Fedora Steering Committee. ** This package is good to go. I won't give it a formal approval yet, since you wouldn't be able to do anything with it yet since you're not a member of the packaging group. Do the one other informal review, and I will sponsor you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review