[Bug 185257] Review Request: raidem - 2d top-down shoot'em up
bugzilla at redhat.com
bugzilla at redhat.com
Wed Mar 15 07:31:42 UTC 2006
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.
More information about the fedora-extras-list
mailing list