diff options
| author | Russell Yanofsky <[email protected]> | 2020-03-04 10:56:07 -0500 |
|---|---|---|
| committer | Russell Yanofsky <[email protected]> | 2020-04-22 08:18:11 -0500 |
| commit | 1e94a2bcbc5ff8ae61eed9f31317ea534649116d (patch) | |
| tree | cb5b66df399fda30bca1872c49d6125d1e094346 /src/httpserver.cpp | |
| parent | Merge #18702: build: fix ASLR for bitcoin-cli on Windows (diff) | |
| download | discoin-1e94a2bcbc5ff8ae61eed9f31317ea534649116d.tar.xz discoin-1e94a2bcbc5ff8ae61eed9f31317ea534649116d.zip | |
depends: Add --sysroot option to mac os native compile flags
Catalina SDK clang stopped automatically searching the SDK include paths when
invoked without --sysroot:
https://github.com/bitcoin/bitcoin/pull/16367#issuecomment-594600985
https://github.com/Homebrew/homebrew-core/issues/45061
This hasn't been a problem for current native depends packages because are
passing their own --sysroot values, and hasn't been a problem for current host
packages because they use `darwin_` commands instead of `build_darwin_`
commands. But the current `build_darwin_CC` and `build_darwin_CXX` commands
are still unnecessarily fragile, and incompatible with new native depends
packages added in https://github.com/bitcoin/bitcoin/pull/18677.
Cory Fields <[email protected]> suggested in
https://github.com/bitcoin/bitcoin/pull/16367#issuecomment-595393546 switching
compiler from SDK clang to native clang (from $PATH) to avoid this problem.
This is easy and makes a certain amount of sense for building native packages,
as opposed to host packages. But fanquake <[email protected]> pointed out in
https://github.com/bitcoin/bitcoin/pull/18677#discussion_r409934309 that it
would be inconsistent use switch to non-SDK compilers while still using other
SDK tools like ranlib and install_name_tool. So simplest, minimal fix seems to
be just adding the missing --sysroot option.
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions