diff options
| author | Patrick Georgi <pgeorgi@google.com> | 2019-04-02 00:21:17 +0200 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2019-04-03 18:14:13 -0700 |
| commit | bc6c680b8ad09596a89451df40ee6375ea4d6274 (patch) | |
| tree | fe32b58bd52ec7d584f6b72042c4abcb87ff1678 | |
| parent | 23a89eccf3143790b89867fd32ad8331e53d49c0 (diff) | |
| download | chrome-ec-bc6c680b8ad09596a89451df40ee6375ea4d6274.tar.gz | |
Makefile.toolchain: strip leading whitespace in set-option
That function is only used to set CROSS_COMPILE, and we never need
leading whitespace in there. It sneaks in when the set-option call is
split across several lines, breaking the analyzestack target.
BUG=chromium:875295
BRANCH=none
TEST=make BOARD=yorp analyzestack works
Change-Id: I1b2dccd9753b0662067d77bd9c58d4ccf6daeb4d
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1547563
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
| -rw-r--r-- | Makefile.toolchain | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Makefile.toolchain b/Makefile.toolchain index f6fa2b51eb..c19ad273c0 100644 --- a/Makefile.toolchain +++ b/Makefile.toolchain @@ -143,4 +143,4 @@ HOST_TEST_LDFLAGS=-Wl,-T core/host/host_exe.lds -lrt -pthread -rdynamic -lm\ # $1: name of variable to set # $2: first default to use # $3: second default to use -set-option = $(eval $1=$$(if $(2),$(2),$(3))) +set-option = $(eval $1=$$(strip $$(if $(2),$(2),$(3)))) |