This repository has been archived by the owner on Jul 4, 2023. It is now read-only.
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reorganize superenv include and library path setup
I found the dual use of CMAKE_*_PATH variables to make it difficult to reason about this code. Now a separate set of variables are used to communicate with the cc wrapper, and less work is performed in the wrapper itself. We no longer pass the SDK include path as a -isystem directory on Xcode-only setups. Doing so is redundant with `--sysroot` and has other side effects, namely changing the include path search order, which can break compilation of some software (e.g. qemu). On Xcode-only 10.9, we can additionally omit `--sysroot`, as the correct paths are built into the tools. A new variable, HOMEBREW_SYSROOT, is used to this information to the wrapper. It will be unset on Xcode-only 10.9. HOMEBREW_SDKROOT will continue to be set, as it is used for other things besides setting the include search path.
- Loading branch information
Showing
2 changed files
with
71 additions
and
65 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters