[coreboot-gerrit] Change in coreboot[master]: util/xcompile: Allow overriding default compiler path

[Patrick Georgi (Code Review)]

Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/56643 )

Change subject: util/xcompile: Allow overriding default compiler path
......................................................................

util/xcompile: Allow overriding default compiler path

When looking for C compilers, xcompile uses the "" prefix to "gcc" and
"clang" as a last-resort option. This fails in environments where such
default names are blocked to prevent "unclean" builds - such as Chrome
OS.

Allow overriding this prefix using the GENERIC_COMPILER_PREFIX variable
that is hopefully both descriptive enough to suggest what it is for and
unusual enough to not trigger by chance.

Change-Id: I16239f66730f1dbcb7482f223cea4ee5957af10c
Signed-off-by: Patrick Georgi 
---
M util/xcompile/xcompile
1 file changed, 4 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/56643/1

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 9fdda03..4950a9e 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -10,6 +10,8 @@
    set -x
 fi

+# GENERIC_COMPILER_PREFIX defaults to empty but can be used to override
+# compiler search behavior
 TMPFILE=""
 XGCCPATH=$1

@@ -421,7 +423,7 @@

# Search toolchain by checking assembler capability.
    for TBFDARCH in $TBFDARCHS; do
-		for gccprefix in $search ""; do
+		for gccprefix in $search "$GENERIC_COMPILER_PREFIX"; do
    		program_exists "${gccprefix}as" || continue
    		for endian in $TENDIAN ""; do
    			{ testas "$gccprefix" "$TWIDTH" "$TBFDARCH" \
@@ -439,7 +441,7 @@
    fi

for clang_arch in $TCLIST invalid; do
-		for clang_prefix in $search $XGCCPATH ""; do
+		for clang_prefix in $search $XGCCPATH "$GENERIC_COMPILER_PREFIX"; do
    		testcc "${clang_prefix}clang" "-target ${clang_arch}-$TABI -c" && break 2
    	done
    done