gh-89886: Properly quote Autoconf macro arguments by erlend-aasland · Pull Request #105062 · python/cpython · GitHub
Skip to content

gh-89886: Properly quote Autoconf macro arguments#105062

Merged
erlend-aasland merged 7 commits intopython:mainfrom
erlend-aasland:autoconf/macro-audit
May 30, 2023
Merged

gh-89886: Properly quote Autoconf macro arguments#105062
erlend-aasland merged 7 commits intopython:mainfrom
erlend-aasland:autoconf/macro-audit

Conversation

@erlend-aasland
Copy link
Copy Markdown
Contributor

@erlend-aasland erlend-aasland commented May 29, 2023

Autoconf 2.70 macros are picky about argument quotation.

Autoconf 2.70 macros are picky about argument quotation.
@bedevere-bot
Copy link
Copy Markdown

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 29, 2023
@erlend-aasland erlend-aasland added the needs backport to 3.12 only security fixes label May 29, 2023
Comment thread pyconfig.h.in Outdated
@erlend-aasland
Copy link
Copy Markdown
Contributor Author

On hold until test_import and test_peg_generator issues have been resolved on main.

Comment thread pyconfig.h.in Outdated
Comment thread configure.ac
Comment thread configure.ac Outdated
@erlend-aasland
Copy link
Copy Markdown
Contributor Author

Thanks for the reviews, y'all!

@erlend-aasland
Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
Member

@gpshead gpshead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for taking this on.

@erlend-aasland erlend-aasland enabled auto-merge (squash) May 30, 2023 20:52
@erlend-aasland erlend-aasland merged commit 2ea34cf into python:main May 30, 2023
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington
Copy link
Copy Markdown
Contributor

Sorry, @erlend-aasland, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 2ea34cfb3a21182b4d16f57dd6c1cfce46362fe2 3.12

@erlend-aasland erlend-aasland deleted the autoconf/macro-audit branch May 30, 2023 21:19
@bedevere-bot
Copy link
Copy Markdown

GH-105118 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label May 30, 2023
erlend-aasland added a commit to erlend-aasland/cpython that referenced this pull request May 30, 2023
…n#105062)

Autoconf 2.70 macros are picky about argument quotation.
erlend-aasland added a commit that referenced this pull request May 30, 2023
…105118)

Autoconf 2.70 macros are picky about argument quotation.
@bedevere-bot
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants