Bug 465646 (python-hachoir-core)
Summary: | Review Request: python-hachoir-core - Hachoir core parser | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Jean-François Martin <lokthare> |
Component: | Package Review | Assignee: | Orcan Ogetbil <oget.fedora> |
Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting, oget.fedora |
Target Milestone: | --- | Flags: | oget.fedora:
fedora-review-
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-10-17 19:51:00 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 201449, 465649 |
Description
Jean-François Martin
2008-10-05 05:00:49 UTC
I reviewed this package. Here are my notes : * The package must obey python modules naming guidelines: Please see: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28python_modules.29 And the spec file will need to be renamed accordingly. * Latest version is not packaged. Please update to the latest version. * You should probably make a %check section and run the tests given in the test_doc.py * There is a doc directory in the tarball. Should (at least, some of) the contents of that directory be packaged? * You must have BuildRequires: python-setuptools-devel Please see: http://fedoraproject.org/wiki/Packaging/Python/Eggs Spec URL: http://lokthare.fedorapeople.org/temp/python-hachoir-core.spec SRPM URL: http://lokthare.fedorapeople.org/temp/python-hachoir-core-1.2-1.fc9.src.rpm - Update to new release - Rename the package to match naming guidelines for python modules - Add some files to the doc - Run test Sorry, give a wrong URL for the SRPM. SRPM URL : http://lokthare.fedorapeople.org/temp/python-hachoir-core-1.2.1-1.fc10.src.rpm Thanks for the update. We need to have one thing corrected and I also have a suggestion: * I guess I should have been a little clearer. The python guidelines are slightly different for non-setuptools packages. You are using the guidelines as if the building is done by setuptools. Please check the setup.py file. In order to use the setuptools you have to pass the --setuptools flag to setup.py (in both %build and %install). OR alternatively, you can follow the non-setuptools guidelines at http://fedoraproject.org/wiki/Packaging/Python/Eggs but this will require a somewhat larger modification. ? Having a closer look, I think you should include all of doc/* in %doc. Those are files for use of graphviz and dia, and they might be useful for some people. ping? re-ping? is there anything wrong? third ping? I'll close this bug if there is no response within 4 weeks. Looks like we got a dead review here. Closing WONTFIX. |