[Bug 240877] Review Request: archivemail - A tool for archiving and compressing old email in mailboxes

bugzilla at redhat.com bugzilla at redhat.com
Tue May 22 19:20:36 UTC 2007


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





------- Additional Comments From faucamp at csir.co.za  2007-05-22 15:20 EST -------
How about putting installing the "archivemail" file as
/usr/share/archivemail/archivemail.py (chmod 644), then writing a small Python
wrapper script to import that module and run the main() method? Something like:

------------
#!/usr/bin/python
if __name__ == '__main__':
    import sys
    sys.path.insert(0, '/usr/share/archivemail')
    from archivemail import main
    main()
------------

Not only will this solve your test_archivemail.py problem (with _little_
tweaking), it doesn't clutter up /usr/bin and would actually give you a (very)
small performance boost as the Python interpreter will only parse and process
the text of your small startup script but will use the precompiled byte-code
.pyc file of archivemail, rather than having to re-parse this large file every
time it's executed. 

Also, is it actually necessary to package the project's unit tests in the final
binary RPM? If I wanted that, i would go for the src.rpm, but maybe that's just
me...

-- 
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