[coreboot-gerrit] New patch to review for coreboot: buildgcc: improve portability of 'type' use

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jun 11 21:09:04 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10517

-gerrit

commit 93c8c5e0444c2ab484a4e3164ef79c2ed4eda6d3
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Thu Jun 11 21:04:31 2015 +0200

    buildgcc: improve portability of 'type' use
    
    The precise phrase returned by 'type' differs between locales and shells.
    It also doesn't matter because it returns an error code when it hasn't found a
    match.
    Let's simply assume there's no build_$OneOfOurPackages commands around that
    could also match.
    
    Change-Id: I44f021243149701e8da9dd74c368ca2ad4509419
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Tested-on: linux bash, linux dash, solaris sh, solaris ksh.
---
 util/crossgcc/buildgcc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 3bee6dc..ecc7ad4 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -235,7 +235,7 @@ unpack_and_patch() {
 
 fn_exists()
 {
-	type $1 2>/dev/null | grep -q 'is a function'
+	type $1 >/dev/null 2>&1
 }
 
 build() {



More information about the coreboot-gerrit mailing list