-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
[3.8] bpo-39562: Prevent collision of future and compiler flags (GH-19230) #19835
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 and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
We need to still maybe add a better way to document this? @vstinner Where would you like for the documentation for this change to live? |
Can you at least backport the NEWS entry? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if you add the NEWS entry.
I tested manually: this change does fix https://bugs.python.org/issue39562#msg365311
One option is to add a "Notable changes in Python 3.8.3" section since the change is backward incompatible. Something like: But it's up to you to decide if it's worth it or not. The 3.9 change is documented in What's New in Python 3.9, in the "Porting to Python 3.9" section: |
Oh, seems like @isidentical used the new
I will update the PR soon to solve this and with @vstinner suggestions |
…onGH-19230) The constant values of future flags in the __future__ module is updated in order to prevent collision with compiler flags. Previously PyCF_ALLOW_TOP_LEVEL_AWAIT was clashing with CO_FUTURE_DIVISION.. (cherry picked from commit 4454057) Co-authored-by: Batuhan Taşkaya <batuhanosmantaskaya@gmail.com>
8112c50
to
5131901
Compare
@pablogsal: Status check is done, and it's a success ✅ . |
The constant values of future flags in the future module
is updated in order to prevent collision with compiler flags.
Previously PyCF_ALLOW_TOP_LEVEL_AWAIT was clashing
with CO_FUTURE_DIVISION..
(cherry picked from commit 4454057)
Co-authored-by: Batuhan Taşkaya batuhanosmantaskaya@gmail.com
https://bugs.python.org/issue39562