diff options
| author | Patrick Georgi <pgeorgi@google.com> | 2016-02-10 12:49:08 +0100 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-02-10 12:44:15 -0800 |
| commit | e7b5e7b0506c5c2528ff008b02217b8b60434de9 (patch) | |
| tree | cca6126f140fa57c9893c33928a3d2ab4a163f40 /core/cortex-m0 | |
| parent | 192806b8da1b8714f6bfcdc548a7e8e7866b8384 (diff) | |
| download | chrome-ec-e7b5e7b0506c5c2528ff008b02217b8b60434de9.tar.gz | |
core/*/ec.lds.S: quote paths containing OUTDIR
If OUTDIR brings in a "@", the build breaks because that delimits the
path, leading to invalid file names.
This can happen (and happened) when building on a Jenkins CI instance
which uses jobname@number as path for parallel checkouts on a
single build node.
BRANCH=none
BUG=none
TEST=build with make out=foo@bar ... failed and works now.
Change-Id: Id0594f0d7312419110091443755ec11b5f8ee2d8
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/327110
Commit-Ready: Patrick Georgi <pgeorgi@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Martin Roth <martinroth@chromium.org>
Diffstat (limited to 'core/cortex-m0')
| -rw-r--r-- | core/cortex-m0/ec.lds.S | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/core/cortex-m0/ec.lds.S b/core/cortex-m0/ec.lds.S index 484029b7a3..6596b5e05d 100644 --- a/core/cortex-m0/ec.lds.S +++ b/core/cortex-m0/ec.lds.S @@ -5,6 +5,9 @@ #include "config.h" #include "rsa.h" +#define STRINGIFY0(name) #name +#define STRINGIFY(name) STRINGIFY0(name) + #define FW_OFF_(section) CONFIG_##section##_MEM_OFF #define FW_OFF(section) (CONFIG_PROGRAM_MEMORY_BASE + FW_OFF_(section)) @@ -36,7 +39,7 @@ SECTIONS } > SHARED_LIB #endif .text : { - OUTDIR/core/CORE/init.o (.text.vecttable) + STRINGIFY(OUTDIR/core/CORE/init.o) (.text.vecttable) . = ALIGN(4); __version_struct_offset = .; KEEP(*(.rodata.ver)) @@ -49,7 +52,7 @@ SECTIONS #else . = ALIGN(4); #endif - OUTDIR/core/CORE/init.o (.text) + STRINGIFY(OUTDIR/core/CORE/init.o) (.text) *(.text*) } > FLASH . = ALIGN(4); |