[Bug 2492] Review request: mame-data-extras - Extra data files for MAME
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2492
--- Comment #13 from Sérgio Basto <sergio(a)serjux.com> 2012-11-16 04:41:12 CET ---
Hi, sorry for this big delay, I have been very very busy.
If you have something that don't understand on this comment, please tell me.
This is based on fedora-review, I think some items could be improved, after
that I will approved the review.
Thanks.
[!] = Fail
[ ] = Manual review needed
Issues:
=======
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. No licenses
found. Please check the source files for licenses manually.
See: http://fedoraproject.org/wiki/Packaging
/LicensingGuidelines#ValidLicenseShortNames
[!]: Large documentation must go in a -doc subpackage.
Note: Documentation size is 14807040 bytes in 13 files.
See: http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation
===== MUST items =====
[ ]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf %{buildroot} present but not required
===== SHOULD items =====
[!]: Spec use %global instead of %define.
Note: %define vernumber 147
Rpmlint
-------
mame-data-extras.src: W: file-size-mismatch messinfo.zip = 1902507,
http://www.progettosnaps.net/messinfo/messinfo.zip = 1940181
mame-data-extras.src: W: file-size-mismatch cheat0147.zip = 759300,
http://cheat.retrogames.com/download/cheat0147.zip = 760065
http://cheat.retrogames.com/download/cheat0147.zip :
CHECKSUM(SHA256) this package :
cc5a46a1814a4124f68880500e7736246991d8eca35f3ca46cec4d4a9739c166
CHECKSUM(SHA256) upstream package :
4bd1524583070180798014a14a97478b6c90e633ae90db530862bcacfb5d257e
http://www.progettosnaps.net/messinfo/messinfo.zip :
CHECKSUM(SHA256) this package :
d0c166fe7acd8294a7caa9703b953dc98551da5a8401f86bdf391d33dbf6507b
CHECKSUM(SHA256) upstream package :
6712cd2340c6484ac8518d8c9a46b851b755691f67c3140bd06dedc309dff83b
mame-data-extras-robby.noarch: W: spurious-executable-perm
/usr/share/mame/roms/robby.zip
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.