Comment 3 for bug 890675

Revision history for this message
Barry Warsaw (barry) wrote :

Aamir, the '-' at the end of the date is left over from trying to strip out the timezone information, which will be different everywhere you run the test suite. Because that's unpredictable data, it must be removed from the test output, or you'll get the failure as you've seen. blacktav's diff in comment #1 is the more correct fix for this problem, because it correctly strips off the +- sign, which indicates whether your timezone is east or west of UTC. Because I'm west of UTC, it's negative for me. I'm guessing your east of UTC, so that's why it is positive for you.

This is definitely an easy fix, but here is my suggestion: why don't you create a branch of lp:mailman, apply blacktav's patch and run the test suite to ensure that it fixes the failure. If so, learn how to use bzr to commit the change locally, and then push your branch to Launchpad, using something like lp:~sys3mw0rm/mailman/bug-890675. Then you can use `bzr lp-propose` to submit a merge proposal into trunk. Please include a NEWS.rst file entry (see examples in that file) and please do give due credit to blacktav as well.

Once you've submitted the merge proposal, I'll review it and commit it to trunk. This will give you a great opportunity to learn a bit about the code, and how we accept contributions. Ping me on IRC if you have questions.