Opened 5 years ago

Closed 5 years ago

#1203 closed enhancement (fixed)

Android is not only ARM

Reported by: nextloop Owned by: str4d
Priority: minor Milestone: 0.9.14.1
Component: apps/android Version: 0.9.11
Keywords: Cc:
Parent Tickets:

Description

net.i2p.android.router/routerjars/jni/build.sh line 50:
Android api level is set to 8.
Plattform 8 only supports arm architecture. I am not sure what happens on other CPU architectures. Will java fallback be used? Is it fast enough?

The app is usually compiled using the precompiled net.i2p.android.router/routerjars/jni/libjbigi.so (which is probably an arm binary). During an F-Droid build i have to remove all binary files, so it is each time newly compiled.

What's against changing the api level to the newest(19) and modifying the script so it produces also jni binaries for x86, MIPS and so on (maybe configureable)?
I believe the NDK api level is independent from the Android one.

I might be completely wrong with my findings, as i only have some basic android knowledge.

Subtickets

Change History (7)

comment:2 Changed 5 years ago by zzz

  • Component changed from other to apps/android
  • Owner set to zzz

comment:3 Changed 5 years ago by zzz

The basic way to fix this is to pass more environment variables and arguments from routerjars/custom_rules.xml to routerjars/jni/build.sh. You'll have to help us understand the F-Droid build environment and how the targets are set.

comment:4 Changed 5 years ago by nextloop

I am not sure whether this is the standard android behaviour, but I think the NDK-dir is usually set in local.properties, maybe custum_rules.xml can pass them to the script?

Going to add more here as time permits.

comment:5 Changed 5 years ago by zzz

  • Milestone changed from 0.9.12 to 0.9.14

related/dup: #718

comment:6 Changed 5 years ago by zzz

  • Owner changed from zzz to str4d
  • Status changed from new to assigned

Reassigning to str4d as he is working on this

comment:7 Changed 5 years ago by str4d

  • Milestone changed from 0.9.14 to 0.9.14.1
  • Resolution set to fixed
  • Status changed from assigned to closed

libjbigi.so files for x86 and MIPS can be built as of a03b4a542c4c26d11441da6d8dbba735aab2127a. They now need to be tested.

Closing this ticket because build.sh has been modified. #718 left open for testing.

Note: See TracTickets for help on using tickets.