Bug 321351 - Review Request: vdr-skinsoppalusikka - The "Soppalusikka" skin for VDR
Summary: Review Request: vdr-skinsoppalusikka - The "Soppalusikka" skin for VDR
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Ville Skyttä
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-10-06 14:44 UTC by Ville-Pekka Vainio
Modified: 2008-01-15 22:07 UTC (History)
2 users (show)

Clone Of:
Last Closed: 2007-10-11 21:27:12 UTC
ville.skytta: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Ville-Pekka Vainio 2007-10-06 14:44:35 UTC
Spec URL: http://www.cs.helsinki.fi/u/vpivaini/rpm/soppalusikka/vdr-skinsoppalusikka.spec
SRPM URL: http://www.cs.helsinki.fi/u/vpivaini/rpm/soppalusikka/vdr-skinsoppalusikka-1.0.4-0.1.fc7.src.rpm
Description: The "Soppalusikka" is a standalone skin is based on "enElchi" text2skin addon. As this skin is a standalone plugin, you don't need text2skin at all to have an enElchi style skin on your VDR.

This is my first package ever, so I need a sponsor. The .spec file has been based on the .spec file of vdr-femon packaged by Ville Skyttä.

The upstream .tgz has some Finnish TV channel logos in it, but I've chosen to leave them out of the binary RPM, because the copyright situation is unclear. Upstream also ships some "symbol" images with the plugin, I have included those, because they are made by the plugin maintainer and should be under GPLv2.

For some reason rpmbuild includes those symbol image files into the debuginfo package and I couldn't find a way to tell it to ignore them. Otherwise the debuginfo looks good to me.

Comment 1 Ville-Pekka Vainio 2007-10-06 16:13:32 UTC
The Mandriva packager of skinsoppalusikka told me that I don't need to include
the  symbols as files into the packages, because they are compiled into the
plugin file.

Here's an updated SRPM:
The .spec file from the first message is updated as well.

The channel logo issue was discussed on #fedora-devel, I need to make a bash
script that removes the channel logos from the .tgz and re-zips, since the logos
can't be distributed even in the .srpm. I'll make yet another updated for that

Comment 2 Ville-Pekka Vainio 2007-10-06 17:16:08 UTC
The undistributable channel logos have now been removed from the .srpm too. I
added a script to download the upstream .tgz, remove the channel logos and
repackage as .tar.bz2. The script is based on a similar script in the vdr-skins

Everything in the skinsoppalusikka package is now distributable under the GPLv2.

Updated SRPM:
The .spec file from the first message is updated as well.

Comment 3 Ville Skyttä 2007-10-07 15:19:08 UTC
Good work, just some minor remarks:

- License should be GPL+, not GPLv2 as the sources or anything in the package
doesn't specify a GPL version and in that case (per the GPL text itself, chapter
9 of GPLv2 COPYING) any version of GPL goes.

- s/femon/skinsoppalusikka/ in vdr-skinsoppalusikka.conf's comments

- The main vdr package already owns the /var/lib/vdr/themes dir, so the %dir
entry for it should be removed from %files in this package.

Comment 4 Ville Skyttä 2007-10-07 15:23:44 UTC
Also, %{datadir} appears to be unused in this package so the %define for it can
be dropped.

Comment 5 Ville-Pekka Vainio 2007-10-07 15:54:40 UTC
Thanks for the feedback. I've fixed all of the issues mentioned (hopefully). See
the spec file in the original message and

Comment 6 Ville Skyttä 2007-10-07 16:25:01 UTC
Looks good, approved and will sponsor; please proceed with the process described
at http://fedoraproject.org/wiki/PackageMaintainers/Join

Remaining bits that can be addressed before the first build, no need to submit a
new SRPM:

- s/soppalusikka/skinsoppalusikka/ in vdr-skinsoppalusikka.conf :)

- Bump release to 1%{?dist} for the first Fedora build

Comment 7 Ville-Pekka Vainio 2007-10-07 17:43:23 UTC
New Package CVS Request
Package Name: vdr-skinsoppalusikka
Short Description: The "Soppalusikka" skin for VDR
Owners: vpv
Branches: F-7
Cvsextras Commits: yes

Comment 8 Kevin Fenzi 2007-10-08 18:22:11 UTC
cvs done.

Comment 9 Ville-Pekka Vainio 2007-10-11 21:27:12 UTC
The packages are built in Koji, I guess closing this as NEXTRELEASE is the right
thing to do? I'll try to push the package into F-7 with Bodhi next.

Comment 10 Ville Skyttä 2007-10-12 06:36:37 UTC
Yep, that's correct.

Note You need to log in before you can comment on or make changes to this bug.