Skip to content
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

bpo-35147: Fix _Py_NO_RETURN for GCC #10300

Merged
merged 1 commit into from Nov 2, 2018

Conversation

izbyshev
Copy link
Contributor

@izbyshev izbyshev commented Nov 2, 2018

Use __GNUC__ instead of non-existing __GNUC_MAJOR__.

https://bugs.python.org/issue35147

Use __GNUC__ instead of non-existing __GNUC_MAJOR__.
Copy link
Member

@vstinner vstinner left a comment

LGTM.

@vstinner vstinner added the 🤖 automerge PR will be merged once it's been approved and all CI passed label Nov 2, 2018
@miss-islington
Copy link
Contributor

miss-islington commented Nov 2, 2018

@izbyshev: Status check is done, and it's a success .

@miss-islington miss-islington merged commit e2ed5ad into python:master Nov 2, 2018
@miss-islington
Copy link
Contributor

miss-islington commented Nov 2, 2018

Thanks @izbyshev for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒🤖 I'm not a witch! I'm not a witch!

@miss-islington
Copy link
Contributor

miss-islington commented Nov 2, 2018

Sorry, @izbyshev, I could not cleanly backport this to 3.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker e2ed5adcb5db2d70cfa72da1ba8446f7aa9e05cd 3.7

@miss-islington miss-islington self-assigned this Nov 2, 2018
@miss-islington
Copy link
Contributor

miss-islington commented Nov 2, 2018

Sorry, @izbyshev, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker e2ed5adcb5db2d70cfa72da1ba8446f7aa9e05cd 3.6

izbyshev added a commit to izbyshev/cpython that referenced this pull request Nov 2, 2018
Use `__GNUC__` instead of non-existing `__GNUC_MAJOR__`.

https://bugs.python.org/issue35147.
(cherry picked from commit e2ed5ad)

Co-authored-by: Alexey Izbyshev <izbyshev@ispras.ru>
@bedevere-bot
Copy link

bedevere-bot commented Nov 2, 2018

GH-10301 is a backport of this pull request to the 3.7 branch.

izbyshev added a commit to izbyshev/cpython that referenced this pull request Nov 2, 2018
Use `__GNUC__` instead of non-existing `__GNUC_MAJOR__`.

https://bugs.python.org/issue35147.
(cherry picked from commit e2ed5ad)

Co-authored-by: Alexey Izbyshev <izbyshev@ispras.ru>
@bedevere-bot
Copy link

bedevere-bot commented Nov 2, 2018

GH-10302 is a backport of this pull request to the 3.6 branch.

vstinner pushed a commit that referenced this pull request Nov 2, 2018
Use `__GNUC__` instead of non-existing `__GNUC_MAJOR__`.

(cherry picked from commit e2ed5ad)
@izbyshev izbyshev deleted the bpo-35147 branch Nov 2, 2018
miss-islington pushed a commit that referenced this pull request Nov 3, 2018
Use `__GNUC__` instead of non-existing `__GNUC_MAJOR__`.

(cherry picked from commit e2ed5ad)


https://bugs.python.org/issue35147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed skip news type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants