http://cachalot.mine.nu/6/SRPMS/vdr-skins.spec http://cachalot.mine.nu/6/SRPMS/vdr-skins-20061119-1.cmn6.src.rpm This package contains a collection of skins for VDR's on-screen display.
- rpmlint checks return: E: vdr-skins non-standard-gid /etc/vdr/themes/[...].theme video E: vdr-skins non-standard-uid /etc/vdr/themes/[...].theme vdr W: vdr-skins dangerous-command-in-%trigger rm W: vdr-skins dangling-symlink /usr/share/vdr/text2skin/Enigma/FontMonoSpaced.ttf /usr/share/fonts/bitstream-vera/VeraMono.ttf W: vdr-skins incoherent-version-in-changelog 20061119-3 20061119-1 W: vdr-skins non-conffile-in-etc /etc/vdr/themes/[...].theme W: vdr-skins no-url-tag W: vdr-skins symlink-should-be-relative /usr/share/vdr/text2skin/Enigma/FontMonoSpaced.ttf /usr/share/fonts/bitstream-vera/VeraMono.ttf - package meets naming guidelines - package meets packaging guidelines - license (GPL) OK, text in %doc, matches source - spec file legible, in am. english - source doesn't match upstream (see below) - package compiles on devel (x86_84) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - no need for -docs - nothing in %doc affects runtime - no need for .desktop file o Some parts under /etc should probably move into /usr/share. o Some URLs are broken like http://www.magoa.net/sttng-blue.theme and http://vdr.pfroen.de/download/DeepBlue-0.1.4.tar.gz. o Due to the latter the prepare script does not work till the end. o The ones that were created by the script had different md5sums (but maybe that's expected due to tar/bz2 timestamping, in that case: do we have some tarball comparison tool?). I could only verify EgalSimple-1.0-demo.tar.bz2 and SilverGreen-0.1.7.tar.bz2 [side note: why are the logos removed? Legal issues?]
(In reply to comment #1) > o Some parts under /etc should probably move into /usr/share. Makes sense, although only the *.theme files remain in /etc/vdr/themes and that dir can't be changed without changing it in the main vdr package as well. Added to my TODO list for future vdr package revisions. > o Some URLs are broken like http://www.magoa.net/sttng-blue.theme and > http://vdr.pfroen.de/download/DeepBlue-0.1.4.tar.gz. > > o Due to the latter the prepare script does not work till the end. Yeah, I've been trying to ping upstreams of skins whose download URLs no longer work, but haven't got replies from all of them :( Archive.org appears to have a copy of sttng-blue.theme: http://web.archive.org/web/*/http://www.magoa.net/sttng-blue.theme So I guess that leaves us only DeepBlue-0.1.4.tar.gz MIA. Suggestions? I've had it in this package for ages, and IMHO upstream availablity/verification is not *that* critical because it's only data. If you feel more comfortable with dropping it for now, will do and reintroduce later if upstream reappears. > o The ones that were created by the script had different md5sums (but maybe > that's expected due to tar/bz2 timestamping, in that case: do we have some > tarball comparison tool?). I could only verify EgalSimple-1.0-demo.tar.bz2 > and SilverGreen-0.1.7.tar.bz2 bunzip2'ing and comparing the uncompressed tarballs could work. And rpmdev-diff might be useful as well. > [side note: why are the logos removed? Legal issues?] They are logos of various TV and radio channels, most of which are likely trademarked, and I haven't found any evidence that they'd be included in upstream tarballs with the trademark holders' consent.
(In reply to comment #2) > > o Some parts under /etc should probably move into /usr/share. > > Makes sense, although only the *.theme files remain in /etc/vdr/themes and > that dir can't be changed without changing it in the main vdr package as well. > Added to my TODO list for future vdr package revisions. Maybe just a symlink in /etc pointing to a /usr/share location? > I've had it in this package for ages, and IMHO upstream > availablity/verification is not *that* critical because it's only data. > If you feel more comfortable with dropping it for now, will do and > reintroduce later if upstream reappears. No, that's not a blocker, at least not for me, and I hope that the guidelines can be interpreted as such. Verification of sources is needed for novice errors and untrusted parties, I'd say in this case we can outrule both. :) I just wanted to mention these issues to keep them in mind, and to have this in the review in case someone digs this up in the future and thinks we didn't do a thorough job, after all I need to write something about whether the sources could be verified or not. I'd say, if you like move the non-config parts out of /etc leaving a symlink behind, but that's just a mild recommendation. I'll just go ahead and approve as is. :) BTW: Perhaps you want to remove the %{?dist} tag, as the package wouldn't change from release to release. But I don't know how to make the exact same package appear in FC5/FC6/F7 simultaneously w/ either plague or brew, so maybe you can't drop it.
Thanks! I'll think about the /etc -> /usr/share change for a bit before the first build; however the symlink approach doesn't sound tempting as symlinks to dirs may cause various annoyances later. Maybe I'll just roll a new vdr package with the themes dir moved before the first build of this one. Looks like a bunch of not that large noarch packages have been accepted for disttagless copying between branches recently (such as various -firmware packages), so I suppose it's ok here as well, will drop the disttag. New Package CVS Request ======================= Package Name: vdr-skins Short Description: Collection of OSD skins for VDR Owners: ville.skytta Branches: FC-6 InitialCC:
Actually, in some situations VDR writes to the *.theme files, so I'll relocate them to /var/lib/vdr/themes instead of /usr/share.
Built for devel a week ago, repocopied to FC6 today, should appear in the next push along with a vdr revision which relocates themes to /var/lib/vdr/themes.