bpo-32030: Add _PyMainInterpreterConfig_ReadEnv() by vstinner · Pull Request #4542 · python/cpython · GitHub
Skip to content

bpo-32030: Add _PyMainInterpreterConfig_ReadEnv() - #4542

Merged
vstinner merged 3 commits into
python:masterfrom
vstinner:mainconfig_read
Nov 24, 2017
Merged

bpo-32030: Add _PyMainInterpreterConfig_ReadEnv()#4542
vstinner merged 3 commits into
python:masterfrom
vstinner:mainconfig_read

Conversation

@vstinner

@vstinner vstinner commented Nov 24, 2017

Copy link
Copy Markdown
Member

Py_GetPath() and Py_Main() now call
_PyMainInterpreterConfig_ReadEnv() to share the same code to get
environment variables.

Changes:

  • Add _PyMainInterpreterConfig_ReadEnv()
  • Add _PyMainInterpreterConfig_Clear()
  • Add _PyMem_RawWcsdup()
  • _PyMainInterpreterConfig: rename pythonhome to home
  • Rename _Py_ReadMainInterpreterConfig() to
    _PyMainInterpreterConfig_Read()
  • Use _Py_INIT_USER_ERR(), instead of _Py_INIT_ERR(), for decoding
    errors: the user is able to fix the issue, it's not a bug in
    Python. Same change was made in _Py_INIT_NO_MEMORY().
  • Remove _Py_GetPythonHomeWithConfig()

https://bugs.python.org/issue32030

Py_GetPath() and Py_Main() now call
_PyMainInterpreterConfig_ReadEnv() to share the same code to get
environment variables.

Changes:

* Add _PyMainInterpreterConfig_ReadEnv()
* Add _PyMainInterpreterConfig_Clear()
* Add _PyMem_RawWcsdup()
* _PyMainInterpreterConfig: rename pythonhome to home
* Rename _Py_ReadMainInterpreterConfig() to
  _PyMainInterpreterConfig_Read()
* Use _Py_INIT_USER_ERR(), instead of _Py_INIT_ERR(), for decoding
  errors: the user is able to fix the issue, it's not a bug in
  Python. Same change was made in _Py_INIT_NO_MEMORY().
* Remove _Py_GetPythonHomeWithConfig()
@vstinner

Copy link
Copy Markdown
Member Author

@vstinner
vstinner merged commit 46972b7 into python:master Nov 24, 2017
@vstinner
vstinner deleted the mainconfig_read branch November 24, 2017 21:55
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