Skip to content

Commit

Permalink
tools/test/Makefile: disable unless we're in DEVELOPER mode.
Browse files Browse the repository at this point in the history
Because it required update-mocks, which is a hack which relies on the
format of linker errors (!) I'd prefer to make this --enable-developer
only.

Signed-off-by: Rusty Russell <[email protected]>
  • Loading branch information
rustyrussell committed Aug 21, 2019
1 parent c74c1e5 commit 278b129
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion tools/test/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,12 @@ TOOL_GEN_OBJS := $(TOOL_GEN_SRC:.c=.o)
TOOL_GEN_HEADER := tools/test/gen_test.h tools/test/gen_print.h
TOOL_TEST_SRC := $(wildcard tools/test/run-*.c)
TOOL_TEST_OBJS := $(TOOL_TEST_SRC:.c=.o)
# Don't require update-mocks hack unless developer mode.
ifeq ($(DEVELOPER),1)
TOOL_TEST_PROGRAMS := $(TOOL_TEST_OBJS:.o=)
else
TOOL_TEST_PROGRAMS :=
endif

TOOL_TEST_COMMON_OBJS := \
common/utils.o
Expand Down Expand Up @@ -41,7 +46,6 @@ tools/test/gen_print.c: $(TOOLS_WIRE_DEPS)
echo '#include "gen_test.h"' > $@
$(BOLT_GEN) -P --page impl ${@:.c=.h} test_type < tools/test/test_cases >> $@


ALL_TEST_PROGRAMS += $(TOOL_TEST_PROGRAMS)

check-tools: $(TOOL_TEST_PROGRAMS:%=unittest/%)
Expand Down

0 comments on commit 278b129

Please sign in to comment.