[coreboot] Patch set updated for coreboot: 0a3f239 crossgcc: properly test for flex

gerrit at coreboot.org gerrit at coreboot.org
Fri Nov 23 14:25:38 CET 2012


 just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1890

-gerrit

commit 0a3f2393aefd8fceea075e3108d581aadec292ab
Author: Patrick Georgi <patrick.georgi at secunet.com>
Date:   Tue Nov 20 12:04:32 2012 +0100

    crossgcc: properly test for flex
    
    This is no GNU tool, so testing for "GNU" in the version string
    is bound to fail.
    We now accept everything that returns success on "flex --version"
    and then hope for the best.
    
    I tested both cases
    
    Change-Id: If325f613fde1648847b998b7e8e5782d0f22b484
    Signed-off-by: Patrick Georgi <patrick.georgi at secunet.com>
    Reviewed-on: http://review.coreboot.org/1884
    Tested-by: build bot (Jenkins)
    Reviewed-by: Zheng Bao <zheng.bao at amd.com>
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
---
 util/crossgcc/buildgcc | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 91ec1cf..28e700b 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -119,7 +119,11 @@ MAKE=`searchgnu make` || exit $?
 
 searchgnu m4 > /dev/null
 searchgnu bison > /dev/null
-searchgnu flex > /dev/null
+
+if ! flex --version > /dev/null 2> /dev/null; then
+	printf "${RED}ERROR:${red} Missing toolchain: flex${NC}\n" >&2
+	exit 1
+fi
 
 cleanup()
 {




More information about the coreboot mailing list