From 58bfdbefc8377bba327324e1b26fae51c3f6d23f Mon Sep 17 00:00:00 2001 From: Nicky Date: Mon, 19 Jun 2017 02:12:57 +0200 Subject: [PATCH] Detect missing AUTOBUILD_VARIABLES_FILE environment variable and early fail configure (or later it will fail more cryptic due to missing LL_BUILD variable). --- scripts/configure_firestorm.sh | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/scripts/configure_firestorm.sh b/scripts/configure_firestorm.sh index 099a47e228..133a83593a 100755 --- a/scripts/configure_firestorm.sh +++ b/scripts/configure_firestorm.sh @@ -299,6 +299,21 @@ if [ $PLATFORM == "linux32" -o $PLATFORM == "linux64" -o $PLATFORM == "darwin" ] fi echo -e " Logging to $LOG" +if [ $PLATFORM == "win32" ] +then + if [ -z "${AUTOBUILD_VSVER}" ] + then + echo "AUTOBUILD_VSVER not set, this can lead to autobuild picking a higher VS version than desired." + echo "If you see this happen you should set the variable to eg 120 for Visual Studio 2013" + fi +fi + +if [ -z "$AUTOBUILD_VARIABLES_FILE" ] +then + echo "AUTOBUILD_VARIABLES_FILE not set." + echo "In order to run autobuild it needs to be set to point to a correct variables file" + exit 1 +fi if [ $PLATFORM == "win32" ] ; then FIND=/usr/bin/find