[coreboot] New patch to review for coreboot: 09f8540 crossgcc: properly test for flex

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Tue Nov 20 12:05:55 CET 2012

Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1884


commit 09f854045b2d419639715c70ae3a9b276837958f
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>
 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

More information about the coreboot mailing list