[issue33128] PathFinder is twice on sys.meta_path

2018-04-24 Thread Nick Coghlan
Nick Coghlan added the comment: Hartmut, thanks for the issue report, and Pablo, thanks for the PR to resolve it! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue33128] PathFinder is twice on sys.meta_path

2018-04-24 Thread miss-islington
miss-islington added the comment: New changeset 52a5a17338dfa7fed259027e1ecceba9c8491189 by Miss Islington (bot) in branch '3.7': bpo-33128 Fix duplicated call to importlib._install_external_importers (GH-6273)

[issue33128] PathFinder is twice on sys.meta_path

2018-04-24 Thread miss-islington
Change by miss-islington : -- pull_requests: +6290 ___ Python tracker ___

[issue33128] PathFinder is twice on sys.meta_path

2018-04-24 Thread Nick Coghlan
Nick Coghlan added the comment: New changeset 0977091dca59709864b14cfc129388f1f0de7bf7 by Nick Coghlan (Pablo Galindo) in branch 'master': bpo-33128 Fix duplicated call to importlib._install_external_importers (GH-6273)

[issue33128] PathFinder is twice on sys.meta_path

2018-03-28 Thread Nick Coghlan
Nick Coghlan added the comment: Calling initexternalimports InitalizeMainInterpreter and from new_interpreter is right, since we only want these importers on the metapath after we know sys.path has been configured appropriately. It's the call from InitializeCore that's

[issue33128] PathFinder is twice on sys.meta_path

2018-03-27 Thread Pablo Galindo Salgado
Change by Pablo Galindo Salgado : -- keywords: +patch pull_requests: +6001 stage: needs patch -> patch review ___ Python tracker

[issue33128] PathFinder is twice on sys.meta_path

2018-03-27 Thread Pablo Galindo Salgado
Pablo Galindo Salgado added the comment: The same problem happens in `new_interpreter` as far as I understand. -- ___ Python tracker

[issue33128] PathFinder is twice on sys.meta_path

2018-03-27 Thread Pablo Galindo Salgado
Pablo Galindo Salgado added the comment: It seems that the problem is that `_Py_InitializeEx_Private` calls `_Py_InitializeCore` and `_Py_InitializeMainInterpreter`. The first one calls at the end `initimport` that in turns calls `importlib._install_external_importers`

[issue33128] PathFinder is twice on sys.meta_path

2018-03-26 Thread Brett Cannon
Change by Brett Cannon : -- keywords: +3.7regression nosy: +brett.cannon ___ Python tracker ___

[issue33128] PathFinder is twice on sys.meta_path

2018-03-23 Thread Ned Deily
Ned Deily added the comment: Thanks for the report. git bisect result: 1abcf6700b4da6207fe859de40c6c1bada6b4fec is the first bad commit commit 1abcf6700b4da6207fe859de40c6c1bada6b4fec Author: Eric Snow Date: Tue May 23 21:46:51 2017 -0700

[issue33128] PathFinder is twice on sys.meta_path

2018-03-23 Thread Hartmut Goebel
New submission from Hartmut Goebel : As of Python 3.7.0b2 _frozen_importlib_external.PathFinder exists twice on sys.meta_path, and it is the same object: $ python -S Python 3.7.0b2 (default, Mar 22 2018, 20:09:00) [GCC 5.5.0] on linux Type "help", "copyright",