Skip to content

Commit 83a0bb7

Browse files
committed
build: Separate lib_LTLIBRARIES initialization
Set lib_LTLIBRARIES with '=' to an empty value at the top of the Makefile.am and append to it from the library-local block for readability. Here's the error you get if you don't set lib_LTLIBRARIES to be empty: error: lib_LTLIBRARIES must be set with '=' before using '+=' [META] In a subsequent commit, we're going to introduce a library and append it to lib_LTLIBRARIES in its local block, this makes things more readable.
1 parent c1e16cb commit 83a0bb7

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/Makefile.am

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ AM_LIBTOOLFLAGS = --preserve-dup-deps
1515
PTHREAD_FLAGS = $(PTHREAD_CFLAGS) $(PTHREAD_LIBS)
1616
EXTRA_LIBRARIES =
1717

18-
lib_LTLIBRARIES = $(LIBBITCOINCONSENSUS)
18+
lib_LTLIBRARIES =
1919
noinst_LTLIBRARIES =
2020

2121
bin_PROGRAMS =
@@ -895,6 +895,8 @@ bitcoin_chainstate-clientversion.$(OBJEXT): obj/build.h
895895

896896
# bitcoinconsensus library #
897897
if BUILD_BITCOIN_LIBS
898+
lib_LTLIBRARIES += $(LIBBITCOINCONSENSUS)
899+
898900
include_HEADERS = script/bitcoinconsensus.h
899901
libbitcoinconsensus_la_SOURCES = support/cleanse.cpp $(crypto_libbitcoin_crypto_base_la_SOURCES) $(libbitcoin_consensus_a_SOURCES)
900902

0 commit comments

Comments
 (0)