[PATCH 00 of 24] PYTHONHASHSEED=random

Bryan O'Sullivan bos at serpentine.com
Mon Dec 17 15:06:22 CST 2012


On Sun, Dec 16, 2012 at 2:33 PM, Mads Kiilerich <mads at kiilerich.com> wrote:

> These changes are the result of an attempt of running the test suite with
> PYTHONHASHSEED=random.
>

I really like this patch series, for the most part. Good thinking for
giving it a try, and for fixing what you found.

There are just a few patches where you changed core code that never
produces user-visible output, instead of fixing tests to use sort instead
of cat. If you could change the tests instead, that would be better.


> * Testing this way might reveal some places where Mercurial or the test
> suite
>   is fragile. Fixing such issues will make Mercurial more stable and might
>   prevent some test failures later on.
>

I kinda, sorta feel that run-tests.py should maybe always have this turned
on.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20121217/beac737e/attachment.html>


More information about the Mercurial-devel mailing list