From c7e8977e60a33cc39294e7a9d2b03efdf2248076 Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Thu, 7 Aug 2025 11:41:43 -0700 Subject: [PATCH] Revert "Support Google Play 16 KB Page Size Requirement (#13470)" This reverts commit edef6e66e98b105124e4a73c43504b354bf7a20a If you need to support the Google Play 16 kiB page size requirement, the recommendation is to use NDK r28c or newer, which automatically aligns binaries correctly. --- Android.mk | 4 ---- CMakeLists.txt | 5 ----- android-project/app/build.gradle | 2 +- android-project/app/jni/Application.mk | 3 --- android-project/app/jni/src/CMakeLists.txt | 5 ----- build-scripts/androidbuildlibs.sh | 7 ------- 6 files changed, 1 insertion(+), 25 deletions(-) diff --git a/Android.mk b/Android.mk index 413967c40d..b5fe8955f9 100644 --- a/Android.mk +++ b/Android.mk @@ -107,10 +107,6 @@ LOCAL_LDLIBS := -ldl -lGLESv1_CM -lGLESv2 -lOpenSLES -llog -landroid LOCAL_LDFLAGS := -Wl,--no-undefined -Wl,--no-undefined-version -Wl,--version-script=$(LOCAL_PATH)/src/dynapi/SDL_dynapi.sym -# https://developer.android.com/guide/practices/page-sizes -LOCAL_LDFLAGS += "-Wl,-z,max-page-size=16384" -LOCAL_LDFLAGS += "-Wl,-z,common-page-size=16384" - ifeq ($(NDK_DEBUG),1) cmd-strip := endif diff --git a/CMakeLists.txt b/CMakeLists.txt index 108c3493fa..eb3e153b1a 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1487,11 +1487,6 @@ if(ANDROID) endif() endif() -if(TARGET SDL3-shared) - target_link_options(SDL3-shared PRIVATE "-Wl,-z,max-page-size=16384") - target_link_options(SDL3-shared PRIVATE "-Wl,-z,common-page-size=16384") -endif() - elseif(EMSCRIPTEN) # Hide noisy warnings that intend to aid mostly during initial stages of porting a new # project. Uncomment at will for verbose cross-compiling -I/../ path info. diff --git a/android-project/app/build.gradle b/android-project/app/build.gradle index f44cf26732..989ef34c71 100644 --- a/android-project/app/build.gradle +++ b/android-project/app/build.gradle @@ -19,7 +19,7 @@ android { abiFilters 'arm64-v8a' } cmake { - arguments "-DANDROID_PLATFORM=android-21", "-DANDROID_STL=c++_static", "-DAPP_SUPPORT_FLEXIBLE_PAGE_SIZES=true" + arguments "-DANDROID_PLATFORM=android-21", "-DANDROID_STL=c++_static" // abiFilters 'armeabi-v7a', 'arm64-v8a', 'x86', 'x86_64' abiFilters 'arm64-v8a' } diff --git a/android-project/app/jni/Application.mk b/android-project/app/jni/Application.mk index 80b73fd6bc..1f7c0c10f9 100644 --- a/android-project/app/jni/Application.mk +++ b/android-project/app/jni/Application.mk @@ -8,6 +8,3 @@ APP_ABI := armeabi-v7a arm64-v8a x86 x86_64 # Min runtime API level APP_PLATFORM=android-21 - -# https://developer.android.com/guide/practices/page-sizes#update-packaging -APP_SUPPORT_FLEXIBLE_PAGE_SIZES := true \ No newline at end of file diff --git a/android-project/app/jni/src/CMakeLists.txt b/android-project/app/jni/src/CMakeLists.txt index df0a4d0f34..41a82f2af0 100644 --- a/android-project/app/jni/src/CMakeLists.txt +++ b/android-project/app/jni/src/CMakeLists.txt @@ -26,9 +26,4 @@ endif() add_library(main SHARED YourSourceHere.c ) - -#https://developer.android.com/guide/practices/page-sizes#update-packaging -target_link_options(main PRIVATE "-Wl,-z,max-page-size=16384") -target_link_options(main PRIVATE "-Wl,-z,common-page-size=16384") - target_link_libraries(main PRIVATE SDL3::SDL3) diff --git a/build-scripts/androidbuildlibs.sh b/build-scripts/androidbuildlibs.sh index 1004a98703..a903f36eea 100755 --- a/build-scripts/androidbuildlibs.sh +++ b/build-scripts/androidbuildlibs.sh @@ -30,7 +30,6 @@ abi="arm64-v8a" # "armeabi-v7a arm64-v8a x86 x86_64" obj= lib= ndk_args= -flexpage=true # Allow an external caller to specify locations and platform. while [ $# -gt 0 ]; do @@ -43,8 +42,6 @@ while [ $# -gt 0 ]; do platform=${arg#APP_PLATFORM=} elif [ "${arg:0:8}" == "APP_ABI=" ]; then abi=${arg#APP_ABI=} - elif [ "${arg:0:32}" == "APP_SUPPORT_FLEXIBLE_PAGE_SIZES=" ]; then - flexpage=${arg#APP_SUPPORT_FLEXIBLE_PAGE_SIZES=} else ndk_args="$ndk_args $arg" fi @@ -70,9 +67,6 @@ done # APP_* variables set in the environment here will not be seen by the # ndk-build makefile segments that use them, e.g., default-application.mk. # For consistency, pass all values on the command line. -# -# Add support for Google Play 16 KB Page size requirement: -# https://developer.android.com/guide/practices/page-sizes#ndk-build ndk-build \ NDK_PROJECT_PATH=null \ NDK_OUT=$obj \ @@ -81,5 +75,4 @@ ndk-build \ APP_ABI="$abi" \ APP_PLATFORM="$platform" \ APP_MODULES="SDL3" \ - APP_SUPPORT_FLEXIBLE_PAGE_SIZES="$flexpage" \ $ndk_args