Spec URL: http://sdz.fedorapeople.org/rpmbuild/zyx-liveinstaller.spec SRPM URL: http://sdz.fedorapeople.org/rpmbuild/zyx-liveinstaller-0.1.9-2.fc11.src.rpm Koji Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1607892 [sebastian@localhost noarch]$ rpmlint ./zyx-liveinstaller-0.1.10-1.fc11.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [sebastian@localhost SRPMS]$ rpmlint zyx-liveinstaller-0.1.10-1.fc11.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
Ah, got the wrong link to the source RPM. This one should be better: http://sdz.fedorapeople.org/rpmbuild/zyx-liveinstaller-0.1.10-1.fc11.src.rpm
url is incorrect, should be http://filteredperception.org/smiley/projects/zyx-rli/
(In reply to comment #2) > url is incorrect, should be > http://filteredperception.org/smiley/projects/zyx-rli/ url is now correct, I fixed the website :) -dmc
== Review == Good: - rpmlint checks return nothing - package meets naming guidelines - package meets packaging guidelines - license (GPLv3) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream (feeaff2f9078bf86ad52a5b15cafbb528cf90e8e) - package compiles on devel (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - .desktop file OK Nice package, approved.
Quick note: Unless you are branching this for EPEL, you don't need to define the buildroot or remove it in %install section anymore.
New Package CVS Request ======================= Package Name: zyx-liveinstaller Short Description: Install a running LiveOS rebootlessly Owners: sdz Branches: F-11
CVS done
@ Spot: Thanks for the review! :) @ Rahul: Won't this have rpmlint complaining on the SRPM?
Yes, it it will. I have filed and got it changed from an error to just a warning and a warning is still useful since packages do get branched for EPEL but remember that our guidelines must be given higher weight than rpmlint always.
zyx-liveinstaller-0.1.10-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/zyx-liveinstaller-0.1.10-2.fc11
Yup, agreed. I fixed it before building. It should be on its way on his way to Rawhide now...
zyx-liveinstaller-0.1.10-2.fc11 has been pushed to the Fedora 11 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 zyx-liveinstaller'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-8926
zyx-liveinstaller-0.1.10-2.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.