Bug 577202

Taking for review as Tryton needs it.

formal review is here, see the notes explaining OK* and BAD statuses below:

OK      source files match upstream:
            2fb091e021147bb404bcf019876a61cab28814ef  pycha-0.5.2.tar.gz
OK      package meets naming and versioning guidelines.
OK      specfile is properly named, is cleanly written and uses macros consistently.
OK      dist tag is present.
BAD     license field matches the actual license.
OK      license is open source-compatible (LGPLv3+). License text included in package.
OK      latest version is being packaged.
OK*     BuildRequires are proper.
N/A     compiler flags are appropriate.
OK      %clean is present.
OK      package builds in mock (Rawhide/x86_64).
N/A     debuginfo package looks complete.
BAD     rpmlint is silent.
BAD     final provides and requires look sane.
OK      %check is present and all tests pass.
OK      no shared libraries are added to the regular linker search paths.
OK      owns the directories it creates.
OK      doesn't own any directories it shouldn't.
OK      no duplicates in %files.
OK      file permissions are appropriate.
OK      no scriptlets present.
OK      code, not content.
OK      documentation is small, so no -docs subpackage is necessary.
OK      %docs are not necessary for the proper functioning of the package.
OK      no headers.
OK      no pkgconfig files.
OK      no libtool .la droppings.
OK      is a GUI app with a desktop file

- the license is LGPLv3+ because the licensing header contains the "or any later vesion" part
- is BR: python-devel really required? Only modules with C code should need it.
- rpmlint complains a bit:
chavier.noarch: W: spelling-error Summary(en_US) pycha -> psych, Pynchon, Psyche
chavier.noarch: W: summary-not-capitalized C a GUI application for exploring the pycha library
    => I would even shorten it to "Application for exploring the pycha library"
chavier.noarch: W: spelling-error %description -l en_US datasets -> data sets, data-sets, databases
    => should be fixed
chavier.noarch: W: spelling-error %description -l en_US pycha -> psych, Pynchon, Psyche
chavier.noarch: W: no-documentation
    => the "pycha" check can be ignored
- pycairo must be set as Requires, it is imported by few files in the pycha module
- I would not include the tests as %doc, examples should be enough

Thanks a lot for your review!

> - the license is LGPLv3+ because the licensing header contains the "or any
> later vesion" part

fixed

> - is BR: python-devel really required? Only modules with C code should need it.

yeah, AFAIK it's required as per: https://fedoraproject.org/wiki/Packaging:Python#BuildRequires
It also fails to build without it: http://koji.fedoraproject.org/koji/taskinfo?taskID=2078411

I changed it to python2-devel instead of python-devel though.

> - rpmlint complains a bit:
> chavier.noarch: W: spelling-error Summary(en_US) pycha -> psych, Pynchon,
> Psyche
> chavier.noarch: W: summary-not-capitalized C a GUI application for exploring
> the pycha library
>     => I would even shorten it to "Application for exploring the pycha library"
I changed it to "GUI application for exploring the pycha library". I feel that the fact that it has a GUI is important.

> chavier.noarch: W: spelling-error %description -l en_US datasets -> data sets,
> data-sets, databases
>     => should be fixed

fixed

> chavier.noarch: W: spelling-error %description -l en_US pycha -> psych,
> Pynchon, Psyche
> chavier.noarch: W: no-documentation
>     => the "pycha" check can be ignored
> - pycairo must be set as Requires, it is imported by few files in the pycha
> module

fixed

> - I would not include the tests as %doc, examples should be enough

removed tests

* Sat Mar 27 2010 Ionuț C. Arțăriși <mapleoin> - 0.5.2-2
- changed license from LGPLv3 to LGPLv3+
- fixed spelling error in %%description
- capitalized chavier package %%summary
- added pycairo Requires
- removed tests

http://mapleoin.fedorapeople.org/pkgs/python-pycha/python-pycha.spec
http://mapleoin.fedorapeople.org/pkgs/python-pycha/python-pycha-0.5.2-2.fc12.src.rpm

All issues are resolved and the package is APPROVED.

If you don't plan to maintain pycha in EPEL, please ask for at least EL-5 branch and set me (=sharkcz) as comaintainer in the CVS request. I will then take care of it.

New Package CVS Request
=======================
Package Name: python-pycha
Short Description: A library for drawing charts with Python and Cairo
Owners: mapleoin sharkcz
Branches: F-12 F-13 EL-5


Sure, EL-5 is all yours. Thanks again!

CVS done (by process-cvs-requests.py).

python-pycha-0.5.2-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/python-pycha-0.5.2-2.fc12

Unfortunately pycairo is too old in RHEL5 too, so I will continue with EL-6.

python-pycha-0.5.2-2.fc12 has been pushed to the Fedora 12 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 python-pycha'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/python-pycha-0.5.2-2.fc12

python-pycha-0.5.2-2.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.