Red Hat Bugzilla – Bug 313431
Review Request: fedorainfinity-kdm-theme - Fedora Infinity KDM theme
Last modified: 2007-11-30 17:12:17 EST
Spec URL: http://repo.calcforge.org/f8/fedorainfinity-kdm-theme.spec SRPM URL: http://repo.calcforge.org/f8/fedorainfinity-kdm-theme-1.0.0-1.fc7.src.rpm Description: This package contains the Fedora Infinity KDM theme that is the default theme in Fedora 8. (I ported the GDM theme to KDM. To use, s/FlyingHigh/Infinity/g in the default kdmrc, for both the theme and the color scheme.)
I'll take this.
A few picky SHOULDS: 1. using %find_lang. I don't see any included translations, so I'd suggest removing the extra bloat wrt %find_lang. 2. use %defattr(-, root, root, -) and MUST: 3. License: GPLv2 but missing %doc COPYING Otherwise, it's simple/clean, fix at item #3 (I'll trust you to do that). APPROVED.
Oh, and it looks real nice btw. :)
The theme doesn't use a logo, yea? Probably don't need the system-logos Requires.
Also, might be good to talk to jeremy about getting this put on http://git.fedoraproject.org/ with the other themes.
> 1. using %find_lang. I don't see any included translations, so I'd suggest > removing the extra bloat wrt %find_lang. That one was lifted from the fedorainfinity-gdm-theme specfile (which I used as the template), maybe the %find_lang should be removed there too? > The theme doesn't use a logo, yea? Probably don't need the system-logos > Requires. Shouldn't the Requires go away in fedorainfinity-gdm-theme too then?
I'm also going to go ahead and put GPL+ into the License tag because that's what the GDM theme now has and I sure don't care about what version of the GPL my changes are used under.
New Package CVS Request ======================= Package Name: fedorainfinity-kdm-theme Short Description: Fedora Infinity KDM theme Owners: kkofler,rdieter,than Branches: InitialCC: Cvsextras Commits: no (Oops, actually putting in a valid CVS request this time. I accidentally set the flag and then commented about other stuff.)
Just for reference, this is what I intend to commit: Spec URL: http://repo.calcforge.org/f8/fedorainfinity-kdm-theme.spec SRPM URL: http://repo.calcforge.org/f8/fedorainfinity-kdm-theme-1.0.0-2.fc7.src.rpm - Change License tag to GPL+ as for the GDM theme - Drop unneeded Requires: system-logos - Pass all 4 arguments to %%defattr - Drop unneeded %%find_lang as there are no translations anyway - Add COPYING as %%doc
Hmmm, is it actually OK to label this, the GDM theme it's based on and all the other redhat-artwork splitoffs as GPL+? The GPL says if there's a reference to "GNU General Public License" without a version specified, then any version can be chosen, but here the only reference to the GPL is the included copy of the GPLv2, so what actually applies here?
Mo, what license is Fedora Infinity under? Is it based on another theme?
ping, cvsadmins (see comment #8 )? Been a couple of days now, kde spin needs this.
Sorry for the delay. cvs done.
Hey Kevin, Can you add an Obsoletes: redhat-artwork-kde to this package?
Yes, I'll do that. It's not ideal because this will also remove stuff like Bluecurve which will have to be reinstalled by hand if the users want it, but that's the nature of package splits.
Obsoletes: redhat-artwork-kde added in fedorainfinity-kdm-theme-1.0.0-3.fc8, built for dist-f8.