Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/25901 )
Change subject: buildgcc: Add missing '' ......................................................................
buildgcc: Add missing ''
That caused the CFLAGS, CXXFLAGS, and '|| touch .failed' to not be taken into account when building binutils.
Change-Id: I94521eb73cefdc5ed01fbf10122966a54cc28166 Signed-off-by: Vivia Nikolaidou vivia.nikolaidou@puri.sm Reviewed-on: https://review.coreboot.org/25901 Reviewed-by: Youness Alaoui snifikino@gmail.com Reviewed-by: Matt DeVillier matt.devillier@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M util/crossgcc/buildgcc 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Matt DeVillier: Looks good to me, approved Youness Alaoui: Looks good to me, approved
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index 743937a..edcea7a 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -681,7 +681,7 @@ --target=${TARGETARCH} --enable-targets=${TARGETARCH}${ADDITIONALTARGET} \ --disable-werror --disable-nls --enable-lto --enable-gold \ --enable-interwork --enable-multilib \ - --enable-plugins --enable-multilibs + --enable-plugins --enable-multilibs \ CFLAGS="$HOSTCFLAGS" \ CXXFLAGS="$HOSTCFLAGS" \ || touch .failed