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: Rewrite _PyMainInterpreterConfig. #4854

Merged
merged 1 commit into from
Dec 14, 2017
Merged

bpo-32030: Rewrite _PyMainInterpreterConfig. #4854

merged 1 commit into from
Dec 14, 2017

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Dec 14, 2017

_PyMainInterpreterConfig now contains Python objects, whereas
_PyCoreConfig contains wchar_t* strings.

Core config:

  • Rename _PyMainInterpreterConfig_ReadEnv() to _PyCoreConfig_ReadEnv()
  • Move 3 strings from _PyMainInterpreterConfig to _PyCoreConfig:
    module_search_path_env, home, program_name.
  • Add _PyCoreConfig_Clear()
  • _PyPathConfig_Calculate() now takes core config rather than main
    config
  • _PyMainInterpreterConfig_Read() now requires also a core config

Main config:

  • Add _PyMainInterpreterConfig.module_search_path: sys.path list
  • Add _PyMainInterpreterConfig.argv: sys.argv list
  • _PyMainInterpreterConfig_Read() now computes module_search_path

https://bugs.python.org/issue32030

_PyMainInterpreterConfig now contains Python objects, whereas
_PyCoreConfig contains wchar_t* strings.

Core config:

* Rename _PyMainInterpreterConfig_ReadEnv() to _PyCoreConfig_ReadEnv()
* Move 3 strings from _PyMainInterpreterConfig to _PyCoreConfig:
  module_search_path_env, home, program_name.
* Add _PyCoreConfig_Clear()
* _PyPathConfig_Calculate() now takes core config rather than main
  config
* _PyMainInterpreterConfig_Read() now requires also a core config

Main config:

* Add _PyMainInterpreterConfig.module_search_path: sys.path list
* Add _PyMainInterpreterConfig.argv: sys.argv list
* _PyMainInterpreterConfig_Read() now computes module_search_path
@vstinner vstinner requested a review from a team as a code owner December 14, 2017 00:58
@vstinner vstinner merged commit b5fd9ad into python:master Dec 14, 2017
@vstinner vstinner deleted the sys_path branch December 14, 2017 01:20
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