Skip to content

Commit 5596ec5

Browse files
Merge bitcoin-core#1203: Do not link bench and ctime_tests to COMMON_LIB
ef39721 Do not link `bench` and `ctime_tests` to `COMMON_LIB` (Hennadii Stepanov) Pull request description: The `bench` and `ctime_tests` binaries are users of the library, they should only be linked to the library, not the objects it was built from. ACKs for top commit: sipa: utACK ef39721 real-or-random: utACK ef39721 Tree-SHA512: 8bf8330adcce9bf6b21aceacf86e6aff7594762ab68b09257cfe2904fa0ce827377d5a13c0bed5acde74a2b420bb49460657c66d0068ecbe36dc162140876be4
2 parents e1817a6 + ef39721 commit 5596ec5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Makefile.am

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ noinst_PROGRAMS =
103103
if USE_BENCHMARK
104104
noinst_PROGRAMS += bench bench_internal bench_ecmult
105105
bench_SOURCES = src/bench.c
106-
bench_LDADD = libsecp256k1.la $(COMMON_LIB)
106+
bench_LDADD = libsecp256k1.la
107107
bench_CPPFLAGS = $(SECP_CONFIG_DEFINES)
108108
bench_internal_SOURCES = src/bench_internal.c
109109
bench_internal_LDADD = $(COMMON_LIB) $(PRECOMPUTED_LIB)
@@ -134,7 +134,7 @@ endif
134134
if USE_CTIME_TESTS
135135
noinst_PROGRAMS += ctime_tests
136136
ctime_tests_SOURCES = src/ctime_tests.c
137-
ctime_tests_LDADD = libsecp256k1.la $(COMMON_LIB)
137+
ctime_tests_LDADD = libsecp256k1.la
138138
ctime_tests_CPPFLAGS = $(SECP_CONFIG_DEFINES)
139139
endif
140140

0 commit comments

Comments
 (0)