Skip to content

Commit

Permalink
Allow $JAVA_HOME without java installed in deploy_tests.py
Browse files Browse the repository at this point in the history
As long as $JAVA_HOME is defined, gradle doesn't need java to be on the path.

Diffs=
aaaf1a206e Allow $JAVA_HOME without java installed in deploy_tests.py (#8699)
f653f3f73f Support 16kb devices by bumping to NDK r27 (#8558)

Co-authored-by: Erik <[email protected]>
  • Loading branch information
csmartdalton and ErikUggeldahl committed Dec 10, 2024
1 parent b341ea3 commit 7da0bfe
Show file tree
Hide file tree
Showing 11 changed files with 31 additions and 27 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,8 @@ jobs:
set -x
echo "y" | sdkmanager --install 'build-tools;34.0.0' platform-tools --sdk_root=${ANDROID_SDK_ROOT}
cd ${ANDROID_HOME}
wget -q https://dl.google.com/android/repository/android-ndk-r25b-linux.zip
unzip -q android-ndk-r25b-linux.zip
wget -q https://dl.google.com/android/repository/android-ndk-r27c-linux.zip
unzip -q android-ndk-r27c-linux.zip
echo "y" | sdkmanager --install 'cmake;3.22.1' --channel=0 --sdk_root=${ANDROID_SDK_ROOT}
- name: Installing pre-requisites
Expand All @@ -57,5 +57,5 @@ jobs:
env:
# ANDROID_SDK_ROOT has been in the env by 'setup-android' above
# and is => /usr/local/lib/android/sdk
NDK_PATH: ${{ env.ANDROID_SDK_ROOT }}/android-ndk-r25b
NDK_PATH: ${{ env.ANDROID_SDK_ROOT }}/android-ndk-r27c
run: ./gradlew kotlin:assembleRelease
6 changes: 3 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ jobs:
set -x
echo "y" | sdkmanager --install 'build-tools;34.0.0' platform-tools --sdk_root=${ANDROID_SDK_ROOT}
cd ${ANDROID_HOME}
wget -q https://dl.google.com/android/repository/android-ndk-r25b-linux.zip
unzip -q android-ndk-r25b-linux.zip
wget -q https://dl.google.com/android/repository/android-ndk-r27c-linux.zip
unzip -q android-ndk-r27c-linux.zip
echo "y" | sdkmanager --install 'cmake;3.22.1' --channel=0 --sdk_root=${ANDROID_SDK_ROOT}
- name: Configure venv
Expand Down Expand Up @@ -109,7 +109,7 @@ jobs:
env:
# ANDROID_SDK_ROOT has been in the env by 'setup-android' above
# and is => /usr/local/lib/android/sdk
NDK_PATH: ${{ env.ANDROID_SDK_ROOT }}/android-ndk-r25b
NDK_PATH: ${{ env.ANDROID_SDK_ROOT }}/android-ndk-r27c
run: ./gradlew kotlin:assembleRelease

# Runs upload, and then closes & releases the repository
Expand Down
2 changes: 1 addition & 1 deletion .rive_head
Original file line number Diff line number Diff line change
@@ -1 +1 @@
572265df3bc40447ed8aa36efa2581eb8c2f844d
aaaf1a206eb823a2531952c67adec0184af35aef
2 changes: 1 addition & 1 deletion app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ plugins {
android {
compileSdk 35
namespace "app.rive.runtime.example"
ndkVersion "25.1.8937393"
ndkVersion "27.2.12479018"

defaultConfig {
applicationId "app.rive.runtime.example"
Expand Down
4 changes: 2 additions & 2 deletions compatibilitytest/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ plugins {
}

android {
compileSdk 35
namespace "app.rive.runtime.compatibilitytest"
ndkVersion "25.1.8937393"
compileSdk 35
ndkVersion "27.2.12479018"

defaultConfig {
applicationId "app.rive.runtime.compatibilitytest"
Expand Down
8 changes: 6 additions & 2 deletions kotlin/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ plugins {

android {
compileSdk 35
ndkVersion "25.1.8937393"
ndkVersion "27.2.12479018"
namespace "app.rive.runtime.kotlin"

defaultConfig {
Expand All @@ -19,7 +19,11 @@ android {
cmake {
abiFilters "arm64-v8a", "armeabi-v7a", "x86", "x86_64"
arguments "-DCMAKE_VERBOSE_MAKEFILE=1", "-DANDROID_ALLOW_UNDEFINED_SYMBOLS=ON",
"-DANDROID_CPP_FEATURES=no-exceptions no-rtti", "-DANDROID_STL=c++_shared"
"-DANDROID_CPP_FEATURES=no-exceptions no-rtti", "-DANDROID_STL=c++_shared",
// Support for 16kb page sizes, necessary for NDK r27
// Can remove when upgrading to r28+
// https://developer.android.com/guide/practices/page-sizes#compile-r27
"-DANDROID_SUPPORT_FLEXIBLE_PAGE_SIZES=ON"
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion kotlin/src/main/cpp/.ndk_version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
25.1.8937393
27.2.12479018
2 changes: 1 addition & 1 deletion kotlin/src/main/cpp/.ndk_version.bots
Original file line number Diff line number Diff line change
@@ -1 +1 @@
r25b
r27c
12 changes: 6 additions & 6 deletions kotlin/src/main/cpp/build.rive.for.sh
Original file line number Diff line number Diff line change
Expand Up @@ -79,13 +79,13 @@ if [[ -z ${NDK_PATH+x} ]]; then
elif [[ ${NDK_PATH} != *${EXPECTED_NDK_VERSION}* ]]; then
echo "Wrong NDK version"
echo "Expected: /Users/<username>/Library/Android/sdk/ndk/${EXPECTED_NDK_VERSION}"
echo " For bot builds, googles NDK distros"
echo " we are currently using: https://github.com/android/ndk/wiki/Unsupported-Downloads, you should be able to get android-ndk-r25b-darwin"
echo " For bot builds, Google's NDK distros,"
echo " we are currently using: https://github.com/android/ndk/wiki, you should be able to get android-ndk-r27c-darwin"
echo " For human builds"
echo " - open android studio"
echo " - settings search for android sdk, then SDK tools"
echo " - check "show package details" at the bottom"
echo " - select 25.1.8937393 in NDK (Side by Side)"
echo " - Open Android Studio"
echo " - Settings > Search for SDK Tools"
echo " - check "Show Package Details" at the bottom"
echo " - select 27.2.12479018 in NDK (Side by Side)"
echo "Found ${NDK_PATH}"
exit 1
fi
Expand Down
12 changes: 6 additions & 6 deletions kotlin/src/main/cpp/test/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,13 @@ if [[ -z ${NDK_PATH+x} ]]; then
elif [[ ${NDK_PATH} != *${EXPECTED_NDK_VERSION}* ]]; then
echo "Wrong NDK version"
echo "Expected: /Users/<username>/Library/Android/sdk/ndk/${EXPECTED_NDK_VERSION}"
echo " For bot builds, googles NDK distros"
echo " we are currently using: https://github.com/android/ndk/wiki/Unsupported-Downloads, you should be able to get android-ndk-r25b-darwin"
echo " For bot builds, Google's NDK distros,"
echo " we are currently using: https://github.com/android/ndk/wiki, you should be able to get android-ndk-r27c-darwin"
echo " For human builds"
echo " - open android studio"
echo " - settings search for android sdk, then SDK tools"
echo " - check "show package details" at the bottom"
echo " - select ${EXPECTED_NDK_VERSION} in NDK (Side by Side)"
echo " - Open Android Studio"
echo " - Settings > Search for SDK Tools"
echo " - check "Show Package Details" at the bottom"
echo " - select 27.2.12479018 in NDK (Side by Side)"
echo "Found ${NDK_PATH}"
exit 1
fi
Expand Down
2 changes: 1 addition & 1 deletion submodules/rive-runtime

0 comments on commit 7da0bfe

Please sign in to comment.