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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          QAContact|fedora-extras-              |fedora-package-
                   |list@xxxxxxxxxx             |review@xxxxxxxxxx

shahms@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|bugzilla-sink@xxxxxxxxxxxxx |shahms@xxxxxxxxxx
OtherBugsDependingO|163776                      |163778
              nThis|                            |




------- Additional Comments From shahms@xxxxxxxxxx  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.


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]