Commit a8f98959 authored by Donald Carr's avatar Donald Carr Committed by Donald Carr
Browse files

Make CFLAGS/CXXFLAGS behave consistently in config.tests


The journald test was failing as sysroot was not being passed to the
compiler in the compile test. This is due to the fact we were explicitly
defining sysroot in the CXXFLAGS and not in CFLAGS.

Change-Id: I4b4bda71f9aabc8e420e0e89ff3cc8fa3bbca201
Reviewed-by: default avatarOswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Showing with 6 additions and 1 deletion
...@@ -13,6 +13,7 @@ DESCRIPTION=$7 ...@@ -13,6 +13,7 @@ DESCRIPTION=$7
shift 7 shift 7
LFLAGS="$SYSROOT_FLAG" LFLAGS="$SYSROOT_FLAG"
INCLUDEPATH="" INCLUDEPATH=""
CFLAGS="$SYSROOT_FLAG"
CXXFLAGS="$SYSROOT_FLAG" CXXFLAGS="$SYSROOT_FLAG"
MAC_ARCH_CXXFLAGS="" MAC_ARCH_CXXFLAGS=""
MAC_ARCH_LFLAGS="" MAC_ARCH_LFLAGS=""
...@@ -28,12 +29,14 @@ while [ "$#" -gt 0 ]; do ...@@ -28,12 +29,14 @@ while [ "$#" -gt 0 ]; do
shift shift
;; ;;
-arch) -arch)
MAC_ARCH_CFLAGS="$MAC_ARCH_CFLAGS -arch $2"
MAC_ARCH_CXXFLAGS="$MAC_ARCH_CXXFLAGS -arch $2" MAC_ARCH_CXXFLAGS="$MAC_ARCH_CXXFLAGS -arch $2"
MAC_ARCH_LFLAGS="$MAC_ARCH_LFLAGS -arch $2" MAC_ARCH_LFLAGS="$MAC_ARCH_LFLAGS -arch $2"
shift shift
;; ;;
-F*|-m*|-x*) -F*|-m*|-x*)
LFLAGS="$LFLAGS \"$PARAM\"" LFLAGS="$LFLAGS \"$PARAM\""
CFLAGS="$CFLAGS \"$PARAM\""
CXXFLAGS="$CXXFLAGS \"$PARAM\"" CXXFLAGS="$CXXFLAGS \"$PARAM\""
;; ;;
-L*|-l*|-pthread) -L*|-l*|-pthread)
...@@ -44,10 +47,12 @@ while [ "$#" -gt 0 ]; do ...@@ -44,10 +47,12 @@ while [ "$#" -gt 0 ]; do
INCLUDEPATH="$INCLUDEPATH \"$INC\"" INCLUDEPATH="$INCLUDEPATH \"$INC\""
;; ;;
-f*|-D*) -f*|-D*)
CFLAGS="$CFLAGS \"$PARAM\""
CXXFLAGS="$CXXFLAGS \"$PARAM\"" CXXFLAGS="$CXXFLAGS \"$PARAM\""
;; ;;
-Qoption) -Qoption)
# Two-argument form for the Sun Compiler # Two-argument form for the Sun Compiler
CFLAGS="$CFLAGS $PARAM \"$2\""
CXXFLAGS="$CXXFLAGS $PARAM \"$2\"" CXXFLAGS="$CXXFLAGS $PARAM \"$2\""
shift shift
;; ;;
...@@ -67,7 +72,7 @@ test -r Makefile && $MAKE distclean >/dev/null 2>&1 ...@@ -67,7 +72,7 @@ test -r Makefile && $MAKE distclean >/dev/null 2>&1
# Make sure output from possible previous tests is gone # Make sure output from possible previous tests is gone
rm -f "$EXE" "${EXE}.exe" 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" "$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_CFLAGS*=$CFLAGS" "QMAKE_CFLAGS+=$MAC_ARCH_CFLAGS" "QMAKE_CXXFLAGS*=$CXXFLAGS" "QMAKE_CXXFLAGS+=$MAC_ARCH_CXXFLAGS" "$SRCDIR/$TEST/$EXE.pro" -o "$OUTDIR/$TEST/Makefile"
if [ "$VERBOSE" = "yes" ]; then if [ "$VERBOSE" = "yes" ]; then
OUTDIR=$OUTDIR "$@" && $MAKE && SUCCESS=yes OUTDIR=$OUTDIR "$@" && $MAKE && SUCCESS=yes
else else
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment