Bug 614299

This is my first package, and I am seeking a sponsor.

If you could review someone else's package so that I have more to evaluate you on I'd appreciate it.  Here's a preliminary review:

NEEDSWORK:
* There is an upstream tarball for this at pypi:
  http://pypi.python.org/pypi/ordereddict
  - The tarball in the srpm does not match the tarball on pypi
* We should be sticking to 80 columns or less in spec files.  If the Summary is longer than 80 columns, chances are that it's too long.
* %define should be changed to %global as %define inside of %{} is only working due to an rpm bug.

You can reach me on IRC: abadger1999 on freenode.net, email toshio fedoraproject.org, or just CC me on a review request that you take.

(James is already sponsored now as a packager.)

Good:
* Named according to Guidelines
* License is MIT in source file and spec
* Spec file is readable
* No locales to process
* No shared libraries to process
* No bundled libraries
* Not relocatable
* Package owns all directories that it creates
* Files are not listed twice
* Permissions set correctly
* Macros used consistently
* Code, not content
* No doc files affect runtime
* Not a GUI package
* Does not own files from other packages
* All filenames utf-8
* Builds in koji with %doc removal and upstream's tarball

Needswork:
* Source does not match upstream.   There is an upstream tarball for this at pypi:
  http://pypi.python.org/pypi/ordereddict
  - The tarball in the srpm does not match the tarball on pypi
* We should be sticking to 80 columns or less in spec files.  If the Summary is
longer than 80 columns, chances are that it's too long.
* %define should be changed to %global as %define inside of %{} is only working
due to an rpm bug.

rpmlint:
python-ordereddict.noarch: W: summary-ended-with-dot C A drop-in substitute for Py2.7's new collections.OrderedDict that works in Python 2.4-2.6.
python-ordereddict.noarch: E: summary-too-long C A drop-in substitute for Py2.7's new collections.OrderedDict that works in Python 2.4-2.6.

Don't need the period at the end, "A " at the beginning  Make it shorter.  Something like "Implementation of Python 2.7's OrderedDict"

python-ordereddict.noarch: W: spelling-error %description -l en_US lookup -> lockup, hookup, look up
python-ordereddict.noarch: W: spelling-error %description -l en_US repr -> rept, rep, repro

These are fine.  Both are spelled correctly in their domain.

python-ordereddict.noarch: E: description-line-too-long C Drop-in substitute for Py2.7's new collections.OrderedDict. The recipe has big-oh performance that matches regular dictionaries (amortized O(1) insertion/deletion/lookup and O(n) iteration/repr/copy/equality_testing).

Noted above.

python-ordereddict.noarch: W: no-documentation

If upstream provides no documentation we don't need to ship any

2 packages and 0 specfiles checked; 4 errors, 7 warnings.


Notes:
* Should query upstream to include license text

Thanks for you comment, i just update the spec file base on your comments and put it in my fedorapeople account. It pass the rpmlint without errors now. I also sent a requirement to upstream maintainer to ask him include the license text in the source. I have to wait his feedback.

When you make changes to the spec file, you need to increment the Release: field and add a changelog entry.

You actually don't need to wait on his feedback.  Shipping a license file is not a requirement as long as the license is clear from what *is* in the tarball.  There's only one code file in the tarball and that file has the MIT license text so it's clear what the status is.  You can update to upstream's tarball without issue.

Thanks, just increment the release field and add a changelog entry. Also update to using upstream's tarball.

Spec URL:
http://jamesni.fedorapeople.org/python-ordereddict/python-ordereddict.spec

SRPM URL:
http://jamesni.fedorapeople.org/python-ordereddict/python-ordereddict-1.1-2.fc13.src.rpm

remaining issues fixed.  APPROVED.

Note:   looks like upstream updated the tarball in place.  There's a new upstream 1.1 tarball with a different md5sum and the license file included.  I verified the tarball in the srpm matched the old tarball.  You might want to update to the new tarball before importing the package.

Ping.  you can go ahead with the SCM Request and getting the package imported now.

I'd love to see this in EPEL5/6 as well as Fedora if you're interested in maintaining it there.

https://fedoraproject.org/wiki/Package_SCM_admin_requests#New_Packages

New Package SCM Request
=======================
Package Name: python-ordereddict
Short Description: OrderedDict that works in Python 2.4-2.6
Owners: jamesni
Branches: f13 f14 el5 el6
InitialCC: jamesni

New Package SCM Request
=======================
Package Name: python-ordereddict
Short Description: OrderedDict that works in Python 2.4-2.6
Owners: jamesni
Branches: f13 el5 el6
InitialCC: jamesni

Git done (by process-git-requests).

I see that you've built package in koji for el5 and f13 but not submitted them to the download repositories.

https://koji.fedoraproject.org/koji/packageinfo?packageID=11231

(The el5 and f13 packages are tagged into *-candidate right now.)

You can submit them to the stable repos in bodhi:

https://admin.fedoraproject.org/updates
http://fedoraproject.org/wiki/Bodhi_Guide#Workflow

You can also close this bug as the review is complete.

Hi, I have built package in koji and sumbit them to the stable repos. You can use yum to install it now. This bug will be closed.