[coreboot-gerrit] New patch to review for coreboot: xcompile: Remove warnings about missing tools & architectures

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Mon Dec 7 22:37:43 CET 2015


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12682

-gerrit

commit e1545c4c9924abfd88cadbb084d73ec118ca5899
Author: Martin Roth <martinroth at google.com>
Date:   Mon Dec 7 14:34:46 2015 -0700

    xcompile: Remove warnings about missing tools & architectures
    
    Let toolchain.inc error out when the architecture or tool is missing.
    
    Change-Id: I39a51e5a2c778d6bbc50354807e5e2b717fa9e52
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/xcompile/xcompile | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 7f44df6..0b864d3 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -56,8 +56,6 @@ if [ "$(${XGCCPATH}/iasl 2>/dev/null | grep -c ACPI)" -gt 0 ]; then
 	IASL=${XGCCPATH}iasl
 elif [ "$(iasl 2>/dev/null | grep -c ACPI)" -gt 0 ]; then
 	IASL=iasl
-else
-	die "no iasl found"
 fi
 
 if program_exists gcc; then
@@ -350,11 +348,6 @@ test_architecture() {
 		# that this will change in the future.
 		CLANG="clang -target ${clang_arch}-${TABI} -ccc-gcc-name ${GCC}"
 	fi
-
-	if [ -z "$GCC" -a -z "$CLANG" ]; then
-		echo "Warning: no suitable compiler for $architecture." >&2
-		return 1
-	fi
 }
 
 # This loops over all supported architectures.



More information about the coreboot-gerrit mailing list