Separate symbol file generation doesn't work for newer Android NDK

[ifarbod]

Xmake Version

2.8.6

Operating System Version and Architecture

Windows 11 23H2

Describe Bug

Based on the following page:

https://xmake.io/#/manual/project_target?id=targetset_symbols

When building an Android project, it should generate a .sym file. However, XMake relies on architecture-based strip.exe, for example: x86_64-linux-android-strip.exe, in later NDK versions, this is removed and replaced with the generic llvm-strip.exe

checking for the symbols stripper (strip: arm-linux-androideabi-strip) ... no

Expected Behavior

I would like XMake to support the changes in the latest NDK version.

Project Configuration

target("myapplicationcpp2")
    set_kind("shared")
    set_symbols("debug")
    add_files("src/**.cpp")

Additional Information and Error Logs

checking for the shared library linker (sh) ... clang++
checking for the linker (ld) ... clang++
[ 25%]: cache compiling.release src\native_lib.cpp
C:\Users\iFarbod\AppData\Local\Android\Sdk\ndk\26.1.10909125\toolchains\llvm\prebuilt\windows-x86_64\bin\clang++ -c --sysroot=C:\Users\iFarbod\AppData\Local\Android\Sdk\ndk\26.1.10909125\toolchains\llvm\prebuilt\windows-x86_64\sysroot -isystem C:\Users\iFarbod\AppData\Local\Android\Sdk\ndk\26.1.10909125\toolchains\llvm\prebuilt\windows-x86_64\sysroot\usr\include\arm-linux-androideabi -Qunused-arguments -target armv7-none-linux-androideabi21 -mthumb -fPIC -g -O3 -std=c++14 -Isrc -DNDEBUG -o build\.objs\myapplicationcpp2\android\armeabi-v7a\release\src\native_lib.cpp.o src\native_lib.cpp
checking for the symbols stripper (strip: arm-linux-androideabi-strip) ... no
[ 50%]: linking.release libmyapplicationcpp2.so
C:\Users\iFarbod\AppData\Local\Android\Sdk\ndk\26.1.10909125\toolchains\llvm\prebuilt\windows-x86_64\bin\clang++ -o build\android\armeabi-v7a\release\libmyapplicationcpp2.so build\.objs\myapplicationcpp2\android\armeabi-v7a\release\src\native_lib.cpp.o -shared -llog -target armv7-none-linux-androideabi21 -mthumb -fPIC -s

build cache stats:
cache directory: build\.build_cache
cache hit rate: 100%
cache hit: 1
cache miss: 0
new cached files: 0
remote cache hit: 0
remote new cached files: 0
preprocess failed: 0
compile fallback count: 0

[100%]: build ok, spent 0.296s

[ifarbod]

This line should be fixed I think:

xmake/xmake/toolchains/ndk/load.lua

Line 94 in 76af94c

toolchain:set("toolset", "strip", gcc_toolchain_bin and path.join(gcc_toolchain_bin, cross .. "strip") or (cross .. "strip"))

toolchain:set("toolset", "strip", gcc_toolchain_bin and path.join(gcc_toolchain_bin, cross .. "strip") or (cross .. "strip"), "llvm-strip")

[ifarbod]

Yeah, that's correct, I'll submit a PR.

checking for Android SDK directory ... C:\Users\iFarbod\AppData\Local\Android\Sdk
checking for Build Tools Version of Android SDK ... 34.0.0
checking for NDK directory ... C:\Users\iFarbod\AppData\Local\Android\Sdk\ndk\26.1.10909125
checking for SDK version of NDK ... 21
[ 25%]: cache compiling.release src\native_lib.cpp
[ 50%]: linking.release libmyapplicationcpp2.so
[ 50%]: generating.release myapplicationcpp2.sym
[100%]: build ok, spent 0.828s

[waruqi]

thanks