[coreboot] New patch to review for coreboot: 1bb5f1d buildgcc: Dont get misled if sysctl gets nothing

Zheng Bao (zheng.bao@amd.com) gerrit at coreboot.org
Fri Sep 28 08:21:15 CEST 2012


Zheng Bao (zheng.bao at amd.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1548

-gerrit

commit 1bb5f1d81602781aecfa807da17fd872ec3fc89b
Author: Zheng Bao <fishbaozi at gmail.com>
Date:   Fri Sep 28 16:16:40 2012 +0800

    buildgcc: Dont get misled if sysctl gets nothing
    
    Change-Id: I7cd63248eb8abb711cecce41e3f8a282b34aa126
    Signed-off-by: Zheng Bao <zheng.bao at amd.com>
    Signed-off-by: Zheng Bao <fishbaozi at gmail.com>
---
 util/crossgcc/buildgcc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index b79c5f6..67477a5 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -315,7 +315,7 @@ if [ `uname` = "Darwin" ]; then
 	# binaries in 10.6 (even if the kernel is 32bit)
 	# For some weird reason, 10.5 autodetects an ABI=64 though
 	# so we're setting the ABI explicitly here.
-	if [ `sysctl -n hw.optional.x86_64` -eq 1 ]; then
+	if [ -n "`sysctl -n hw.optional.x86_64 2>/dev/null`" ] && [ `sysctl -n hw.optional.x86_64 2>/dev/null` -eq 1 ]; then
 		OPTIONS="ABI=64"
 	else
 		OPTIONS="ABI=32"




More information about the coreboot mailing list