Skip to content

Commit 2cd4e3c

Browse files
committed
Drop no longer used SECP_{LIBS,INCLUDE} variables
The last usage of the `SECP_INCLUDE` variable was removed in bitcoin-core/secp256k1#1169.
1 parent 613626f commit 2cd4e3c

File tree

3 files changed

+17
-20
lines changed

3 files changed

+17
-20
lines changed

Makefile.am

+17-17
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ noinst_LTLIBRARIES = $(PRECOMPUTED_LIB)
7676
libsecp256k1_precomputed_la_SOURCES = src/precomputed_ecmult.c src/precomputed_ecmult_gen.c
7777
# We need `-I$(top_srcdir)/src` in VPATH builds if libsecp256k1_precomputed_la_SOURCES have been recreated in the build tree.
7878
# This helps users and packagers who insist on recreating the precomputed files (e.g., Gentoo).
79-
libsecp256k1_precomputed_la_CPPFLAGS = -I$(top_srcdir)/src $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
79+
libsecp256k1_precomputed_la_CPPFLAGS = -I$(top_srcdir)/src $(SECP_CONFIG_DEFINES)
8080

8181
if USE_EXTERNAL_ASM
8282
COMMON_LIB = libsecp256k1_common.la
@@ -95,31 +95,31 @@ endif
9595
endif
9696

9797
libsecp256k1_la_SOURCES = src/secp256k1.c
98-
libsecp256k1_la_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
99-
libsecp256k1_la_LIBADD = $(SECP_LIBS) $(COMMON_LIB) $(PRECOMPUTED_LIB)
98+
libsecp256k1_la_CPPFLAGS = $(SECP_CONFIG_DEFINES)
99+
libsecp256k1_la_LIBADD = $(COMMON_LIB) $(PRECOMPUTED_LIB)
100100
libsecp256k1_la_LDFLAGS = -no-undefined -version-info $(LIB_VERSION_CURRENT):$(LIB_VERSION_REVISION):$(LIB_VERSION_AGE)
101101

102102
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 $(SECP_LIBS) $(COMMON_LIB)
106+
bench_LDADD = libsecp256k1.la $(COMMON_LIB)
107107
bench_CPPFLAGS = $(SECP_CONFIG_DEFINES)
108108
bench_internal_SOURCES = src/bench_internal.c
109-
bench_internal_LDADD = $(SECP_LIBS) $(COMMON_LIB) $(PRECOMPUTED_LIB)
110-
bench_internal_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
109+
bench_internal_LDADD = $(COMMON_LIB) $(PRECOMPUTED_LIB)
110+
bench_internal_CPPFLAGS = $(SECP_CONFIG_DEFINES)
111111
bench_ecmult_SOURCES = src/bench_ecmult.c
112-
bench_ecmult_LDADD = $(SECP_LIBS) $(COMMON_LIB) $(PRECOMPUTED_LIB)
113-
bench_ecmult_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
112+
bench_ecmult_LDADD = $(COMMON_LIB) $(PRECOMPUTED_LIB)
113+
bench_ecmult_CPPFLAGS = $(SECP_CONFIG_DEFINES)
114114
endif
115115

116116
TESTS =
117117
if USE_TESTS
118118
TESTS += noverify_tests
119119
noinst_PROGRAMS += noverify_tests
120120
noverify_tests_SOURCES = src/tests.c
121-
noverify_tests_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
122-
noverify_tests_LDADD = $(SECP_LIBS) $(COMMON_LIB) $(PRECOMPUTED_LIB)
121+
noverify_tests_CPPFLAGS = $(SECP_CONFIG_DEFINES)
122+
noverify_tests_LDADD = $(COMMON_LIB) $(PRECOMPUTED_LIB)
123123
noverify_tests_LDFLAGS = -static
124124
if !ENABLE_COVERAGE
125125
TESTS += tests
@@ -134,19 +134,19 @@ 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 $(SECP_LIBS) $(COMMON_LIB)
137+
ctime_tests_LDADD = libsecp256k1.la $(COMMON_LIB)
138138
ctime_tests_CPPFLAGS = $(SECP_CONFIG_DEFINES)
139139
endif
140140

141141
if USE_EXHAUSTIVE_TESTS
142142
noinst_PROGRAMS += exhaustive_tests
143143
exhaustive_tests_SOURCES = src/tests_exhaustive.c
144-
exhaustive_tests_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
144+
exhaustive_tests_CPPFLAGS = $(SECP_CONFIG_DEFINES)
145145
if !ENABLE_COVERAGE
146146
exhaustive_tests_CPPFLAGS += -DVERIFY
147147
endif
148148
# Note: do not include $(PRECOMPUTED_LIB) in exhaustive_tests (it uses runtime-generated tables).
149-
exhaustive_tests_LDADD = $(SECP_LIBS) $(COMMON_LIB)
149+
exhaustive_tests_LDADD = $(COMMON_LIB)
150150
exhaustive_tests_LDFLAGS = -static
151151
TESTS += exhaustive_tests
152152
endif
@@ -190,12 +190,12 @@ EXTRA_PROGRAMS = precompute_ecmult precompute_ecmult_gen
190190
CLEANFILES = $(EXTRA_PROGRAMS)
191191

192192
precompute_ecmult_SOURCES = src/precompute_ecmult.c
193-
precompute_ecmult_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
194-
precompute_ecmult_LDADD = $(SECP_LIBS) $(COMMON_LIB)
193+
precompute_ecmult_CPPFLAGS = $(SECP_CONFIG_DEFINES)
194+
precompute_ecmult_LDADD = $(COMMON_LIB)
195195

196196
precompute_ecmult_gen_SOURCES = src/precompute_ecmult_gen.c
197-
precompute_ecmult_gen_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
198-
precompute_ecmult_gen_LDADD = $(SECP_LIBS) $(COMMON_LIB)
197+
precompute_ecmult_gen_CPPFLAGS = $(SECP_CONFIG_DEFINES)
198+
precompute_ecmult_gen_LDADD = $(COMMON_LIB)
199199

200200
# See Automake manual, Section "Errors with distclean".
201201
# We don't list any dependencies for the prebuilt files here because

configure.ac

-2
Original file line numberDiff line numberDiff line change
@@ -406,8 +406,6 @@ fi
406406
###
407407

408408
AC_CONFIG_FILES([Makefile libsecp256k1.pc])
409-
AC_SUBST(SECP_INCLUDES)
410-
AC_SUBST(SECP_LIBS)
411409
AC_SUBST(SECP_CFLAGS)
412410
AC_SUBST(SECP_CONFIG_DEFINES)
413411
AM_CONDITIONAL([ENABLE_COVERAGE], [test x"$enable_coverage" = x"yes"])

libsecp256k1.pc.in

-1
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,4 @@ URL: https://github.com/bitcoin-core/secp256k1
99
Version: @PACKAGE_VERSION@
1010
Cflags: -I${includedir}
1111
Libs: -L${libdir} -lsecp256k1
12-
Libs.private: @SECP_LIBS@
1312

0 commit comments

Comments
 (0)