[Bug 236798] Review Request: ballz - Platform game with some puzzle elements
bugzilla at redhat.com
bugzilla at redhat.com
Fri Apr 20 19:41:24 UTC 2007
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: ballz - Platform game with some puzzle elements
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236798
musuruan at gmail.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
Flag|fedora-review? |fedora-review+
------- Additional Comments From musuruan at gmail.com 2007-04-20 15:41 EST -------
- Package meets naming and packaging guidelines. OK
- Spec file matches base package name. OK
- Spec has consistant macro usage. OK
- Meets Packaging Guidelines. OK
- License: BSD
- License field in spec matches. OK
- License file included in package. OK
- Spec in American English. OK
- Spec is legible. OK
- Sources match upstream. OK md5sum: 1a98a0a87985f2425d2fde5715be50ed
- Package needs ExcludeArch. N/A
- BuildRequires correct. OK.
- Spec handles locales/find_lang. N/A
- Package is relocatable and has a reason to be. N/A
- Package has %defattr and permissions on files is good. OK
- Package has a correct %clean section. OK
- Package has correct buildroot. OK
- Package is code or permissible content. OK
- Doc subpackage needed/used. N/A
- Packages %doc files don't affect runtime. OK
- Headers/static libs in -devel subpackage. N/A
- Spec has needed ldconfig in post and postun. N/A
- .pc files in -devel subpackage/requires pkgconfig. N/A
- .so files in -devel subpackage. N/A
- -devel package Requires: %{name} = %{version}-%{release}. N/A
- .la files are removed. N/A
- Package is a GUI app and has a .desktop file. OK
- Package compiles and builds on at least one arch. OK fc6/i386
- Package has no duplicate files in %files. OK
- Package doesn't own any directories other packages own. OK
- Package owns all the directories it creates. OK
- No rpmlint output. OK
- final provides and requires are sane:
/bin/sh
/bin/sh
hicolor-icon-theme
libaldmb-0.9.3.so
liballeg.so.4.2
libdumb-0.9.3.so
libguichan-0.6.1.so
libguichan_allegro-0.6.1.so
rtld(GNU_HASH)
SHOULD Items:
- Should build in mock. OK fc6/i386
- Should function as described. OK
- Should have sane scriptlets. OK
- Should have subpackages require base package with fully versioned depend. N/A
- Should have dist tag. OK
- Should package latest version. OK
Issues:
1. Not a blocker, but on the PC I used for the review, the viewport was not
centered at fullscreen. BTW, why did you patched the sources to use
fullscreen instead of the default window mode?
APPROVED.
--
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-package-review
mailing list