Bug 661354

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.