Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 22 additions & 1 deletion .github/workflows/publish-android.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
ref: ${{ github.event_name == 'workflow_dispatch' && inputs.version || github.event.release.tag_name }}
- name: Set up JDK 17
uses: actions/setup-java@v4
with:
Expand All @@ -29,6 +31,22 @@ jobs:
- name: Setup Gradle
uses: gradle/actions/setup-gradle@v4

- name: Install native build dependencies
run: sudo apt-get update && sudo apt-get install -y pkg-config protobuf-compiler

- name: Set up Android SDK
uses: android-actions/setup-android@v3

- name: Set up Android NDK
id: setup-ndk
uses: nttld/setup-ndk@v1
with:
ndk-version: r28c
add-to-path: true
Comment thread
ovitrif marked this conversation as resolved.

- name: Export Android NDK root
run: echo "ANDROID_NDK_ROOT=${{ steps.setup-ndk.outputs.ndk-path }}" >> "$GITHUB_ENV"

- name: Extract version from input or tag
id: version
shell: bash
Expand All @@ -39,6 +57,9 @@ jobs:
fi
echo "version=${VERSION#v}" >> $GITHUB_OUTPUT

- name: Generate Android bindings
run: scripts/uniffi_bindgen_generate_kotlin_android.sh

- name: Build with Gradle
working-directory: bindings/kotlin/ldk-node-android
run: ./gradlew build -Pversion=${{ steps.version.outputs.version }}
Expand All @@ -47,6 +68,6 @@ jobs:
working-directory: bindings/kotlin/ldk-node-android
env:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.ORG_PACKAGES_TOKEN }}
GITHUB_TOKEN: ${{ github.token }}
GITHUB_REPO: ${{ github.repository }}
run: ./gradlew publish -Pversion=${{ steps.version.outputs.version }}
4 changes: 3 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
# 0.7.0-rc.46 (Synonym Fork)
# 0.7.0-rc.48 (Synonym Fork)

## Bug Fixes

- Kept Android native debug metadata in published `ldk-node-android` artifacts and
failed Android binding generation when generated JNI libraries lack usable symbols.
- Persist missing announced channel peers from the network graph during
build-time restore and after Rapid Gossip Sync graph updates. Automatic
recovery respects explicit disconnects and last-channel closes during the
Expand Down
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ exclude = ["bindings/uniffi-bindgen"]

[package]
name = "ldk-node"
version = "0.7.0-rc.46"
version = "0.7.0-rc.48"
authors = ["Elias Rohrer <dev@tnull.de>"]
homepage = "https://lightningdevkit.org/"
license = "MIT OR Apache-2.0"
Expand Down
4 changes: 2 additions & 2 deletions Package.swift
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@

import PackageDescription

let tag = "v0.7.0-rc.46"
let checksum = "06558c2834c186d3da188389ef197942e948ac4f6f3261746e3594420ab0ee23"
let tag = "v0.7.0-rc.48"
let checksum = "afadaec20a4d580b877282da609d7882233a8a1137f8cb621c894892aa3dbaae"
let url = "https://github.com/synonymdev/ldk-node/releases/download/\(tag)/LDKNodeFFI.xcframework.zip"

let package = Package(
Expand Down
2 changes: 1 addition & 1 deletion bindings/kotlin/ldk-node-android/gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ android.useAndroidX=true
android.enableJetifier=true
kotlin.code.style=official
group=com.synonym
version=0.7.0-rc.46
version=0.7.0-rc.48
107 changes: 107 additions & 0 deletions bindings/kotlin/ldk-node-android/lib/build.gradle.kts
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
import java.io.ByteArrayOutputStream
import java.io.File

plugins {
id("com.android.library")
kotlin("android")
Expand Down Expand Up @@ -39,6 +42,12 @@ android {
jvmTarget = "1.8"
}

packaging {
jniLibs {
keepDebugSymbols += listOf("**/libldk_node.so")
}
}

publishing {
singleVariant("release") {
withSourcesJar()
Expand All @@ -58,6 +67,104 @@ dependencies {
api("org.slf4j:slf4j-api:1.7.30")
}

val androidNativeAbis = listOf("armeabi-v7a", "arm64-v8a", "x86_64")

fun executableFromPath(name: String): String? {
return System.getenv("PATH")
?.split(File.pathSeparator)
?.asSequence()
?.map { File(it, name) }
?.firstOrNull { it.canExecute() }
?.absolutePath
}

fun findReadelf(): String {
executableFromPath("llvm-readelf")?.let { return it }
executableFromPath("readelf")?.let { return it }

return listOf("ANDROID_NDK_ROOT", "ANDROID_NDK_HOME", "NDK_HOME")
.mapNotNull { System.getenv(it) }
.map { File(it, "toolchains/llvm/prebuilt") }
.firstNotNullOfOrNull { prebuiltDir ->
if (!prebuiltDir.isDirectory) return@firstNotNullOfOrNull null

prebuiltDir
.walkTopDown()
.firstOrNull { it.name == "llvm-readelf" && it.canExecute() }
?.absolutePath
}
?: throw GradleException(
"llvm-readelf or readelf is required to validate Android native debug symbols"
)
}

fun Project.runReadelf(readelf: String, vararg args: String): Pair<Int, String> {
val stdout = ByteArrayOutputStream()
val stderr = ByteArrayOutputStream()
val result = exec {
commandLine(readelf, *args)
standardOutput = stdout
errorOutput = stderr
isIgnoreExitValue = true
}

return result.exitValue to stdout.toString().ifBlank { stderr.toString() }
}

fun String.parseElfAlignment(): Long {
return if (startsWith("0x")) {
removePrefix("0x").toLong(16)
} else {
toLong()
}
}

val validateReleaseNativeLibraries by tasks.registering {
group = "verification"
description = "Validates release JNI libraries keep full DWARF metadata and 16 KB LOAD alignment."

doLast {
val readelf = findReadelf()
val loadAlignmentRegex = Regex("""^\s*LOAD\s+.*\s+(0x[0-9a-fA-F]+|\d+)\s*$""")

androidNativeAbis.forEach { abi ->
val lib = layout.projectDirectory.file("src/main/jniLibs/$abi/libldk_node.so").asFile
if (!lib.isFile) {
throw GradleException("Android native library missing at '${lib.path}'")
}

val (sectionsExit, sections) = runReadelf(readelf, "-S", lib.absolutePath)
if (sectionsExit != 0 || !Regex("""\.debug_""").containsMatchIn(sections)) {
throw GradleException("Android native library has no full DWARF debug metadata: '${lib.path}'")
}

val wideHeaders = runReadelf(readelf, "-W", "-l", lib.absolutePath)
val headers = if (wideHeaders.first == 0) {
wideHeaders.second
} else {
val fallbackHeaders = runReadelf(readelf, "-l", lib.absolutePath)
if (fallbackHeaders.first != 0) {
throw GradleException("Unable to inspect Android native library headers: '${lib.path}'")
}
fallbackHeaders.second
}

val alignments = headers
.lineSequence()
.mapNotNull { loadAlignmentRegex.matchEntire(it)?.groupValues?.get(1)?.parseElfAlignment() }
.toList()

if (alignments.isEmpty() || alignments.any { it < 16_384 }) {
throw GradleException("Android native library is not 16 KB page-size aligned: '${lib.path}'")
}
}
}
}

tasks.matching { it.name == "bundleReleaseAar" || it.name.startsWith("publish") }.configureEach {
dependsOn(validateReleaseNativeLibraries)
}

afterEvaluate {
publishing {
publications {
Expand Down
Binary file not shown.
Binary file not shown.
Binary file not shown.
Loading