[BUG] Chiptool for Android is not 16KB aligned

[OlivierGre]

Reproduction steps

When I build chiptool for Android and launch it from Android Studio, I have the following warning:

Is there an on-going action to fix this issue?

@andy31415 If I remember one of our discussion about Android NDK version, this problem is related to this NDK version. FYI, I'm currently on ndkVersion "28.2.13676358" (which is what is indicated in chiptool's app/build.gradle file).

Bug prevalence

Whenever I do this

GitHub hash of the SDK that was being used

51de415

Platform

android

Platform Version(s)

No response

Anything else?

No response

[andy31415]

This should have been fixed by #40455 and was cherrypicked in various builds.

[andy31415]

From what I can tell, the java build was updated to have ldflags += [ "-Wl,-z,max-page-size=16384" ]

I believe this was assumed fixed.

[andy31415]

Given that the error says that even libc++ is not ok, I believe you may actually compile with a different NDK than what you think you are compiling with.

[OlivierGre]

Let me check

[OlivierGre]

I have built the latest version. Here is the log (see build_android.log). The log shows that the NDK version is:
ANDROID_NDK_HOME: /local/android/Sdk/ndk/28.2.13676358/

I'm then opening the Chiptool app in Android Studio. Android Studio will rebuild the application and launch it. I do have the warning about 16KB aligment issue. I'm providing the log of the build in Android Studio.

Do you know someone who could try the same?
Thank you

build_android.log
build_android_studio.log

[OlivierGre]

The warning displayed on the smartphone (as shown at the beginning of this issue) indicates alignment problems for 4 libraries. Android studio displays a warning only for 2 libraries:

[OlivierGre]

I have fixed the case of libbarhopper_v3.so and ibimage_processing_util_jni.so. It can be fixed by replacing:

implementation 'com.google.mlkit:barcode-scanning:17.0.2'
def camerax_version = "1.1.0"

by a more recent version:

implementation 'com.google.mlkit:barcode-scanning:17.3.0'
def camerax_version = "1.4.1"

Now that I have fixed that, I don't have anymore warning about aligment problems! So the other ones were probably spurious warnings.

[OlivierGre]

For your information, a fix (not related) is also needed in BluetoothManager.kt:

                 val producerScope: ProducerScope<BluetoothDevice> = this@callbackFlow
                 if (producerScope.channel.isClosedForSend) {
                   Log.w(TAG, "Bluetooth device was scanned, but channel is already closed")
                 } else {
-                  offer(device)
+                    trySend(device).isSuccess
                 }
               }