diff options
author | Nicolas Boichat <drinkcat@chromium.org> | 2018-06-28 08:15:21 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-06-28 07:06:45 -0700 |
commit | 6f38ed23f3d467937f15cf0474b8f323564bcc0d (patch) | |
tree | 1e3e0e645638fd176ef184ce80debedf35d646c6 /Makefile.rules | |
parent | cacc1c8ac679b66876cf3e65c58f2bf874498050 (diff) | |
download | chrome-ec-6f38ed23f3d467937f15cf0474b8f323564bcc0d.tar.gz |
Makefile.toolchain: Switch from cpp to $(CC) -E
Makes it more straightforward to switch to clang preprocessor later
on.
This requires a few other modifications in the Makefiles (gcc -E does
not read from stdin by default, and does not process files that do not
end in .c/.h).
BRANCH=none
BUG=chromium:854924
TEST=make buildall -j
TEST=Before and after the change, all ec.bin binaries are identical:
git checkout m/master
make REAL_SIGNER=dummy buildall -j
mv build build-master
find build-master -type d | sed -e 's/-master//' | \
xargs -I{} mkdir -p {}
find build-master -name ec_version.h | sed -e 's|build-master/||' | \
xargs -I{} cp build-master/{} build/{}
git checkout this-cl
=> comment out .PHONY: $(out)/ec_version.h in Makefiles.rules
make REAL_SIGNER=dummy buildall -j
find build-master -name ec.bin | sed -e 's|build-master/||' | \
xargs -I{} diff build-master/{} build/{}
=> No difference
Change-Id: If07f033dc7e9a73245499c656562bb906dcd4130
Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1117721
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'Makefile.rules')
-rw-r--r-- | Makefile.rules | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Makefile.rules b/Makefile.rules index 50fe0a017d..9138b35ed9 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -286,7 +286,7 @@ $(foreach b, $(cts_boards), \ ) cov-test-targets=$(foreach t,$(test-list-host),build/host/$(t).info) -bldversion=$(shell (./util/getversion.sh ; echo VERSION) | $(CPP) -P) +bldversion=$(shell (./util/getversion.sh ; echo VERSION) | $(CPP) -P -) # lcov fails when multiple instances run at the same time. # We need to run them sequentially by using flock |