Skip to content

gh-156895: Document CXX in ./configure --help output - #157046

Merged
vstinner merged 2 commits into
python:mainfrom
lazerg:fix/issue-156895-document-cxx-help
Oct 8, 2026
Merged

vstinner merged 2 commits into
python:mainfrom
lazerg:fix/issue-156895-document-cxx-help

Conversation

@lazerg

@lazerg lazerg commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

configure.ac detects and uses CXX for the C++ compiler, but never declares it with AC_ARG_VAR, so ./configure --help never lists it next to CC. This adds AC_ARG_VAR([CXX], [C++ compiler command]) so it shows up the same way CC already does, and regenerates configure with autoconf 2.72.

Fixes #156895

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't need a news entry.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed the news entry in 094cf3e. The PR is now only the configure.ac change and the regenerated configure.

@bedevere-app

bedevere-app Bot commented Sep 7, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@lazerg

lazerg commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Hey @StanFromIreland @vstinner, please take a look when you have time

@vstinner
vstinner merged commit 2423814 into python:main Oct 8, 2026
48 checks passed
@vstinner

vstinner commented Oct 8, 2026

Copy link
Copy Markdown
Member

I merged your change, thanks. I don't think that it's worth it to backport it to 3.14 and 3.15 branches.

Note: the option is documented at https://docs.python.org/dev/using/configure.html#envvar-CXX.

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.

Configure script output doesn't mention CXX

3 participants