check qmake/make exit code instead of looking for an executable
it's a bit braindead to look whether we managed to build an executable if we can simply use the exit codes of qmake and make to test for success. the windows equivalent is already doing that. this also allows us to do tests that can fail despite building an executable, or not build one in the first place. Change-Id: Ib69f9d005309d55a790dd3d89db1ee913e45b26b Reviewed-by: Wolfgang Bremer <wbremer@blackberry.com> Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
parent
ff9afeed51
commit
f946a1d0c1
@ -69,15 +69,11 @@ rm -f "$EXE" "${EXE}.exe"
|
|||||||
|
|
||||||
set -- "$OUTDIR/bin/qmake" -nocache -spec "$QMKSPEC" "CONFIG+=$QMAKE_CONFIG" "CONFIG+=android_app" "CONFIG-=debug_and_release app_bundle lib_bundle" "LIBS*=$LFLAGS" "LIBS+=$MAC_ARCH_LFLAGS" "INCLUDEPATH*=$INCLUDEPATH" "QMAKE_CXXFLAGS*=$CXXFLAGS" "QMAKE_CXXFLAGS+=$MAC_ARCH_CXXFLAGS" "QT_BUILD_TREE=$OUTDIR" "$SRCDIR/$TEST/$EXE.pro" -o "$OUTDIR/$TEST/Makefile"
|
set -- "$OUTDIR/bin/qmake" -nocache -spec "$QMKSPEC" "CONFIG+=$QMAKE_CONFIG" "CONFIG+=android_app" "CONFIG-=debug_and_release app_bundle lib_bundle" "LIBS*=$LFLAGS" "LIBS+=$MAC_ARCH_LFLAGS" "INCLUDEPATH*=$INCLUDEPATH" "QMAKE_CXXFLAGS*=$CXXFLAGS" "QMAKE_CXXFLAGS+=$MAC_ARCH_CXXFLAGS" "QT_BUILD_TREE=$OUTDIR" "$SRCDIR/$TEST/$EXE.pro" -o "$OUTDIR/$TEST/Makefile"
|
||||||
if [ "$VERBOSE" = "yes" ]; then
|
if [ "$VERBOSE" = "yes" ]; then
|
||||||
OUTDIR=$OUTDIR "$@"
|
OUTDIR=$OUTDIR "$@" && $MAKE && SUCCESS=yes
|
||||||
$MAKE
|
|
||||||
else
|
else
|
||||||
OUTDIR=$OUTDIR "$@" >/dev/null 2>&1
|
OUTDIR=$OUTDIR "$@" >/dev/null 2>&1 && $MAKE >/dev/null 2>&1 && SUCCESS=yes
|
||||||
$MAKE >/dev/null 2>&1
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
( [ -f "$EXE" ] || [ -f "${EXE}.exe" ] ) && SUCCESS=yes
|
|
||||||
|
|
||||||
# done
|
# done
|
||||||
if [ "$SUCCESS" != "yes" ]; then
|
if [ "$SUCCESS" != "yes" ]; then
|
||||||
[ "$VERBOSE" = "yes" ] && echo "$DESCRIPTION disabled."
|
[ "$VERBOSE" = "yes" ] && echo "$DESCRIPTION disabled."
|
||||||
|
Loading…
Reference in New Issue
Block a user