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.
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
Changes by Nadeem Vawda nadeem.va...@gmail.com:
--
nosy: +nadeem.vawda
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14583
___
___
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
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
___
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
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
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
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
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
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
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
[...]
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
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
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14583
___
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
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
nosy: +eric.snow
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14583
___
___
Python-bugs-list
17 matches
Mail list logo