On Mon, 25 Apr 2011 04:47:03 +0200
Jesus Cea <[email protected]> wrote:
>
> And yes, I fully realized that I should try to compile locally first.
> Dealing with this unexpected merge when merging my own patch was...
> unexpected, and the code seemed sensible enough.
You should *always* recompile and run the affected tests before checking
in a change. Even if the changes look "trivial".
By trying to save a little time on your side your may lose a lot of
other people's time.
> Do we have some hat-of-shame I should wear because breaking the build? :).
The tests are still broken it seems:
======================================================================
ERROR: test_issue9319 (test.test_imp.ImportTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File
"/Users/pythonbuildbot/buildarea/3.x.hansen-osx-x86-2/build/Lib/test/test_imp.py",
line 181, in test_issue9319
imp.find_module, "test/badsyntax_pep3120")
File
"/Users/pythonbuildbot/buildarea/3.x.hansen-osx-x86-2/build/Lib/unittest/case.py",
line 574, in assertRaises
callableObj(*args, **kwargs)
ImportError: No module named 'test/badsyntax_pep3120'
Regards
Antoine.
_______________________________________________
Python-Dev mailing list
[email protected]
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe:
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com