[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Brett Cannon : -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 __

[issue16880] Importing "imp" will fail if dynamic loading not supported

Brett Cannon added the comment: In honour of Jeffrey's lightning talk at PyCon 2013, I just committed the fix! -- assignee: christian.heimes -> brett.cannon resolution: -> fixed stage: commit review -> committed/rejected ___ Python tracker

[issue16880] Importing "imp" will fail if dynamic loading not supported

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

[issue16880] Importing "imp" will fail if dynamic loading not supported

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

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 i

[issue16880] Importing "imp" will fail if dynamic loading not supported

Brett Cannon added the comment: Yes, Christian's patch should do the trick for fixing the problem you reported, Jeffrey. -- ___ Python tracker ___ __

[issue16880] Importing "imp" will fail if dynamic loading not supported

Jeffrey Armstrong added the comment: Is Christian's patch going to be sufficient for the time being? Just curious. -- ___ Python tracker ___

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- Removed message: http://bugs.python.org/msg179816 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- Removed message: http://bugs.python.org/msg179814 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- Removed message: http://bugs.python.org/msg179810 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- nosy: -Nurhusien2 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- Removed message: http://bugs.python.org/msg179806 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Terry J. Reedy : -- Removed message: http://bugs.python.org/msg179807 ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue16880] Importing "imp" will fail if dynamic loading not supported

Nurhusien Hasen added the comment: Pleas stop your from me all isuss maseges pleas stop On 1/13/13, Nurhusien Hasen wrote: > On 1/13/13, Nurhusien Hasen wrote: >> >> Nurhusien Hasen added the comment: >> >> Pleas stop your from me all isuss maseges >> >> On 1/13/13, Nurhusien Hasen wrote: >>>

[issue16880] Importing "imp" will fail if dynamic loading not supported

Nurhusien Hasen added the comment: On 1/13/13, Nurhusien Hasen wrote: > > Nurhusien Hasen added the comment: > > Pleas stop your from me all isuss maseges > > On 1/13/13, Nurhusien Hasen wrote: >> >> Nurhusien Hasen added the comment: >> >> Pleas stop your from me all isuss masege >> >> On 1/13

[issue16880] Importing "imp" will fail if dynamic loading not supported

Nurhusien Hasen added the comment: Pleas stop your from me all isuss maseges On 1/13/13, Nurhusien Hasen wrote: > > Nurhusien Hasen added the comment: > > Pleas stop your from me all isuss masege > > On 1/13/13, Nurhusien Hasen wrote: >> Pleas stop your from me all isuss masege >> >> On 1/13/1

[issue16880] Importing "imp" will fail if dynamic loading not supported

Nurhusien Hasen added the comment: Pleas stop your from me all isuss masege On 1/13/13, Nurhusien Hasen wrote: > Pleas stop your from me all isuss masege > > On 1/13/13, Nick Coghlan wrote: >> >> Nick Coghlan added the comment: >> >> The misleading docs Terry pointed out should also be fixed t

[issue16880] Importing "imp" will fail if dynamic loading not supported

Nurhusien Hasen added the comment: Pleas stop your from me all isuss masege On 1/13/13, Nick Coghlan 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 importlib really d

[issue16880] Importing "imp" will fail if dynamic loading not supported

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). -- ___

[issue16880] Importing "imp" will fail if dynamic loading not supported

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). -- ___

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 link

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 mo

[issue16880] Importing "imp" will fail if dynamic loading not supported

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. -- _

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 __

[issue16880] Importing "imp" will fail if dynamic loading not supported

Brett Cannon added the comment: I should say LGTM in terms of looking at the patch, not actually trying it out. =) -- ___ Python tracker ___

[issue16880] Importing "imp" will fail if dynamic loading not supported

Brett Cannon added the comment: LGTM -- stage: needs patch -> commit review ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Christian Heimes : -- keywords: +patch Added file: http://bugs.python.org/file28592/imp_dynamic.patch ___ Python tracker ___ __

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 _

[issue16880] Importing "imp" will fail if dynamic loading not supported

Changes by Antoine Pitrou : -- nosy: +brett.cannon, ncoghlan stage: -> needs patch versions: +Python 3.4 ___ Python tracker ___ ___ P

[issue16880] Importing "imp" will fail if dynamic loading not supported

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 attempt