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: archivemail - A tool for archiving and compressing old email in mailboxes https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240877 Jochen@xxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag| |fedora-review? ------- Additional Comments From Jochen@xxxxxxxxxxxxxxx 2007-05-22 13:20 EST ------- Good: + Naming seems ok. + License seems ok. + Tar ball matches with upstream + Rpmlint is quite on source package + Rpmlint is quite on binary package + Pakcage contains verbain copy of the license text + Mock build works fine. Bad: - Strangs Build stanza: You make a 'chmod 0644' on setup.py and test_archivmail.py, but you didn't call any of this python scripts. - Calling of the text script shows following error message: python test_archivemail.py The archivemail script needs to be called 'archivemail.py' and should be in the current directory in order to be imported and tested. Sorry. -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review