Spec URL: http://jgorig.fedorapeople.org/bird/bird.spec SRPM URL: http://jgorig.fedorapeople.org/bird/bird-1.2.4-1.fc13.src.rpm Description: BIRD is dynamic routing daemon supporting IPv4 and IPv6 versions of routing protocols BGP, RIP and OSPF.
I will take this.
MUST ITEMS [!!] rpmlint bird.src: W: name-repeated-in-summary C BIRD bird.x86_64: W: name-repeated-in-summary C BIRD Seems to be fine. (However "Internet Routing Daemon" can be used as well.) bird.x86_64: W: no-manual-page-for-binary bird6 bird.x86_64: W: no-manual-page-for-binary birdc6 bird.x86_64: W: no-manual-page-for-binary bird bird.x86_64: W: no-manual-page-for-binary birdc According to guidelines, all binaries/scripts should have manual page. This is not as important to reject the review, but you should work with upstream on this. bird.x86_64: E: incoherent-subsys /etc/rc.d/init.d/bird bird6 bird.x86_64: E: incoherent-subsys /etc/rc.d/init.d/bird bird6 This is bad. I suggest creating separate initscripts and bird6 subpackage. [ok] naming [ok] licensing source package doesn't contain file with license [ok] spec: American English [ok] spec: legible [ok] spec: sources match the upstream sources [ok] possible to build on rawhide (f15) x86_64 in mock [ok] buildrequires [ok] locales none included [ok] shared libraries none included [ok] no bundled libraries [ok] clean section [ok] relocatable, /usr prefix [ok] owns all directories [ok] no duplicates in %files [ok] permissions on files set properly [ok] consistent use of macros [ok] contains code or permissible content [ok] large documentation goes in -doc subpackage not needed [ok] will run properly without %doc files [ok] headers in -devel subpackage not needed [ok] static libraries in -static subpackage not needed [ok] dynamic libraries without version specification in -devel subpackage not needed [ok] devel package dependencies not needed [ok] no libtool archives [ok] desktop file for gui application not gui application [ok] doesn't own already owned directories [ok] all filenames valid utf-8 SHOULD ITEMS [--] license file in upstream release please, query upstream to include license file [--] package translation for Non-English languages IMHO this is not needed for this package [ok] builds in mock rawhide x86_64 [ok] package functions installs, starts, runs [ok] sane scriptlets [ok] subpackage dependencies fully versioned [ok] pkgconfig files not included [ok] dependencies on packages instead of binaries [--] man pages for all binaries and scripts INITSCRIPT ISSUES [!!] separate bird and bird6 Please, create separate initscripts for bird a bird6. And move all IPv6 stuff into bird6 subpackage. It will make many administrators happy. ;-) It's done the same for example in Debian. [!!] doesn't conform with initscripts guidelines http://fedoraproject.org/wiki/Packaging:SysVInitScript - missing condrestart and try-restart - validate exit codes OTHER ISSUES [--] description It's not important, but I think the description is really short. Maybe you should point out command-line interface, soft reconfiguration and powerful language for routing (as written on upstream website). [!!] requires for post and preun You are missing requirements for post and preun scriptlets. Requires(post): chkconfig Requires(preun): chkconfig [!!] restart after upgrade, stop before removing You should condrestart the service when the package is upgraded. You should stop the service when the package is being removed. (Don't forget to add 'Requires(...): initscripts' where necessary.) SUMMARY following issuses have to be fixed: - initscript(s) - specfile: requires for scriptlets - specfile: restart/stop the service on upgrade/removal
Fixed: - New initscript based on Fedora template. - IPv4 and IPv6 versions are now in separated packages. - Added requires. - Added upgrading and removing scriptlets. - Documentation is now in separated package. SRPM and specfile are on same location.
[ok] rpmlint bird.src: W: name-repeated-in-summary C BIRD bird.x86_64: W: name-repeated-in-summary C BIRD bird.x86_64: W: no-manual-page-for-binary bird bird.x86_64: W: no-manual-page-for-binary birdc bird.x86_64: W: incoherent-subsys /etc/rc.d/init.d/bird $prog bird6.x86_64: W: no-manual-page-for-binary bird6 bird6.x86_64: W: no-manual-page-for-binary birdc6 bird6.x86_64: W: incoherent-subsys /etc/rc.d/init.d/bird6 $prog 5 packages and 0 specfiles checked; 0 errors, 8 warnings. [ok] initscripts [ok] requires [ok] start/stop on upgrade/removal All found issues were fixed.
ACCEPT.
New Package SCM Request ======================= Package Name: bird Short Description: BIRD Internet Routing Daemon Owners: jgorig Branches: f13 f14 InitialCC:
Git done (by process-git-requests).
bird-1.2.4-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/bird-1.2.4-1.fc13
bird-1.2.4-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/bird-1.2.4-1.fc14
bird-1.2.4-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update bird'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/bird-1.2.4-1.fc14
bird-1.2.5-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/bird-1.2.5-1.fc13
bird-1.2.5-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/bird-1.2.5-1.fc14
bird-1.2.5-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
bird-1.2.5-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.