diff options
| author | Colin Snover | 2017-10-25 01:27:36 -0500 |
|---|---|---|
| committer | Bastien Bouclet | 2019-05-11 20:35:31 +0200 |
| commit | 30c49838bbd8d8f51ae0c4c42a4b423b1572623c (patch) | |
| tree | f2ae343d5eda9c7cc0484694ad10a904a9ce17ef /configure | |
| parent | 0b4acfeed04f741ac0faaebdea976e52243ef618 (diff) | |
| download | scummvm-rg350-30c49838bbd8d8f51ae0c4c42a4b423b1572623c.tar.gz scummvm-rg350-30c49838bbd8d8f51ae0c4c42a4b423b1572623c.tar.bz2 scummvm-rg350-30c49838bbd8d8f51ae0c4c42a4b423b1572623c.zip | |
BUILD: Remove bad & outdated flags from Android builds
Most of these flags don't work with modern Android NDKs. The ones
that may, like setting sysroot and include directories, don't work
properly with standalone toolchains. Users compiling for Android
should make sure that they properly set up include paths in their
environment CXXFLAGS/LDFLAGS as needed for the compiler to run,
instead of relying on configure to do it, since configure can't do
it in a toolchain-agnostic manner.
Diffstat (limited to 'configure')
| -rwxr-xr-x | configure | 34 |
1 files changed, 2 insertions, 32 deletions
@@ -2495,7 +2495,8 @@ case $_host_os in android | android-arm) append_var CXXFLAGS "-march=armv5te" append_var CXXFLAGS "-mtune=xscale" - append_var CXXFLAGS "-msoft-float" + append_var CXXFLAGS "-mfloat-abi=softfp" + append_var LDFLAGS "-mthumb" ABI="armeabi" ANDROID_PLATFORM=9 ANDROID_PLATFORM_ARCH="arm" @@ -2536,10 +2537,6 @@ case $_host_os in ;; esac - # Setup platform version and arch - append_var CXXFLAGS "--sysroot=$ANDROID_NDK/platforms/android-$ANDROID_PLATFORM/arch-$ANDROID_PLATFORM_ARCH" - append_var LDFLAGS "--sysroot=$ANDROID_NDK/platforms/android-$ANDROID_PLATFORM/arch-$ANDROID_PLATFORM_ARCH" - append_var CXXFLAGS "-fpic" append_var CXXFLAGS "-ffunction-sections" append_var CXXFLAGS "-funwind-tables" @@ -2550,36 +2547,9 @@ case $_host_os in append_var CXXFLAGS "-fomit-frame-pointer" append_var CXXFLAGS "-fstrict-aliasing" fi - append_var CXXFLAGS "-finline-limit=300" _optimization_level=-Os - if test "$_host" = android -o "$_host" = android-arm; then - append_var CXXFLAGS "-mthumb-interwork" - # FIXME: Why is the following in CXXFLAGS and not in DEFINES? Change or document this. - append_var CXXFLAGS "-D__ARM_ARCH_5__" - append_var CXXFLAGS "-D__ARM_ARCH_5T__" - append_var CXXFLAGS "-D__ARM_ARCH_5E__" - append_var CXXFLAGS "-D__ARM_ARCH_5TE__" - fi - - # surpress 'mangling of 'va_list' has changed in GCC 4.4' warning - append_var CXXFLAGS "-Wno-psabi" - - if test "$_host" = android -o "$_host" = android-arm; then - append_var LDFLAGS "-mthumb-interwork" - fi - - append_var INCLUDES "-isystem $ANDROID_NDK/sources/cxx-stl/gnu-libstdc++/`$CXX -dumpversion`/include/" - append_var INCLUDES "-isystem $ANDROID_NDK/sources/cxx-stl/gnu-libstdc++/`$CXX -dumpversion`/libs/$ABI/include/" - append_var LDFLAGS "-L$ANDROID_NDK/sources/cxx-stl/gnu-libstdc++/`$CXX -dumpversion`/libs/$ABI/" - append_var LIBS "-lsupc++" add_line_to_config_mk "ANDROID_SDK = $ANDROID_SDK" - if test -d "$ANDROID_SDK"/build-tools; then - _build_tools_version=`cd "$ANDROID_SDK"/build-tools && ls -1 | sort -rn | head -1` - add_line_to_config_mk "ANDROID_BTOOLS = build-tools/$_build_tools_version" - else - add_line_to_config_mk "ANDROID_BTOOLS = platform-tools" - fi _seq_midi=no ;; beos*) |