[Bug 456042] Review Request: python-mwlib - MediaWiki conversion library for Python
bugzilla at redhat.com
bugzilla at redhat.com
Thu Jul 24 08:28:23 UTC 2008
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-mwlib - MediaWiki conversion library for Python
https://bugzilla.redhat.com/show_bug.cgi?id=456042
rickyz at cmu.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-review+
------- Additional Comments From rickyz at cmu.edu 2008-07-24 04:28 EST -------
Build cleanly in mock.
* rpmlint clean
* package meets naming guidelines
* uses macros consistently
* build root is ok
X license isn't completely correct, mwlib/cdb.py was taken from the spambayes
project, and should be under the Python license (include both in the license tag
and add a comment explaining it)
* Requires/BuildRequires are OK
X The %check section currently doesn't run the tests. I don't see any docs on
how to run them (and there's a chance that it might require py.test, which isn't
in Fedora), so it might be best to kill the section entirely.
* Installs/uninstalls fine
* Permissions are fine
APPROVED (just fix the above issues)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the Fedora-package-review
mailing list