Spec URL: http://smilner.fedorapeople.org/identicurse/identicurse.spec SRPM URL: http://smilner.fedorapeople.org/identicurse/identicurse-0.4.1-1.fc14.src.rpm Description: A simple but powerful Identi.ca client with a curses-based UI.
Upstream: http://identicurse.net/ rpmlint: $ rpmlint /home/steve/rpmbuild/RPMS/noarch/identicurse-0.4.1-1.fc14.noarch.rpm identicurse.spec identicurse.noarch: W: no-manual-page-for-binary identicurse 1 packages and 1 specfiles checked; 0 errors, 1 warnings. $
Okay, I'll take this. Some quick drive-by comments from a first glance at the spec: - BuildRequires needs to be on python2-devel, as it's a python2 package as far as I can see some cosmetical stuff: - you don't actually need to define the python_sitearch, RPM does that automatically now (assuming you won't build this on Fedora 12 or even lower) - you can leave the BuildRoot tag out as well and don't need to clean the buildroot before %install now - you can leave out the %clean section
Sounds good, I'll make those changes now.
Updated for all above items. Note that rpmlint does spit out warnings on the cosmetic changes but I assume it just has not caught up yet :-) Spec URL: http://smilner.fedorapeople.org/identicurse/identicurse.spec SRPM URL: http://smilner.fedorapeople.org/identicurse/identicurse-0.4.1-2.fc14.src.rpm
(In reply to comment #4) > Updated for all above items. Note that rpmlint does spit out warnings on the > cosmetic changes but I assume it just has not caught up yet :-) Yup, that's correct, and therefore rpmlint has a couple of things to complain about, but they're not relevant: [makerpm@Julians-Notebook rpmbuild]$ rpmlint ./SPECS/identicurse.spec SRPMS/identicurse-0.4.1-2.fc14.src.rpm ./RPMS/noarch/identicurse-0.4.1-2.fc14.noarch.rpm ./SPECS/identicurse.spec: W: no-cleaning-of-buildroot %install ./SPECS/identicurse.spec: W: no-cleaning-of-buildroot %clean ./SPECS/identicurse.spec: W: no-buildroot-tag ./SPECS/identicurse.spec: W: no-%clean-section identicurse.src: W: no-cleaning-of-buildroot %install identicurse.src: W: no-cleaning-of-buildroot %clean identicurse.src: W: no-buildroot-tag identicurse.src: W: no-%clean-section identicurse.noarch: W: no-manual-page-for-binary identicurse 2 packages and 1 specfiles checked; 0 errors, 9 warnings. [x] Package is named according to the Package Naming Guidelines. [1] [x] Spec file name must match the base package %{name}, in the format %{name}.spec. [x] Spec file is legible and written in American English. [x] Spec file lacks Packager, Vendor, PreReq tags. [x] Spec uses macros instead of hard-coded directory names. [x] Package consistently uses macros. [x] Macros in Summary, %description expandable at SRPM build time. [x] PreReq is not used. [x] Requires correct, justified where necessary. [x] All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [2] [x] The spec file handles locales properly. [x] Changelog in prescribed format. Mock builds the package just fine. I also saw the patch you use is already applied in upstream trunk, awesome. It wouldn't hurt to list %{_python_sitelib}/%{name}/ instead of just %{_python_sitelib}/* in %files though. Apart from that it looks all very sane.
Excellent. I'll make that change as well.
Spec URL: http://smilner.fedorapeople.org/identicurse/identicurse.spec SRPM URL: http://smilner.fedorapeople.org/identicurse/identicurse-0.4.1-3.fc14.src.rpm Change: -%{_python_sitelib}/* +%{python_sitelib}/%{name}/ +%{python_sitelib}/%{name}-*.egg-info/
Awesome, this package is alright. APPROVED
New Package SCM Request ======================= Package Name: identicurse Short Description: A simple but powerful Identi.ca client with a curses-based UI. Owners: smilner Branches: f14
Git done (by process-git-requests).
identicurse-0.4.1-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/identicurse-0.4.1-3.fc14
identicurse-0.4.1-3.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 identicurse'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/identicurse-0.4.1-3.fc14
identicurse-0.5.1-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 identicurse'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/identicurse-0.5.1-1.fc14
identicurse-0.5.1-2.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 identicurse'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/identicurse-0.5.1-2.fc14
identicurse-0.6.2-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 identicurse'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/identicurse-0.6.2-1.fc14
identicurse-0.6.2-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.