Skip to content

bpo-33483: more correctly handle finding the C compiler #6780

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

Merged
merged 2 commits into from
May 15, 2018

Conversation

grimreaper
Copy link
Contributor

@grimreaper grimreaper commented May 13, 2018

Instead of passing configure args such as --without-gcc or --with-icc,
instead prefer to rely on the native way of finding the compiler: passing
CC (or CPP or CXX depending).

This allows configure to find the correct compiler instead of having to be
explicitly told. It also more correctly builds on both macOS and FreeBSD
since the system compiler is used by default (cc)

https://bugs.python.org/issue33483

@grimreaper grimreaper force-pushed the eax/build/no_gcc_req branch from 39636c9 to 32316d1 Compare May 13, 2018 19:06
@grimreaper grimreaper changed the title bpo-33483: Correctly handle finding the C compiler bpo-33483: more correctly handle finding the C compiler May 14, 2018
Instead of passing configure args such as --without-gcc or --with-icc,
instead prefer to rely on the native way of finding the compiler:
passing CC (or CPP or CXX depending).

This allows configure to find the correct compiler instead of having to
be explicitly told. It also more correctly builds on both macOS and
FreeBSD since the system compiler is used by default (cc)
@grimreaper grimreaper force-pushed the eax/build/no_gcc_req branch from 32316d1 to ab95333 Compare May 15, 2018 02:39
@benjaminp benjaminp merged commit 98929b5 into python:master May 15, 2018
@grimreaper grimreaper deleted the eax/build/no_gcc_req branch May 15, 2018 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants