diff options
| author | Dominyk Tiller | 2016-07-15 03:36:04 +0100 |
|---|---|---|
| committer | Dominyk Tiller | 2016-07-15 05:14:51 +0100 |
| commit | 706e7e71a07784f14abbdf5f019428e04afcfae6 (patch) | |
| tree | 2dcd97530c8cf9c6d5fccd2a5146a3a5663c0e67 /Library/ENV | |
| parent | f6dad377b63946d403df7ed755fb14426a33df31 (diff) | |
| download | brew-706e7e71a07784f14abbdf5f019428e04afcfae6.tar.bz2 | |
mac/super: restore definition of effective_sysroot
In https://github.com/Homebrew/brew/commit/0d189fae57bad6c209b471eba9e0b254a2b40886
we completely removed `effective_sysroot`, which consequently left all the
`effective_sysroot`/usr/include calls pointing at `/usr/include`. This is wildly
problematic on systems where the Command Line Tools aren't installed because
`/usr/include` is not a default-created folder prior to CLT installation.
I'm unsure if `effective_sysroot` should still be mentioned in Library/Homebrew/extend/ENV/super.rb
at all. If it can be deleted, feel free to do that without waiting for me to review.
This seems to fix:
* https://github.com/Homebrew/homebrew-core/issues/2991
* https://github.com/Homebrew/homebrew-core/issues/2986
* https://github.com/Homebrew/homebrew-core/issues/2962
Diffstat (limited to 'Library/ENV')
0 files changed, 0 insertions, 0 deletions