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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?




------- Additional Comments From [EMAIL PROTECTED]  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
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to