[issue14583] try/except import fails --without-threads

2012-05-07 Thread Roundup Robot
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset d6324941b739 by Antoine Pitrou in branch 'default': Issue #14583: Fix importlib bug when a package's __init__.py would first import one of its modules then raise an error.

[issue14583] try/except import fails --without-threads

2012-05-07 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: Should be fixed now. The buildbot has been able to launch the test suite. -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Nadeem Vawda
Changes by Nadeem Vawda nadeem.va...@gmail.com: -- nosy: +nadeem.vawda ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14583 ___ ___

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: Here is a patch. The __import__ function's crazy API never ceases to amaze me. -- keywords: +patch nosy: +pitrou Added file: http://bugs.python.org/file25481/impstuff.patch ___ Python tracker

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Changes by Antoine Pitrou pit...@free.fr: Removed file: http://bugs.python.org/file25481/impstuff.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14583 ___

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: Oops, there was a duplicate test. -- stage: needs patch - patch review Added file: http://bugs.python.org/file25482/impstuff.patch ___ Python tracker rep...@bugs.python.org

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Brett Cannon
Brett Cannon br...@python.org added the comment: Only two comments, otherwise LGTM (and I can't believe the solution was to go back through the import system just to pull out the cached module; the things we would change if we were doing this from scratch). One, you have some XXX False

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: One, you have some XXX False markers in the tests. Should those get deleted or replaced with something? Well, I don't know what to replace them with. I would have expected pkg.module to end up in sys.modules, but as mentioned in the comments

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: One, you have some XXX False markers in the tests. Should those get deleted or replaced with something? Well, I don't know what to replace them with. I would have expected pkg.module to end up in sys.modules, but as mentioned in the

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Brett Cannon
Brett Cannon br...@python.org added the comment: So I was going to try to figure out the logic, so I manually created the test files to start debugging, but I didn't get the ImportError but instead the 1/0 error for the relative import. Maybe it's specific to lack of threads or the change you

[issue14583] try/except import fails --without-threads

2012-05-06 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: So I was going to try to figure out the logic, so I manually created the test files to start debugging, but I didn't get the ImportError but instead the 1/0 error for the relative import. Maybe it's specific to lack of threads or the change

[issue14583] try/except import fails --without-threads

2012-04-27 Thread Stefan Krah
Stefan Krah stefan-use...@bytereef.org added the comment: This issue is now apparently causing a segfault: (gdb) r ./Tools/scripts/run_tests.py -j 1 -u all -W --timeout=3600 Starting program: /home/stefan/pydev/cpython/python ./Tools/scripts/run_tests.py -j 1 -u all -W --timeout=3600 [...]

[issue14583] try/except import fails --without-threads

2012-04-16 Thread Stefan Krah
Stefan Krah stefan-use...@bytereef.org added the comment: The SystemError has changed to a KeyError: http://www.python.org/dev/buildbot/all/builders/AMD64%20Fedora%20without%20threads%203.x/builds/2004/steps/test/logs/stdio At least that makes it easy to spot the location in import.c. The

[issue14583] try/except import fails --without-threads

2012-04-16 Thread Brett Cannon
Brett Cannon br...@python.org added the comment: Just to clarify the failure for the bug history, somehow multiprocessing is not ending up in sys.modules as expected. It changed from a SystemError to a KeyError because I started to properly check a PyDict_GetItem() return value instead of

[issue14583] try/except import fails --without-threads

2012-04-16 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- nosy: +Arfrever ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14583 ___

[issue14583] try/except import fails --without-threads

2012-04-14 Thread Stefan Krah
New submission from Stefan Krah stefan-use...@bytereef.org: In the build --without-threads, catching an ImportError in support.py fails. Fedora buildbot: http://www.python.org/dev/buildbot/all/builders/AMD64%20Fedora%20without%20threads%203.x/builds/1986/steps/test/logs/stdio ./python

[issue14583] try/except import fails --without-threads

2012-04-14 Thread Eric Snow
Changes by Eric Snow ericsnowcurren...@gmail.com: -- nosy: +eric.snow ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14583 ___ ___ Python-bugs-list