[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2012-07-15 Thread Roundup Robot
Roundup Robot added the comment: New changeset ce0687a8383b by Nick Coghlan in branch 'default': Issue #9319: Remove the workaround for this since fixed problem from pydoc http://hg.python.org/cpython/rev/ce0687a8383b -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-09-22 Thread Ezio Melotti
Ezio Melotti added the comment: Can the workaround be removed from Lib/pydoc.py:2001 ? I tried to remove it from 3.2 and help('modules spam') seems to work fine. -- nosy: +ezio.melotti ___ Python tracker __

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-25 Thread STINNER Victor
STINNER Victor added the comment: > Correctly merging #9319 into 3.3? No, this issue was fixed differently in Python 3.3. I see that you reverted (bb62908896fe) this merge (except the test, which is a good idea). -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-24 Thread Jesús Cea Avión
Changes by Jesús Cea Avión : -- nosy: +jcea ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-24 Thread Roundup Robot
Roundup Robot added the comment: New changeset bb62908896fe by Jesus Cea in branch 'default': Correctly merging #9319 into 3.3? http://hg.python.org/cpython/rev/bb62908896fe -- ___ Python tracker __

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-22 Thread Roundup Robot
Roundup Robot added the comment: New changeset 701069f9888c by Victor Stinner in branch '3.2': Issue #9319: Fix the unit test http://hg.python.org/cpython/rev/701069f9888c -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-22 Thread STINNER Victor
STINNER Victor added the comment: Fixed in 3.2 (fa5e348889c2) and 3.3 (7b8d625eb6e4). The bug is a regression introduced in Python 3.2, so Python 3.1 doesn't need to be fixed. -- status: open -> closed ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-22 Thread Roundup Robot
Roundup Robot added the comment: New changeset fa5e348889c2 by Victor Stinner in branch '3.2': Issue #9319: Fix a crash on parsing a Python source code without encoding http://hg.python.org/cpython/rev/fa5e348889c2 -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-22 Thread R. David Murray
Changes by R. David Murray : -- status: closed -> open ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://ma

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-18 Thread Andreas Stührk
Andreas Stührk added the comment: How about applying the workaround patch to Python 3.2? An unprecise error message is way better than a segfault. -- ___ Python tracker ___

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-04 Thread STINNER Victor
STINNER Victor added the comment: > Hum, I'm not sure that my patch works if the locale encoding is not > UTF-8: import.c manipulates path in the filesystem encoding, whereas > PyTokenizer_FindEncodingFilename() expects UTF-8 filename. Thanks to my work on #3080, the import machinery (and other

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-04-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 7b8d625eb6e4 by Victor Stinner in branch 'default': Issue #9319: Include the filename in "Non-UTF8 code ..." syntax error. http://hg.python.org/cpython/rev/7b8d625eb6e4 -- nosy: +python-dev ___ Python tr

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-03-11 Thread STINNER Victor
STINNER Victor added the comment: I'm working on #3080 which changes the import machinery to use Unicode instead of byte strings, and so it will be possible to fix this issue correctly. -- ___ Python tracker _

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2011-03-11 Thread Łukasz Langa
Changes by Łukasz Langa : -- nosy: +lukasz.langa ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2010-12-23 Thread STINNER Victor
STINNER Victor added the comment: See also #9738 (Document the encoding of functions bytes arguments of the C API) to check which encoding is expected :-p -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2010-12-23 Thread STINNER Victor
STINNER Victor added the comment: p3k_i9313.diff is just a workaround, not the correct fix. The problem is that PyTokenizer_FindEncoding() doesn't get the filename. I wrote tokenizer_encoding_filename.patch which add PyTokenizer_FindEncodingFilename() and patch import.c and traceback.c to pas

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2010-12-15 Thread Andreas Stührk
Changes by Andreas Stührk : -- nosy: +Trundle ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2010-12-15 Thread Ron Adam
Ron Adam added the comment: Pydoc skips the badsysntax_pep3120 file for now. When this gets fixed that workaround should be removed. The work around is commented and refers to this issue #. -- ___ Python tracker

[issue9319] imp.find_module('test/badsyntax_pep3120') causes segfault

2010-12-15 Thread R. David Murray
R. David Murray added the comment: help no longer segfaults, but the find_module call still does; updating title. The patch does cure the segfault, but as Stefan says it isn't the best fix since having '' in the error message instead of the real file name isn't very useful. -- nosy: