Skip to content

[WIP] bpo-36301: Add _Py_PreInitialize() function#12351

Closed
vstinner wants to merge 1 commit into
python:masterfrom
vstinner:py_preinitialize
Closed

[WIP] bpo-36301: Add _Py_PreInitialize() function#12351
vstinner wants to merge 1 commit into
python:masterfrom
vstinner:py_preinitialize

Conversation

@vstinner

@vstinner vstinner commented Mar 15, 2019

Copy link
Copy Markdown
Member

Add also _PyRuntimeState.preconfig field. _PyPreConfig_Write() now
writes the applied pre-configuration into _PyRuntimeState.preconfig.

https://bugs.python.org/issue36301

* Add _PyRuntimeState.preconfig field. _PyPreConfig_Write() now
  writes the applied pre-configuration into _PyRuntimeState.preconfig.
* Add _Py_PreInitialize() and _Py_PreInitializeFromPreConfig()
  functions
* _PyCoreConfig_Read() now calls _Py_PreInitialize() to ensure that
  Python is pre-configured. It reads the pre-configuration from
  _PyRuntime.
@vstinner vstinner changed the title bpo-36301: Add _Py_PreInitialize() function [WIP] bpo-36301: Add _Py_PreInitialize() function Mar 18, 2019
@vstinner

Copy link
Copy Markdown
Member Author

I'm not sure about this change. I experimented a different implementation in local branches. I need more time to think about the overall API and how it is supposed to work.

@vstinner

Copy link
Copy Markdown
Member Author

This PR combines too many changes. I wrote PR #12457 which is shorter and so less risky.

@vstinner vstinner closed this Mar 20, 2019
@vstinner vstinner deleted the py_preinitialize branch March 20, 2019 01:00
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