[Python-Dev] PEP 451 update (original) (raw)

Nick Coghlan ncoghlan at gmail.com
Sat Nov 2 16:38:48 CET 2013

On 3 November 2013 01:21, Brett Cannon <brett at python.org> wrote:

On Sat, Nov 2, 2013 at 5:46 AM, Nick Coghlan <ncoghlan at gmail.com> wrote:

For the same reason, I also have a mild preference for "target" (or the more explicit "loadtarget") as the name, although I won't object if you and Brett prefer "existing". I would go with "module" or "target" and that is where I shall end the bikeshedding.

I had a quick reread of the runpy docs: http://docs.python.org/dev/library/runpy#runpy.run_module

I can definitely make "target" work (that's why I suggested it originally, re-reading the docs just confirmed that), but I don't see any non-confusing way to use "module".

Eric: since "target" was on both Brett's list and mine, it looks like s/existing/target/ and clarifying that "incompatible target module" should be reported by throwing ImportError from find_spec() will get you the pronouncement you're looking for :)

Cheers, Nick.

-- Nick Coghlan | ncoghlan at gmail.com | Brisbane, Australia

More information about the Python-Dev mailing list