Skip to content

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 [email protected]

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 <[email protected]>
@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 <[email protected]>
@bedevere-bot
Copy link

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

@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 <[email protected]>
miss-islington added a commit that referenced this pull request Oct 22, 2021
(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <[email protected]>
miss-islington added a commit that referenced this pull request Oct 22, 2021
(cherry picked from commit f6e8b80)

Co-authored-by: Christian Heimes <[email protected]>
@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