[Bug 228425] Review Request: gtkpod - Graphical song management program for Apple's iPod

bugzilla at redhat.com bugzilla at redhat.com
Sun Feb 18 04:29:30 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gtkpod - Graphical song management program for Apple's iPod
Alias: gtkpod-review

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228425





------- Additional Comments From tmz at pobox.com  2007-02-17 23:29 EST -------
Xavier, would you please show me where this is required or recommended?  Just
saying I need to do something isn't very good in my opinion.  I want to know why
it should be changed so that I understand it for next time.  This will also help
others reading the review to learn why one path is better than another.

Better than that, if I understand why it's better in one place than the other I
can most likely get it installed there upstream so I don't have to worry about
doing it in the specfile.

A quick grep of the specfiles in extras-development shows me there are 63 specs
that install something to %{_datadir}/pixmaps and 39 installing to
%{_datadir}/icons.  It seems to me that unless there is a requirement to use one
or the other, that this is just a matter of preference and it should not be a
blocker.  If you know differently or have good reason why I should install under
%{_datadir}/icons, please enlighten me.  I am always insterested in learning.

No matter where it ends up going, since the file is one that's included with the
upstream tarball and is already installed in %{_datadir}/%{name}/pixmaps, I
don't see why a symlink isn't better than a copy of the file.  Please explain
why a symlink isn't any good to you.

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the package-review mailing list