bpo-38304: Add PyConfig.struct_size by vstinner · Pull Request #16451 · python/cpython (original) (raw)
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 andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation3 Commits1 Checks0 Files changed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
Add a new struct_size field to PyPreConfig and PyConfig structures to
allow to modify these structures in the future without breaking the
backward compatibility.
- Replace private _config_version field with public struct_size field
in PyPreConfig and PyConfig. - Public PyPreConfig_InitIsolatedConfig() and
PyPreConfig_InitPythonConfig()
return type becomes PyStatus, instead of void. - Internal _PyConfig_InitCompatConfig(),
_PyPreConfig_InitCompatConfig(), _PyPreConfig_InitFromConfig(),
_PyPreConfig_InitFromPreConfig() return type becomes PyStatus,
instead of void. - Remove _Py_CONFIG_VERSION
- Update the Initialization Configuration documentation.
https://bugs.python.org/issue38304
Add a new struct_size field to PyPreConfig and PyConfig structures to allow to modify these structures in the future without breaking the backward compatibility.
- Replace private _config_version field with public struct_size field in PyPreConfig and PyConfig.
- Public PyPreConfig_InitIsolatedConfig() and PyPreConfig_InitPythonConfig() return type becomes PyStatus, instead of void.
- Internal _PyConfig_InitCompatConfig(), _PyPreConfig_InitCompatConfig(), _PyPreConfig_InitFromConfig(), _PyPreConfig_InitFromPreConfig() return type becomes PyStatus, instead of void.
- Remove _Py_CONFIG_VERSION
- Update the Initialization Configuration documentation.
Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖
Sorry @vstinner, I had trouble checking out the 3.8
backport branch.
Please backport using cherry_picker on command line.cherry_picker 441b10cf2855955c86565f8d59e72c2efc0f0a57 3.8
vstinner added a commit that referenced this pull request
Add a new struct_size field to PyPreConfig and PyConfig structures to allow to modify these structures in the future without breaking the backward compatibility.
- Replace private _config_version field with public struct_size field in PyPreConfig and PyConfig.
- Public PyPreConfig_InitIsolatedConfig() and PyPreConfig_InitPythonConfig() return type becomes PyStatus, instead of void.
- Internal _PyConfig_InitCompatConfig(), _PyPreConfig_InitCompatConfig(), _PyPreConfig_InitFromConfig(), _PyPreConfig_InitFromPreConfig() return type becomes PyStatus, instead of void.
- Remove _Py_CONFIG_VERSION
- Update the Initialization Configuration documentation.
(cherry picked from commit 441b10c)
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request
Add a new struct_size field to PyPreConfig and PyConfig structures to allow to modify these structures in the future without breaking the backward compatibility.
- Replace private _config_version field with public struct_size field in PyPreConfig and PyConfig.
- Public PyPreConfig_InitIsolatedConfig() and PyPreConfig_InitPythonConfig() return type becomes PyStatus, instead of void.
- Internal _PyConfig_InitCompatConfig(), _PyPreConfig_InitCompatConfig(), _PyPreConfig_InitFromConfig(), _PyPreConfig_InitFromPreConfig() return type becomes PyStatus, instead of void.
- Remove _Py_CONFIG_VERSION
- Update the Initialization Configuration documentation.
Reviewers
brettcannon Awaiting requested review from brettcannon
encukou Awaiting requested review from encukou
ericsnowcurrently Awaiting requested review from ericsnowcurrently
ncoghlan Awaiting requested review from ncoghlan
warsaw Awaiting requested review from warsaw