From 0feb1aadfdd0654f17b523035e57ddf35efd11cf Mon Sep 17 00:00:00 2001 From: Friedemann Kleint Date: Wed, 15 May 2013 13:51:46 +0200 Subject: [PATCH] configure/Mac: Handle errors by xcrun. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit xcrun occasionally prompts for a license or outputs errors which results in: configure: line 2488: [: : integer expression expected Check the output and bail out on error. Change-Id: Ic1ae62b5f19cf87365c38901e98d6b385cdb39a4 Reviewed-by: Tor Arne Vestbø --- configure | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a7c72c449e5..f13debc20dd 100755 --- a/configure +++ b/configure @@ -2483,7 +2483,10 @@ if [ -z "$PLATFORM" ]; then # about the OS version, since we're not using the clang version that comes # with the system. We use 'xcrun' to check the clang version that's part of # the Xcode installation. - if [ "$(/usr/bin/xcrun -sdk macosx clang -v 2>&1 | sed -n 's/.*version \([0-9]\).*/\1/p')" -ge 3 ]; then + XCRUN=`/usr/bin/xcrun -sdk macosx clang -v 2>&1` + CLANGVERSION=`echo "$XCRUN" | sed -n 's/.*version \([0-9]\).*/\1/p'` + expr "$CLANGVERSION" : '[0-9]' > /dev/null || { echo "Unable to determine CLANG version from output of xcrun: $XCRUN" ; exit 2 ; } + if [ "$CLANGVERSION" -ge 3 ]; then PLATFORM=macx-clang # Advertise g++ as an alternative on Lion and below