Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556346 --- Comment #8 from Tim Niemueller <tim@xxxxxxxxxxxxx> 2010-02-11 16:18:20 EST --- And here my quick review for Stage. MUST * OK: rpmlint You shold report the shared-lib-calls-exit to upstream. The library is calling exit all over the place which could be avoided. * OK: package name * OK: package version and release * OK: spec file name * OK: package guideline-compliant * OK: license complies with guidelines * OK: license field accurate * OK: license file not deleted * OK: spec in US English * OK: spec legible * OK: source matches upstream sha256sum 90fb5160655b0c692f6a607d8cd680daacaf4093855937d436abc671b756ab13 * OK: builds under >= 1 archs, others excluded * OK: dependencies (requires) * OK: build dependencies complete * N/A: locales handled using %find_lang, no %{_datadir}/locale * OK: library -> ldconfig * N/A: relocatable: give reason * OK: own all directories * OK: no dupes in %files * OK: permission * OK: %clean RPM_BUILD_ROOT * OK: macros used consistently * OK: Package contains code * OK: large docs => -doc * OK: doc not runtime dependent * OK: headers in -devel * N/A: static in -static * OK: if contains *.pc, req pkgconfig * OK: if libfiles are suffixed, the non-suffixed goes to devel * OK: devel requires versioned base package * N/A: desktop file uses desktop-file-install * OK: clean buildroot before install * OK: filenames UTF-8 SHOULD * N/A: if license text missing, ask upstream to include it * N/A: desc and summary contain translations if available * OK: package build in mock on all architectures Koji builds given by packager * OK: package functioned as described Works during short test. Used standalone (stage /usr/share/stage/worlds/simple.world) and in Player (player /usr/share/stage/worlds/simple.cfg). * OK: scriplets are sane * N/A: other subpackages should require versioned base * N/A: if main pkg is development-wise, pkgconfig can go in main package * OK: require package not files -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review