Merge pull request #210 from alexanderwiederin/android-CI

Add `Generate Kotlin Android` to CI

Author: tnull

.github/workflows/kotlin.yml

@@ -20,6 +20,12 @@ jobs:
           distribution: temurin
           java-version: 11
 
+      - name: Set default Rust version to 1.73.0
+        run: rustup default 1.73.0
+
+      - name: Show default version of NDK
+        run: echo $ANDROID_NDK_ROOT
+
       - name: Run ktlintCheck on ldk-node-jvm
         run: |
           cd $LDK_NODE_JVM_DIR
@@ -33,6 +39,9 @@ jobs:
       - name: Generate Kotlin JVM
         run: ./scripts/uniffi_bindgen_generate_kotlin.sh
 
+      - name: Generate Kotlin Android
+        run: ./scripts/uniffi_bindgen_generate_kotlin_android.sh
+
       - name: Start bitcoind and electrs
         run: docker compose up -d
 

scripts/uniffi_bindgen_generate_kotlin_android.sh

@@ -1,17 +1,43 @@
 #!/bin/bash
+
 BINDINGS_DIR="bindings/kotlin"
 TARGET_DIR="target"
 PROJECT_DIR="ldk-node-android"
-PACKAGE_DIR="org/lightningdevkit/ldknode"
 UNIFFI_BINDGEN_BIN="cargo run --manifest-path bindings/uniffi-bindgen/Cargo.toml"
-ANDROID_NDK_ROOT="/opt/homebrew/share/android-ndk"
-LLVM_ARCH_PATH="darwin-x86_64"
+
+export_variable_if_not_present() {
+  local name="$1"
+  local value="$2"
+
+  # Check if the variable is already set
+  if [ -z "${!name}" ]; then
+    export "$name=$value"
+    echo "Exported $name=$value"
+  else
+    echo "$name is already set to ${!name}, not exporting."
+  fi
+}
+
+case "$OSTYPE" in
+    linux-gnu)
+      export_variable_if_not_present "ANDROID_NDK_ROOT" "/opt/android-ndk"
+      export_variable_if_not_present "LLVM_ARCH_PATH" "linux-x86_64"
+      ;;
+    darwin*)
+      export_variable_if_not_present "ANDROID_NDK_ROOT" "/opt/homebrew/share/android-ndk"
+      export_variable_if_not_present "LLVM_ARCH_PATH" "darwin-x86_64"
+      ;;
+    *)
+      echo "Unknown operating system: $OSTYPE"
+      ;;
+    esac
+
 PATH="$ANDROID_NDK_ROOT/toolchains/llvm/prebuilt/$LLVM_ARCH_PATH/bin:$PATH"
 
 rustup target add x86_64-linux-android aarch64-linux-android armv7-linux-androideabi
-CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_X86_64_LINUX_ANDROID_LINKER="x86_64-linux-android25-clang" CC="x86_64-linux-android25-clang" cargo build --profile release-smaller --features uniffi --target x86_64-linux-android || exit 1
-CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_ARMV7_LINUX_ANDROIDEABI_LINKER="armv7a-linux-androideabi25-clang" CC="armv7a-linux-androideabi25-clang" cargo build --profile release-smaller --features uniffi --target armv7-linux-androideabi || exit 1
-CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_AARCH64_LINUX_ANDROID_LINKER="aarch64-linux-android25-clang" CC="aarch64-linux-android25-clang" cargo build --profile release-smaller --features uniffi --target aarch64-linux-android || exit 1
+CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_X86_64_LINUX_ANDROID_LINKER="x86_64-linux-android21-clang" CC="x86_64-linux-android21-clang" cargo build --profile release-smaller --features uniffi --target x86_64-linux-android || exit 1
+CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_ARMV7_LINUX_ANDROIDEABI_LINKER="armv7a-linux-androideabi21-clang" CC="armv7a-linux-androideabi21-clang" cargo build --profile release-smaller --features uniffi --target armv7-linux-androideabi || exit 1
+CFLAGS="-D__ANDROID_MIN_SDK_VERSION__=21" AR=llvm-ar CARGO_TARGET_AARCH64_LINUX_ANDROID_LINKER="aarch64-linux-android21-clang" CC="aarch64-linux-android21-clang" cargo build --profile release-smaller --features uniffi --target aarch64-linux-android || exit 1
 $UNIFFI_BINDGEN_BIN generate bindings/ldk_node.udl --language kotlin -o "$BINDINGS_DIR"/"$PROJECT_DIR"/lib/src/main/kotlin || exit 1
 
 JNI_LIB_DIR="$BINDINGS_DIR"/"$PROJECT_DIR"/lib/src/main/jniLibs/