[Bug 186002] Review Request: python-archmod - Extensible reader/decompiler of files in CHM format

bugzilla at redhat.com bugzilla at redhat.com
Tue Apr 18 17:40:16 UTC 2006


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-archmod - Extensible reader/decompiler of files in CHM format


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=186002


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          QAContact|fedora-extras-              |fedora-package-
                   |list at redhat.com             |review at redhat.com

shahms at shahms.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|bugzilla-sink at leemhuis.info |shahms at shahms.com
OtherBugsDependingO|163776                      |163778
              nThis|                            |




------- Additional Comments From shahms at shahms.com  2006-04-18 13:40 EST -------
Good:
 - rpmlint gives no errors
 - builds in mach just fine
 - spec file and package name match (but see below)
 - GPL license, license included, etc.
 - md5sum matches upstream
 - it meets the rest of the MUST clauses

The only "bad" thing I could find is that I don't think python-archmod is the
most appropriate name as the python module itself is secondary to the binary
functionality, if I'm not mistaken. It seems similar in this regard to bzr and
yum, and should be named appropriately.

Other than that, the package looks good.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list