Gen context make #284

pull theuni wants to merge 2 commits into bitcoin-core:master from theuni:gen-context-make changing 1 files +7 −6

theuni commented at 5:37 PM on July 31, 2015: contributor

BUILT_SOURCES simply forces the header to be built first when no target is specified.

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

Also fixup the build so that "gen_context.exe" is used for windows.

build: don't use BUILT_SOURCES for the static context header

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.

b8e39ac5e4

build: fix hard-coded usage of "gen_context"
The binary needs an extension on some platforms.
e4ce393683
sipa commented at 5:43 PM on July 31, 2015: contributor

Looks good, thanks!
gmaxwell commented at 6:35 PM on August 3, 2015: contributor

Looks good to me. utACK.
sipa merged this on Aug 3, 2015
sipa closed this on Aug 3, 2015
sipa referenced this in commit ff3a5dfc97 on Aug 3, 2015

Contributors

theuni

sipa

gmaxwell