Skip to content

Commit e862c4a

Browse files
committed
Makefile: add -I$(top_srcdir)/src to CPPFLAGS for precomputed
When performing an out-of-source-tree build, regenerating the source files for the precomputed ecmult tables places them outside the source tree. Then, when they are to be compiled, they cannot find the headers they need because the source tree is absent from their include search path. This appears to have been an oversight, as the relevant -I options are present in libsecp256k1_la_CPPFLAGS but were missing from libsecp256k1_precomputed_la_CPPFLAGS. This commit adds them.
1 parent b627ba7 commit e862c4a

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

Makefile.am

+3-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,9 @@ noinst_HEADERS += examples/random.h
7373
PRECOMPUTED_LIB = libsecp256k1_precomputed.la
7474
noinst_LTLIBRARIES = $(PRECOMPUTED_LIB)
7575
libsecp256k1_precomputed_la_SOURCES = src/precomputed_ecmult.c src/precomputed_ecmult_gen.c
76-
libsecp256k1_precomputed_la_CPPFLAGS = $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
76+
# We need `-I$(top_srcdir)/src` in VPATH builds if libsecp256k1_precomputed_la_SOURCES have been recreated in the build tree.
77+
# This helps users and packagers who insist on recreating the precomputed files (e.g., Gentoo).
78+
libsecp256k1_precomputed_la_CPPFLAGS = -I$(top_srcdir)/src $(SECP_INCLUDES) $(SECP_CONFIG_DEFINES)
7779

7880
if USE_EXTERNAL_ASM
7981
COMMON_LIB = libsecp256k1_common.la

0 commit comments

Comments
 (0)