Skip to content

Commit

Permalink
Android - Renaming the library generated .so files (#2201)
Browse files Browse the repository at this point in the history
  • Loading branch information
Sideness authored Mar 27, 2024
1 parent a79f984 commit 7599611
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/android-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ jobs:
- name: Run Android unit tests
run: make run-android-unit-test

- name: Build libmapbox-gl.so for arm-v8
- name: Build libmaplibre.so for arm-v8
run: make android-lib-arm-v8

- name: Copy developer config with API key for UI tests
Expand Down
4 changes: 2 additions & 2 deletions platform/android/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ run-android-core-test-$1-%: android-core-test-$1

# Push all needed files to the device
adb push $(MLN_ANDROID_CORE_TEST_DIR)/classes.dex $(MLN_ANDROID_LOCAL_WORK_DIR) > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/intermediate-jars/$(buildtype)/jni/$2/libmapbox-gl.so $(MLN_ANDROID_LOCAL_WORK_DIR) > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/intermediate-jars/$(buildtype)/jni/$2/libmaplibre.so $(MLN_ANDROID_LOCAL_WORK_DIR) > /dev/null 2>&1
adb push test/fixtures $(MLN_ANDROID_LOCAL_WORK_DIR)/test > /dev/null 2>&1
adb push scripts/style-spec-reference/v8.json $(MLN_ANDROID_LOCAL_WORK_DIR)/scripts/style-spec-reference > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/cmake/$(buildtype)/obj/$2/mbgl-test $(MLN_ANDROID_LOCAL_WORK_DIR) > /dev/null 2>&1
Expand Down Expand Up @@ -124,7 +124,7 @@ run-android-benchmark-$1-%: android-benchmark-$1

# Push compiled java sources, test data and executable to device
adb push $(MLN_ANDROID_BENCHMARK_DIR)/classes.dex $(MLN_ANDROID_LOCAL_BENCHMARK_DIR) > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/intermediate-jars/$(buildtype)/jni/$2/libmapbox-gl.so $(MLN_ANDROID_LOCAL_BENCHMARK_DIR) > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/intermediate-jars/$(buildtype)/jni/$2/libmaplibre.so $(MLN_ANDROID_LOCAL_BENCHMARK_DIR) > /dev/null 2>&1
adb push benchmark/fixtures $(MLN_ANDROID_LOCAL_BENCHMARK_DIR)/benchmark > /dev/null 2>&1
adb push test/fixtures $(MLN_ANDROID_LOCAL_BENCHMARK_DIR)/test > /dev/null 2>&1
adb push MapboxGLAndroidSDK/build/intermediates/cmake/$(buildtype)/obj/$2/mbgl-benchmark $(MLN_ANDROID_LOCAL_BENCHMARK_DIR) > /dev/null 2>&1
Expand Down
2 changes: 1 addition & 1 deletion platform/android/MapboxGLAndroidSDK/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ android {
}

// build native libraries
List nativeTargets = ["mapbox-gl"]
List nativeTargets = ["maplibre"]
if (project.hasProperty("mapbox.with_test")) {
nativeTargets.add("mbgl-test")
}
Expand Down
10 changes: 5 additions & 5 deletions platform/android/MapboxGLAndroidSDK/src/cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)

target_compile_options(mbgl-compiler-options INTERFACE -frtti)

add_library(mapbox-gl SHARED
add_library(maplibre SHARED
${PROJECT_SOURCE_DIR}/main.cpp
${PROJECT_SOURCE_DIR}/jni_native.cpp
${PROJECT_SOURCE_DIR}/jni_native.hpp
Expand Down Expand Up @@ -193,20 +193,20 @@ add_library(mapbox-gl SHARED
${PROJECT_SOURCE_DIR}/util/tile_server_options.hpp)

target_include_directories(
mapbox-gl
maplibre
PRIVATE ${PROJECT_SOURCE_DIR}
../../../../../src
../../../src
../../../../default/include)

target_compile_options(mapbox-gl
target_compile_options(maplibre
PRIVATE $<$<CONFIG:Release>:-Oz>
$<$<CONFIG:Release>:-Qunused-arguments>
$<$<CONFIG:Release>:-flto>
$<$<CONFIG:Release>:-fvisibility=hidden>
$<$<CONFIG:Release>:-fvisibility-inlines-hidden>)

target_link_libraries(mapbox-gl
target_link_libraries(maplibre
PRIVATE $<$<CONFIG:Release>:-Oz>
$<$<CONFIG:Release>:-Wl,--icf=all>
$<$<CONFIG:Release>:-Wl,--gc-sections>
Expand All @@ -216,4 +216,4 @@ target_link_libraries(mapbox-gl
mbgl-core
mbgl-vendor-unique_resource)

install(TARGETS mapbox-gl LIBRARY DESTINATION lib)
install(TARGETS maplibre LIBRARY DESTINATION lib)
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public static void setLibraryLoader(LibraryLoader libraryLoader) {
}

/**
* Loads "libmapbox-gl.so" native shared library.
* Loads "libmaplibre.so" native shared library.
* <p>
* Catches UnsatisfiedLinkErrors and prints a warning to logcat.
* </p>
Expand All @@ -40,7 +40,7 @@ public static synchronized void load() {
try {
if (!loaded) {
loaded = true;
loader.load("mapbox-gl");
loader.load("maplibre");
}
} catch (UnsatisfiedLinkError error) {
loaded = false;
Expand Down
8 changes: 4 additions & 4 deletions platform/android/scripts/metrics.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ set -e
set -o pipefail

# Track individual architectures
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/armeabi-v7a/libmapbox-gl.so" "Android arm-v7"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/arm64-v8a/libmapbox-gl.so" "Android arm-v8"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/x86/libmapbox-gl.so" "Android x86"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/x86_64/libmapbox-gl.so" "Android x86_64"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/armeabi-v7a/libmaplibre.so" "Android arm-v7"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/arm64-v8a/libmaplibre.so" "Android arm-v8"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/x86/libmaplibre.so" "Android x86"
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/intermediates/intermediate-jars/release/jni/x86_64/libmaplibre.so" "Android x86_64"

# Track overall library size
scripts/check_binary_size.js "MapboxGLAndroidSDK/build/outputs/aar/MapboxGLAndroidSDK-release.aar" "Android AAR"
Expand Down

0 comments on commit 7599611

Please sign in to comment.