diff options
| author | fanquake <fanquake@gmail.com> | 2021-10-21 08:51:47 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-10-21 09:31:20 +0800 |
| commit | 4229f71bf8c25eb7a95b16a5f9ae1d6c362a3265 (patch) | |
| tree | 410369bb3afffd49b2f85425b37cd91f2bb72357 /src/Makefile.am | |
| parent | 69986de383ffea500e2e22975f97f96427941724 (diff) | |
| parent | 17ae2601c786e6863cee1bd62297d79521219295 (diff) | |
| download | bitcoin-4229f71bf8c25eb7a95b16a5f9ae1d6c362a3265.tar.xz | |
Merge bitcoin/bitcoin#23282: build: remove build stubs for external leveldb
17ae2601c786e6863cee1bd62297d79521219295 build: remove build stubs for external leveldb (Cory Fields)
Pull request description:
Presumably these stubs indicate to packagers that external leveldb is meant to be supported in some way. It is not. Remove the stubs to avoid sending any mixed messages.
For context, this was reported on IRC:
> \<Talkless> bitcoind fails to start with undefined symbol: _ZTIN7leveldb6LoggerE in Debian Sid after leveldb upgraded from 1.22 to 1.23: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=996486
ACKs for top commit:
fanquake:
ACK 17ae2601c786e6863cee1bd62297d79521219295
hebasto:
ACK 17ae2601c786e6863cee1bd62297d79521219295. I have reviewed the code and it looks OK, I agree it can be merged.
Tree-SHA512: 2f1ac2cb30dac64791933a245a2b66ce237bde3955e6f4a6b7ec181248f77a9b1b10597d865d3e2c2b6def696af70de40e905ec274e4ae7cccd1daf461473957
Diffstat (limited to 'src/Makefile.am')
| -rw-r--r-- | src/Makefile.am | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/Makefile.am b/src/Makefile.am index 68c81c1a12..e5cf8d9b74 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -841,11 +841,8 @@ nodist_libbitcoin_ipc_a_SOURCES = $(libbitcoin_ipc_mpgen_output) CLEANFILES += $(libbitcoin_ipc_mpgen_output) endif -if EMBEDDED_LEVELDB include Makefile.crc32c.include include Makefile.leveldb.include -endif - include Makefile.test_util.include include Makefile.test_fuzz.include |