Red Hat Bugzilla – Bug 354741
Review Request: clonekeen - "Commander Keen: Invasion of the Vorticons" clone
Last modified: 2007-11-30 17:12:20 EST
Spec URL: http://people.atrpms.net/~hdegoede/clonekeen.spec
SRPM URL: http://people.atrpms.net/~hdegoede/clonekeen-0.8.3-1.fc9.src.rpm
CloneKeen is an almost complete clone of the old classic DOS game,
Commander Keen: Invasion of the Vorticons by by ID Software.
CloneKeen requires the original gamedata files to work.
If you posess the original DOS games. You can play all three episodes of the
game. If you don't, you can can still play the shareware episode one. Which can
be freely downloaded from Apogee, but cannot be distributed as a part of
Fedora. When you start CloneKeen for the first time it will offer to download
the shareware datafiles for you.
Initially I figured that there had to be some kind of trademark issue here, but
I looked around and I really can't find any indication that any part of
"Commander Keen: Invasion of the Vorticons" is trademarked. Still, I'm no
expert, so I'll review the packaging but you'll really need to get an ack from
someone who knows about this kind of thing.
Ok, I've asked Spot to take a look and add a comment here.
How did you determine the version of the GPL in use? I only see "GPL", no copy
of the license itself and no mention of GPL or version in any of the source
files except for what's in the scale2x directory. I guess that makes the
original code "GPL+" and the linked product "GPLv2+", but if you have something
more definitive it would be nice to have it in the package.
* source files match upstream:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
? summary is OK (assuming no trademark issues)
? description is OK (assuming no trademark issues)
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
clonekeen = 0.8.3-1.fc8
* Game tested manually; seems to work fine.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.
* desktop file looks OK.
APPROVED, subject to the trademark thing.
(In reply to comment #3)
> How did you determine the version of the GPL in use? I only see "GPL", no copy
> of the license itself and no mention of GPL or version in any of the source
> files except for what's in the scale2x directory. I guess that makes the
> original code "GPL+" and the linked product "GPLv2+", but if you have something
> more definitive it would be nice to have it in the package.
Yes the clonekeen code is GPL+, but since the binary also contains scale2x code
its GPLv2+, and so are the included extraction utilities I wrote. But indeed
other then the reference to the GPL in the docs (and prominently on the
website), there is no GPL copyright header for the clonekeen code itself.
I've gotten a green light on the trademark thing from Spot by private mail, and
thanks for the review!
New Package CVS Request
Package Name: clonekeen
Short Description: "Commander Keen: Invasion of the Vorticons" clone
Branches: F-7 F-8 devel
Cvsextras Commits: yes
Imported, build and updates for F-8 and F-7 pushed, closing.