bpo-44113: Deprecate old functions to config Python init #26060
Merged
+98
−10
Conversation
Don't we have to change the signature of API like this? from Include/sysmodule.h Py_DEPRECATED(3.11) PyAPI_FUNC(void) PySys_AddWarnOption(const wchar_t *); |
@corona10: Oh right, I added Py_DEPRECATED(). I rebased my PR and also deprecated PySys_HasWarnOptions(). |
Deprecate the following functions to configure the Python initialization: * PySys_AddWarnOption() * PySys_AddWarnOptionUnicode() * PySys_AddXOption() * PySys_HasWarnOptions() * Py_SetPath() * Py_SetProgramName() * Py_SetPythonHome() * Py_SetStandardStreamEncoding() * _Py_SetProgramFullPath() Use the new PyConfig API of the Python Initialization Configuration instead (PEP 587).
I also deprecated the undocumented _Py_SetProgramFullPath() function. |
@corona10: Would you mind to review the updated PR? |
lgtm |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Deprecate the following functions to configure the Python
initialization:
Use the new PyConfig API of the Python Initialization Configuration
instead (PEP 587).
https://bugs.python.org/issue44113