[3.7] bpo-33042: Fix pre-initialization sys module configuration (GH-6157) by miss-islington · Pull Request #6232 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Merged
miss-islington merged 1 commit intopython:3.7from
Mar 25, 2018Merged
Conversation
Copy link Copy Markdown
Contributor
- new test case for pre-initialization of sys.warnoptions and sys._xoptions
- restored ability to call these APIs prior to Py_Initialize
- updated the docs for the affected APIs to make it clear they can be
called before Py_Initialize - also enhanced the existing embedding test cases
to check for expected settings in the sys module
(cherry picked from commit bc77eff)
Co-authored-by: Nick Coghlan ncoghlan@gmail.com
…6157) - new test case for pre-initialization of sys.warnoptions and sys._xoptions - restored ability to call these APIs prior to Py_Initialize - updated the docs for the affected APIs to make it clear they can be called before Py_Initialize - also enhanced the existing embedding test cases to check for expected settings in the sys module (cherry picked from commit bc77eff) Co-authored-by: Nick Coghlan <ncoghlan@gmail.com>
bedevere-bot
mentioned this pull request
the-knights-who-say-ni
added
the
CLA signed
label
bedevere-bot
added
the
awaiting review
label
ncoghlan approved these changes Mar 25, 2018
bedevere-bot
added
awaiting merge
and removed
awaiting review
labels
Copy link Copy Markdown
Contributor Author
miss-islington
commented
Mar 25, 2018
miss-islington commented
Mar 25, 2018@ncoghlan: Backport status check is done, and it's a success ✅ .
miss-islington
merged commit
c6d94c3
into
python:3.7
Copy link Copy Markdown
Contributor Author
miss-islington
commented
Mar 25, 2018
miss-islington commented
Mar 25, 2018Thanks!
bedevere-bot
removed
the
awaiting merge
label
miss-islington
deleted the
backport-bc77eff-3.7
branch
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment