[Bug 185257] Review Request: raidem - 2d top-down shoot'em up

[bugzilla at redhat.com]

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: raidem - 2d top-down shoot'em up


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





------- Additional Comments From wart at kobold.org  2006-03-15 02:31 EST -------
rpmlint warnings:

W: raidem invalid-license zlib License
W: raidem-debuginfo invalid-license zlib License

These can be ignored.  This is an acceptible license.

W: raidem uncompressed-zip /usr/share/raidem/data/wraith.zip
W: raidem uncompressed-zip /usr/share/raidem/data/proj-maser.zip
W: raidem uncompressed-zip /usr/share/raidem/data/sea-urchin.zip
W: raidem uncompressed-zip /usr/share/raidem/data/conquerer.zip
W: raidem uncompressed-zip /usr/share/raidem/data/spirtle.zip
W: raidem uncompressed-zip /usr/share/raidem/data/zippy.zip
W: raidem uncompressed-zip /usr/share/raidem/data/bolt.zip
W: raidem uncompressed-zip /usr/share/raidem/data/schemer.zip
W: raidem uncompressed-zip /usr/share/raidem/data/proj-pulselaser.zip

These can also be ignored.  These files contain game data that are
extracted by the game engine at runtime.

MUST
====

* Package and spec named appropriately
* License 'zlib' ok, license file included
* spec file legible and in Am. English
* Builds and runs on FC-5 i386
* No excessive or inappropriate BR: or Requires:
* No locales
* No shared libraries
* Not relocatable
* Owns all directories that it creates
* No duplicates %files
* Permissions look ok
* No -devel package necessary
* Contains both code and permissible content (game data)
* Macro use is consistent and sane
* buildroot cleaned during %install and %clean
* No .la archives or static libs
* desktop file included, looks ok

NON-BLOCKING
============
* Source does not match upstream, for a good reason.  Upstream source
  contains a mp3 encoder that can't be included in FE.  The game still
  functions without it, so the offending code has been removed.

MUSTFIX
=======
* Delete the duplicate copies of the various libs during %prep, not by
  modifying the upstream tarball.


-- 
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.