Changes by Brett Cannon br...@python.org:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
___
Python-bugs-list
Roundup Robot added the comment:
New changeset d5aa922f97f9 by Brett Cannon in branch '3.3':
Issue #16880: _imp.load_dynamic() is not defined on a platform that
http://hg.python.org/cpython/rev/d5aa922f97f9
--
nosy: +python-dev
___
Python tracker
Brett Cannon added the comment:
And this was merged into default in
http://hg.python.org/cpython/rev/3c3c9ad7c297 but for some reason it didn't
show up attached to this issue.
--
___
Python tracker rep...@bugs.python.org
Jeffrey Armstrong added the comment:
Is Christian's patch going to be sufficient for the time being? Just curious.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Brett Cannon added the comment:
Yes, Christian's patch should do the trick for fixing the problem you reported,
Jeffrey.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Jeffrey Armstrong added the comment:
Sorry, I think my question was a bit vague. Christian's patch does, in fact,
work fine for fixing the problem as reported. I was wondering if the patch was
sufficient to close the bug with a commit. I didn't know if other work was
ongoing to close this
Brett Cannon added the comment:
A commit to Python 3.3 and 3.4 would be enough to close this bug. Just a matter
of Christian or someone else finding the time.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
Brett Cannon added the comment:
load_dynamic should probably be documented since it does something you can't do
on your own and importlib itself uses it.
As for the exception test, it should be to make sure ImportError is raised
(i.e. the 'else' clause is hit).
--
Nick Coghlan added the comment:
The misleading docs Terry pointed out should also be fixed to note that imp
is still used to expose some functionality where importlib really does need
help from the underlying platform (such as loading dynamic modules).
--
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss masege
On 1/13/13, Nick Coghlan rep...@bugs.python.org wrote:
Nick Coghlan added the comment:
The misleading docs Terry pointed out should also be fixed to note that
imp is still used to expose some functionality where
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss masege
On 1/13/13, Nurhusien Hasen nurhusien.hase...@gmail.com wrote:
Pleas stop your from me all isuss masege
On 1/13/13, Nick Coghlan rep...@bugs.python.org wrote:
Nick Coghlan added the comment:
The misleading docs
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss maseges
On 1/13/13, Nurhusien Hasen rep...@bugs.python.org wrote:
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss masege
On 1/13/13, Nurhusien Hasen nurhusien.hase...@gmail.com wrote:
Pleas stop
Nurhusien Hasen added the comment:
On 1/13/13, Nurhusien Hasen rep...@bugs.python.org wrote:
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss maseges
On 1/13/13, Nurhusien Hasen rep...@bugs.python.org wrote:
Nurhusien Hasen added the comment:
Pleas stop your from me
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss maseges pleas stop
On 1/13/13, Nurhusien Hasen nurhusien.hase...@gmail.com wrote:
On 1/13/13, Nurhusien Hasen rep...@bugs.python.org wrote:
Nurhusien Hasen added the comment:
Pleas stop your from me all isuss maseges
On
Changes by Terry J. Reedy tjre...@udel.edu:
--
Removed message: http://bugs.python.org/msg179807
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
Removed message: http://bugs.python.org/msg179806
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
nosy: -Nurhusien2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
___
Python-bugs-list
Changes by Terry J. Reedy tjre...@udel.edu:
--
Removed message: http://bugs.python.org/msg179810
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
Removed message: http://bugs.python.org/msg179814
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
Removed message: http://bugs.python.org/msg179816
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Terry J. Reedy added the comment:
Given the current (3.3) imp.py docstring
This module provides the components needed to build your own __import__
function. Undocumented functions are obsolete.
In most cases it is preferred you consider using the importlib module's
functionality over this
Jeffrey Armstrong added the comment:
Terry: If I'm understanding this correctly, C_EXTENSION implies a dynamically
loaded module authored in C. The issue on FreeMiNT, the operating system in
question, is that there is no support for true shared objects. Any C modules
must be statically
New submission from Jeffrey Armstrong:
On a platform where dynamic loading is unsupported, the function
imp_load_dynamic is not compiled (Python/import.c:1775), and the Python
function load_dynamic (Python/import.c:1845) will not be included in the _imp
module. However, Lib/imp.py attempts
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +brett.cannon, ncoghlan
stage: - needs patch
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Christian Heimes added the comment:
The attached patch should fix the issue. Can you give it a try, please?
--
assignee: - christian.heimes
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
Changes by Christian Heimes li...@cheimes.de:
--
keywords: +patch
Added file: http://bugs.python.org/file28592/imp_dynamic.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Brett Cannon added the comment:
LGTM
--
stage: needs patch - commit review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
___
Brett Cannon added the comment:
I should say LGTM in terms of looking at the patch, not actually trying it out.
=)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16880
___
Brett Cannon added the comment:
I did just realize, though, there is no test for temporarily setting
load_dynamic() to None in imp.py to make sure the expected exception is raised.
--
___
Python tracker rep...@bugs.python.org
Jeffrey Armstrong added the comment:
The patch works with respect to allowing the interpreter to start, and
importing modules from the interpreter seems to be working fine. I can't speak
to Brett's concerns about possible side effects of setting load_dynamic to None.
--
30 matches
Mail list logo
