Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-32030: Add _PyCoreConfig.module_search_path_env #4504

Merged
merged 1 commit into from
Nov 22, 2017
Merged

bpo-32030: Add _PyCoreConfig.module_search_path_env #4504

merged 1 commit into from
Nov 22, 2017

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Nov 22, 2017

Changes:

  • Py_Main() initializes _PyCoreConfig.pythonpath from the PYTHONPATH
    environment variable.
  • PyInterpreterState_New() now initializes core_config and config
    fields
  • Compute sys.path a little bit ealier in
    _Py_InitializeMainInterpreter() and new_interpreter()
  • Add _Py_GetPathWithConfig() private function.

https://bugs.python.org/issue32030

@vstinner vstinner requested a review from a team as a code owner November 22, 2017 17:50
@vstinner vstinner changed the title bpo-32030: Add _PyCoreConfig.pythonpath field bpo-32030: Add _PyCoreConfig.env_path field Nov 22, 2017
Changes:

* Py_Main() initializes _PyCoreConfig.module_search_path_env from
  the PYTHONPATH environment variable.
* PyInterpreterState_New() now initializes core_config and config
  fields
* Compute sys.path a little bit ealier in
  _Py_InitializeMainInterpreter() and new_interpreter()
* Add _Py_GetPathWithConfig() private function.
@vstinner vstinner changed the title bpo-32030: Add _PyCoreConfig.env_path field bpo-32030: Add _PyCoreConfig.module_search_path_env field Nov 22, 2017
@vstinner vstinner changed the title bpo-32030: Add _PyCoreConfig.module_search_path_env field bpo-32030: Add _PyCoreConfig.module_search_path_env Nov 22, 2017
@vstinner vstinner merged commit d434110 into python:master Nov 22, 2017
@vstinner vstinner deleted the pymain_pythonpath branch November 22, 2017 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants