[coreboot-gerrit] Patch set updated for coreboot: buildgcc: Quote command substitution

Nico Huber (nico.h@gmx.de) gerrit at coreboot.org
Sun Sep 11 11:57:07 CEST 2016


Nico Huber (nico.h at gmx.de) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15820

-gerrit

commit 7262a41a2a252efcbefef344a79da7d0774facaa
Author: Idwer Vollering <vidwer at gmail.com>
Date:   Sun Jul 24 02:10:19 2016 +0200

    buildgcc: Quote command substitution
    
    There are shells where the result of a command substitution is subject
    to word splitting (e.g. dash when assigning a value inside an export
    statement).
    
    Change-Id: I70a5bc124af7ee621da2bdb4777f3eaba8adafbb
    Signed-off-by: Idwer Vollering <vidwer at gmail.com>
    Signed-off-by: Nico Huber <nico.h at gmx.de>
---
 util/crossgcc/buildgcc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index d858919..9eb72f9 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -442,8 +442,8 @@ have_hostcflags_from_gmp() {
 set_hostcflags_from_gmp() {
 	# Now set CFLAGS to match GMP CFLAGS but strip out -pedantic
 	# as GCC 4.6.x fails if it's there.
-	export HOSTCFLAGS=$(grep __GMP_CFLAGS $DESTDIR$TARGETDIR/include/gmp.h |cut -d\" -f2 |\
-	    sed s,-pedantic,,)
+	export HOSTCFLAGS="$(grep __GMP_CFLAGS $DESTDIR$TARGETDIR/include/gmp.h |cut -d\" -f2 |\
+	    sed s,-pedantic,,)"
 }
 
 build_GMP() {



More information about the coreboot-gerrit mailing list