diff options
author | Brian Brooks <brian.brooks@arm.com> | 2017-06-22 17:05:38 -0500 |
---|---|---|
committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2017-06-23 12:24:58 +0300 |
commit | e7e92031e08fe525e421ab30223b7398881ed42e (patch) | |
tree | 11dd46b814bffc330f816f125cb58fccc9a3d889 /3ca26420f0e06fdb1b2cb9192461dbc32a421de5 | |
parent | 09a7800c4c4a093fb962e362952f9cf562d2fc98 (diff) |
build: fix conditional compilation of sources
Explicitly add all arch/<arch>/* files to respective _SOURCES
variables instead of using @ARCH_DIR@ substitution.
This patch fixes the broken build for ARM, PPC, and MIPS
introduced by [1] and the similar issue reported while
testing [2].
From the Autoconf manual [3]:
You can't put a configure substitution (e.g., '@FOO@' or
'$(FOO)' where FOO is defined via AC_SUBST) into a _SOURCES
variable. The reason for this is a bit hard to explain, but
suffice to say that it simply won't work.
Here be dragons..
[1] https://lists.linaro.org/pipermail/lng-odp/2017-April/030324.html
[2] https://lists.linaro.org/pipermail/lng-odp/2017-June/031598.html
[3] https://www.gnu.org/software/automake/manual/html_node/Conditional-Sources.html
Signed-off-by: Brian Brooks <brian.brooks@arm.com>
Reviewed-by: Kevin Wang <kevin.wang@arm.com>
Reviewed-by: Yi He <yi.he@arm.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
Reviewed-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Diffstat (limited to '3ca26420f0e06fdb1b2cb9192461dbc32a421de5')
0 files changed, 0 insertions, 0 deletions