diff options
author | fanquake <fanquake@gmail.com> | 2020-06-29 12:44:12 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2020-06-29 13:08:33 +0800 |
commit | dd191b583294b09cf59f1318de6afc889bbe22da (patch) | |
tree | 611e92e492d9fe253e3406ce223b8cdb9880d310 /.appveyor.yml | |
parent | d3a5dbfd1f18b89dd990ee83ece7c1dd9ba94b1a (diff) | |
parent | 8578c6fccd11404412d2c60f9bede311b79ca0d0 (diff) |
Merge #19356: build: Fix search for brew-installed BDB 4 on OS X
8578c6fccd11404412d2c60f9bede311b79ca0d0 build: Fix search for brew-installed BDB 4 on OS X (Glenn Willen)
Pull request description:
~~NOTE: This PR contains one important fix that I need (to make Bitcoin Core build cleanly on my system without shenanigans), plus some related general cleanup that is not really necessary, and could be annoying. (I am prepared to defend my argument that BDB_CFLAGS is wrong here, and BDB_CPPFLAGS is right, but this could bite anybody who has gotten in the habit of -- or scripted -- setting the former.)~~
Ok, I have been convinced that I was too clever with the refactor and I have removed it. Now it's just the tiny change to fix the build on my local machine.
---
On OS X, when searching Homebrew keg-only packages for BDB 4.8, if we find it,
use BDB_CPPFLAGS and BDB_LIBS instead of CFLAGS and LIBS for the result. This
is (1) more correct, and (2) necessary in order to give this location
priority over other directories in the include search path, which may include
system include directories with other versions of BDB.
ACKs for top commit:
theuni:
ACK 8578c6fccd11404412d2c60f9bede311b79ca0d0.
Tree-SHA512: a28f48fc81a25736f7e77c663f21cd9a6ae1cd115682031c5aa695c94cb5afa11920330a60cd6a54832822a2aec1eb23123ac2e2dcd4f0b3835aef9c9339ac97
Diffstat (limited to '.appveyor.yml')
0 files changed, 0 insertions, 0 deletions