Spec URL: http://fab.fedorapeople.org/packages/SRPMS/screenie.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/screenie-1.30.0-1.fc9.src.rpm Description: Screenie is a small and lightweight screen wrapper that is designed to be a session handler that simplifies the process of administrating detached jobs by providing an interactive menu. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1022715 rpmlint output: [fab@laptop024 noarch]$ rpmlint screenie-1.30.0-1.fc9.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [fab@laptop024 SRPMS]$ rpmlint screenie-1.30.0-1.fc9.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
You must add awk and screen as Requires, rpmbuild does not pick them and without them screenie bails out. sort is also a required program, but it is provided by coreutils and I guess it's safe enough to assume that coreutils is installed on any system where screenie would be used Everything else seems fine, package is too simple (it's just a shell script) to include a full review. - RPM name is OK - rpmlint is empty - license is OK (GPLv2+) - source screenie_1.30.0.orig.tar.gz is the same as upstream 1f04b055c30f5f8e4f35160bf50cfa060472a61a screenie_1.30.0.orig.tar.gz - builds fine in mock (devel/x86_64)) and being a simple shell script it's safe to assume it will work on any arch - file list is OK, even timestamps are preserved - works OK (when awk and screen are installed!) ====== Package APPROVED but please add manual requires for awk and screen ======
Thanks Manuel for the review. I will add the missing 'Requires' before cvs import. BTW, happy new year
New Package CVS Request ======================= Package Name: screenie Short Description: A small and lightweight screen wrapper Owners: fab Branches: F-9 F-10 InitialCC:
cvs done.
screenie-1.30.0-2.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/screenie-1.30.0-2.fc9
screenie-1.30.0-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/screenie-1.30.0-2.fc10
screenie-1.30.0-2.fc10 has been pushed to the Fedora 10 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 screenie'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-0069
screenie-1.30.0-2.fc9 has been pushed to the Fedora 9 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-newkey update screenie'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-0162
I'm switching the review flag on again, because this package should not have passed. It is causing broken deps in rawhide: http://koji.fedoraproject.org/mash/rawhide-20090108/logs/depcheck screenie-1.30.0-2.fc11.noarch requires awk This is due to a bogus "Requires: awk". There is no such package in the Fedora collection, there is only "gawk". and I'm surprised that this wasn't picked up here, it suggests that maybe an install attempt was never made. Actually it seems to be a reviewer error from comment #1, suggesting "awk" rather than "gawk". Nevertheless the packager should have verified that it installed on their local system before applying a suggested "Requires". Please fix in rawhide (and in pending bodhi requests).
Thanks for pointing this out. This is my fault. The reviewer discovered the issue and I fixed it the wrong way.
http://cvs.fedoraproject.org/viewvc/rpms/screenie/devel/screenie.spec?r1=1.1&r2=1.2&pathrev=screenie-1_30_0-3_fc11
screenie-1.30.0-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/screenie-1.30.0-3.fc10
screenie-1.30.0-3.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/screenie-1.30.0-3.fc9
screenie-1.30.0-3.fc10 has been pushed to the Fedora 10 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 screenie'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-0408
screenie-1.30.0-3.fc9 has been pushed to the Fedora 9 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-newkey update screenie'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-0493
screenie-1.30.0-3.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
screenie-1.30.0-3.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.