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-45571: use PY_CFLAGS_NODIST for shared Modules/Setup (GH-29161) #29161

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

tiran
Copy link
Member

@tiran tiran commented Oct 22, 2021

Modules/Setup now use PY_CFLAGS_NODIST instead of PY_CFLAGS to
compile shared modules.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue45571

``Modules/Setup`` now use ``PY_CFLAGS_NODIST`` instead of ``PY_CFLAGS`` to
compile shared modules.

Signed-off-by: Christian Heimes <christian@python.org>
@tiran tiran added type-bug An unexpected behavior, bug, or error needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Oct 22, 2021
@tiran tiran requested review from brettcannon and Yhg1s October 22, 2021 13:30
@tiran tiran changed the title bpo-45571: use PY_CFLAGS_NODIST for shared Modules/Setup bpo-45571: use PY_CFLAGS_NODIST for shared Modules/Setup (GH-29161) Oct 22, 2021
@tiran tiran merged commit f6e8b80 into python:main Oct 22, 2021
@tiran tiran deleted the bpo-45571-makesetup branch October 22, 2021 18:02
@miss-islington
Copy link
Contributor

Thanks @tiran for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2021
)

(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <christian@python.org>
@bedevere-bot
Copy link

GH-29168 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed needs backport to 3.10 only security fixes needs backport to 3.9 only security fixes labels Oct 22, 2021
@bedevere-bot
Copy link

GH-29169 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2021
)

(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <christian@python.org>
miss-islington added a commit that referenced this pull request Oct 22, 2021
(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <christian@python.org>
miss-islington added a commit that referenced this pull request Oct 22, 2021
(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <christian@python.org>
@psphilippe

This comment has been minimized.

@psphilippe

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants