[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: The initial issue has been fixed, I close the issue. > I would like to keep the issue open since I plan to do one more change: use > PyWideStringList in _PyPathConfig for module_search_paths to support paths > which contains DELIM (":" on Unix). I may use

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: Oh, the main issue has been fixed and I added a lot of tests. So I remove the "release blocker" priority. I would like to keep the issue open since I plan to do one more change: use PyWideStringList in _PyPathConfig for module_search_paths to support paths

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by Kubilay Kocak : -- nosy: +koobs ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: > Oh, there is a regression: (...) PyInit__tkinter() calls Py_GetProgramName(). > But Python initialization no longer copies PyConfig path configuration into > _Py_path_info (the global configuration). So Python has to recompute the path > configuration

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 96c8475362acb41decd1d7db9243f328973e5de7 by Victor Stinner in branch '3.8': [3.8] bpo-38234: Backport init path config changes from master (GH-16423) https://github.com/python/cpython/commit/96c8475362acb41decd1d7db9243f328973e5de7 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +16002 pull_request: https://github.com/python/cpython/pull/16423 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 12f2f177fc483723406d7917194e7f655a20631b by Victor Stinner in branch 'master': bpo-38234: Py_Initialize() sets global path configuration (GH-16421) https://github.com/python/cpython/commit/12f2f177fc483723406d7917194e7f655a20631b --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by Pablo Galindo Salgado : -- priority: -> release blocker ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +16000 pull_request: https://github.com/python/cpython/pull/16421 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: Oh, there is a regression: vstinner@apu$ ./python Python 3.9.0a0 (heads/master:88feaecd46, Sep 26 2019, 03:26:37) [GCC 9.2.1 20190827 (Red Hat 9.2.1-1)] on linux Type "help", "copyright", "credits" or "license" for more information. >>> import _tkinter Could

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 49d99f01e6e51acec5ca57a02e857f0796bc418b by Victor Stinner in branch 'master': bpo-38234: Fix test_embed.test_init_setpath_config() on FreeBSD (GH-16406) https://github.com/python/cpython/commit/49d99f01e6e51acec5ca57a02e857f0796bc418b

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15988 pull_request: https://github.com/python/cpython/pull/16406 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 88feaecd46a8f427e30ef7ad8cfcddfe392a2402 by Victor Stinner in branch 'master': bpo-38234: Complete init config documentation (GH-16404) https://github.com/python/cpython/commit/88feaecd46a8f427e30ef7ad8cfcddfe392a2402 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15985 pull_request: https://github.com/python/cpython/pull/16404 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 8bf39b606ef7b02c0279a80789f3c4824b0da5e9 by Victor Stinner in branch 'master': bpo-38234: Add test_init_setpath_config() to test_embed (GH-16402) https://github.com/python/cpython/commit/8bf39b606ef7b02c0279a80789f3c4824b0da5e9 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15984 pull_request: https://github.com/python/cpython/pull/16402 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 00508a7407d7d300b487532e2271534b20e378a7 by Victor Stinner in branch 'master': bpo-38234: Fix test_embed pathconfig tests (GH-16390) https://github.com/python/cpython/commit/00508a7407d7d300b487532e2271534b20e378a7 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15972 pull_request: https://github.com/python/cpython/pull/16390 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: And on FreeBSD. AMD64 FreeBSD CURRENT Shared 3.x: https://buildbot.python.org/all/#/builders/168/builds/1510 == FAIL: test_init_pybuilddir (test.test_embed.InitConfigTests)

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: The new tests fail on macOS. x86-64 High Sierra 3.x: https://buildbot.python.org/all/#/builders/145/builds/2469 == FAIL: test_init_pybuilddir (test.test_embed.InitConfigTests)

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 221fd84703c545408bbb4a6e0b58459651331f5c by Victor Stinner in branch 'master': bpo-38234: Cleanup getpath.c (GH-16367) https://github.com/python/cpython/commit/221fd84703c545408bbb4a6e0b58459651331f5c --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15947 pull_request: https://github.com/python/cpython/pull/16367 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 52ad33abbfb6637d74932617c7013bae0ccf6e32 by Victor Stinner in branch 'master': bpo-38234: test_embed: test pyvenv.cfg and pybuilddir.txt (GH-16366) https://github.com/python/cpython/commit/52ad33abbfb6637d74932617c7013bae0ccf6e32 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15946 pull_request: https://github.com/python/cpython/pull/16366 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset bb6bf7d342b4503a6227fd209fac934905b6a1aa by Victor Stinner in branch 'master': bpo-38234: Add tests for Python init path config (GH-16358) https://github.com/python/cpython/commit/bb6bf7d342b4503a6227fd209fac934905b6a1aa --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15937 pull_request: https://github.com/python/cpython/pull/16358 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 1ce152a42eaa917d7763bce93f1e1ca72530d7ca by Victor Stinner in branch 'master': bpo-38234: Py_SetPath() uses the program full path (GH-16357) https://github.com/python/cpython/commit/1ce152a42eaa917d7763bce93f1e1ca72530d7ca --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15936 pull_request: https://github.com/python/cpython/pull/16357 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

miss-islington added the comment: New changeset 7f7cd899e3d773b803b7af5b0c19eeff83dd69fe by Miss Islington (bot) in branch '3.8': bpo-38234: read_pth_file() now returns PyStatus (GH-16338) https://github.com/python/cpython/commit/7f7cd899e3d773b803b7af5b0c19eeff83dd69fe -- nosy:

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by miss-islington : -- pull_requests: +15917 pull_request: https://github.com/python/cpython/pull/16340 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 85ce0a7178801b538160cbb5cf9ef50a713c45bf by Victor Stinner in branch 'master': bpo-38234: read_pth_file() now returns PyStatus (GH-16338) https://github.com/python/cpython/commit/85ce0a7178801b538160cbb5cf9ef50a713c45bf --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15916 pull_request: https://github.com/python/cpython/pull/16339 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15915 pull_request: https://github.com/python/cpython/pull/16338 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 3f5409a3f13c59baa34656bccefdc3728e46c9ef by Victor Stinner in branch '3.8': bpo-38234: Fix _PyConfig_InitPathConfig() (GH-16335) (GH-16336) https://github.com/python/cpython/commit/3f5409a3f13c59baa34656bccefdc3728e46c9ef --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15913 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/16336 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 9c42f8cda552694f3b47d6388d4ae84d61731872 by Victor Stinner in branch 'master': bpo-38234: Fix _PyConfig_InitPathConfig() (GH-16335) https://github.com/python/cpython/commit/9c42f8cda552694f3b47d6388d4ae84d61731872 --

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: > When calling Py_SetPath(), the value that is passed in is ignored. Hum, in fact the bug was even worse: Py_SetPythonHome() and Py_SetProgramName() calls are also ignored. I'm not sure when I introduced these regressions. PR 16335 fix this bug as well.

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15912 pull_request: https://github.com/python/cpython/pull/16335 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

Steve Dower added the comment: > _PyPathConfig_Init() initializes _Py_dll_path global variable which is only > used by _Py_CheckPython3(). Ah okay, maybe I'll take the time later on (in a separate issue) to get rid of it fully. We may as well load python3.dll at startup anyway, it's not

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: Steve: > Why are we keeping the DLL path around at all? It should only be being used > in 1-2 places during path calculation. _PyPathConfig_Init() initializes _Py_dll_path global variable which is only used by _Py_CheckPython3(). _Py_CheckPython3() is

[issue38234] The value of Py_SetPath is not used to populate the configuration

Steve Dower added the comment: Why are we keeping the DLL path around at all? It should only be being used in 1-2 places during path calculation. -- nosy: +steve.dower ___ Python tracker

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: > When calling Py_SetPath(), the value that is passed in is ignored. I fixed this regression. Thanks for the bug report. -- priority: release blocker -> resolution: -> fixed stage: patch review -> resolved status: open -> closed

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset 9f3dcf802eefeb5ab821ce3c7204ab46557d53d7 by Victor Stinner in branch '3.8': [3.8] bpo-38234: Fix PyConfig_Read() when Py_SetPath() was called (GH-16298) (GH-16313)

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15895 pull_request: https://github.com/python/cpython/pull/16313 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by miss-islington : -- pull_requests: +15894 pull_request: https://github.com/python/cpython/pull/16312 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset e267793aa4101b2771ed0e66aaff5743d23f59af by Victor Stinner in branch 'master': bpo-38234: Fix PyConfig_Read() when Py_SetPath() was called (GH-16298) https://github.com/python/cpython/commit/e267793aa4101b2771ed0e66aaff5743d23f59af --

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: New changeset c422167749f92d4170203e996a2c619c818335ea by Victor Stinner in branch 'master': bpo-38234: Remove _PyPathConfig.dll_path (GH-16307) https://github.com/python/cpython/commit/c422167749f92d4170203e996a2c619c818335ea --

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- pull_requests: +15893 pull_request: https://github.com/python/cpython/pull/16307 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: > Fatal Python error: init_fs_encoding: failed to get the Python codec of the > filesystem encoding I created bpo-38236 "Dump the Python path configuration at the first import error" which should ease to debug such issues. --

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: That's a regression that I introduced between Python 3.7 and 3.8 when refactoring _PyCoreConfig/_PyPathConfig code into PyConfig/_PyPathConfig. Previously, if Py_SetPath() was called, the specified string was used to fill config->module_search_paths. Now,

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by STINNER Victor : -- keywords: +patch pull_requests: +15884 stage: -> patch review pull_request: https://github.com/python/cpython/pull/16298 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

Pablo Galindo Salgado added the comment: A very simple way to reproduce the bug: Using pyInstaller, after applying these patches (to fix other Python3.8 issues) https://github.com/pyinstaller/pyinstaller/pull/4441 https://github.com/pyinstaller/pyinstaller/pull/4440 you will get Fatal

[issue38234] The value of Py_SetPath is not used to populate the configuration

STINNER Victor added the comment: It's a 3.8 regression: I set the priority to release blocker. -- nosy: +lukasz.langa priority: normal -> release blocker ___ Python tracker

[issue38234] The value of Py_SetPath is not used to populate the configuration

New submission from Pablo Galindo Salgado : When calling Py_SetPath(), the value that is passed in is ignored. -- components: Interpreter Core keywords: 3.8regression messages: 352856 nosy: pablogsal, vstinner priority: normal severity: normal status: open title: The value of