Skip to content

Commit

Permalink
build: don't use BUILT_SOURCES for the static context header
Browse files Browse the repository at this point in the history
BUILT_SOURCES simply forces the header to be built first.

Use real dependencies instead. This fixes builds with dependency tracking
disabled where a real target is specified.

As a bonus, this also fixes the issue of gen_context and the header not being
rebuilt when gen_context.c is touched.
  • Loading branch information
theuni committed Jul 31, 2015
1 parent ae4f0c6 commit b8e39ac
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Makefile.am
Expand Up @@ -86,7 +86,8 @@ gen_%.o: src/gen_%.c
gen_context$(BUILD_EXEEXT): $(gen_context_OBJECTS)
$(CC_FOR_BUILD) $^ -o $@

BUILT_SOURCES = src/ecmult_static_context.h
$(libsecp256k1_la_OBJECTS): src/ecmult_static_context.h
$(tests_OBJECTS): src/ecmult_static_context.h

src/ecmult_static_context.h: gen_context
./gen_context
Expand Down

0 comments on commit b8e39ac

Please sign in to comment.