diff options
| author | Damien Lespiau <damien.lespiau@intel.com> | 2015-07-02 14:59:25 +0100 |
|---|---|---|
| committer | Damien Lespiau <damien.lespiau@intel.com> | 2015-07-02 15:23:35 +0100 |
| commit | ec9e5996d1c54d71c1c9cd14755ec6eddfdb0680 (patch) | |
| tree | 816c0404dc6d7540f2d7eae27891528c2f66ef5d /lib | |
| parent | 64295c67e908d59606fafe35e2d2363802c7faec (diff) | |
build: Don't use automake's conditional in a Makefile.sources
I just remembered that those Makefile.sources files where also
included by the Android build system, so we can't use automake's
conditionals in there.
So, we want to use GNU make's one. Unfortunately, after all those years,
GNU automake still doesn't do anything useful with GNU make's ifeq:
lib/Makefile.sources:66: error: else without if
automake will helpfully signal that the 'else' corresponding to the
'ifeq' doesn't have a corresponding 'if'. Well, yeah, thanks.
Fortunately, we can work around this by cunningly inserting a space
before 'ifeq', 'else' and 'endif' and fool automake's regex-based
checks.
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/Makefile.sources | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/Makefile.sources b/lib/Makefile.sources index a19ffd92..7f88b652 100644 --- a/lib/Makefile.sources +++ b/lib/Makefile.sources @@ -60,10 +60,11 @@ libintel_tools_la_SOURCES = \ .PHONY: version.h.tmp -if SKIP_GIT_HASH +# leaving a space here to work around automake's conditionals + ifeq ($(GIT_HASH),no) $(IGT_LIB_PATH)/version.h.tmp: @echo '#define IGT_GIT_SHA1 "git"' >> $@ -else + else $(IGT_LIB_PATH)/version.h.tmp: @touch $@ @if test -d $(GPU_TOOLS_PATH)/.git; then \ @@ -77,7 +78,7 @@ $(IGT_LIB_PATH)/version.h.tmp: else \ echo '#define IGT_GIT_SHA1 "NOT-GIT"' ; \ fi >> $@ -endif # SKIP_GIT_HASH + endif # GIT_HASH $(IGT_LIB_PATH)/version.h: $(IGT_LIB_PATH)/version.h.tmp |