Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Dec 3, 2020

On master (a0489f3) the configure script is not able to determine that Homebrew's berkeley-db4 package is uninstalled. This causes a compile error on macOS.

With this PR, and with the uninstalled berkeley-db4 package:

% ./configure -q                  
configure: error: libdb_cxx headers missing, Bitcoin Core requires this library for BDB wallet support (--without-bdb to disable BDB wallet support)

Related #20478.

@hebasto
Copy link
Member Author

hebasto commented Dec 3, 2020

cc @jonasschnelli

@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 4, 2020

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@promag
Copy link
Contributor

promag commented Dec 4, 2020

Tested ACK d3ef947.

$ brew list --versions berkeley-db4
$ ./autogen.sh && ./configure
checking for Berkeley DB C++ headers... no
configure: error: libdb_cxx headers missing, Bitcoin Core requires this library for BDB wallet support (--without-bdb to disable BDB wallet support)

Copy link
Member

@willcl-ark willcl-ark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK d3ef947

will in bitcoin on  201203-bdb [$?] via 🐍 3.6.12 ➜ ./configure -q --without-gui
configure: error: libdb_cxx headers missing, Bitcoin Core requires this library for BDB wallet support (--without-bdb to disable BDB wallet support)

Copy link
Contributor

@jonasschnelli jonasschnelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup. This is much nicer.

utACK d3ef947

@jonasschnelli jonasschnelli merged commit 257cf05 into bitcoin:master Dec 4, 2020
@hebasto hebasto deleted the 201203-bdb branch December 4, 2020 09:43
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 4, 2020
maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Dec 10, 2020
@maflcko
Copy link
Member

maflcko commented Dec 10, 2020

Backported in #20612 because #20527 has been marked for backport

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants