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@xxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review