[coreboot-gerrit] Patch set updated for coreboot: xcompile: Redirect the objdump stderr to /dev/null

Zheng Bao (zheng.bao@amd.com) gerrit at coreboot.org
Thu Nov 5 11:51:29 CET 2015


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

-gerrit

commit 754637536da99fb849fcb938ac1c7044b6f3073b
Author: zbao <fishbaozi at gmail.com>
Date:   Fri Sep 18 06:17:09 2015 -0700

    xcompile: Redirect the objdump stderr to /dev/null
    
    On system with clang, "as" is available but "objdump" is not by default.
    So if ${gccprefix} is empty, "as" can run successfully and the "objdump"
    below might report error. Mask that output.
    
    Change-Id: I9940f069f66e097973ed6138cf3c696087fa5531
    Signed-off-by: Zheng Bao <zheng.bao at amd.com>
    Signed-off-by: Zheng Bao <fishbaozi at gmail.com>
---
 util/xcompile/xcompile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 76c3698..a8134a0 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -111,7 +111,7 @@ testas() {
 		2>/dev/null || return 1
 
 	# Check output content type.
-	local obj_type="$(LANG=C LC_ALL= ${gccprefix}objdump -p $obj_file)"
+	local obj_type="$(LANG=C LC_ALL= ${gccprefix}objdump -p $obj_file 2>/dev/null)"
 	local obj_arch="$(expr "$obj_type" : '.*format \(.[a-z0-9-]*\)')"
 	[ "$obj_arch" = "$full_arch" ] || return 1
 



More information about the coreboot-gerrit mailing list