python-for-android icon indicating copy to clipboard operation
python-for-android copied to clipboard

scipy: update to v1.15.2

Open T-Dynamos opened this issue 10 months ago • 5 comments

Closes #3116

Cons:

  1. Can't build on macos
  2. Needs android 7 and 7+
  3. Only 64 bit arches supported

Extra Notes:

Removes need to use LEGACY_NDK as it uses fortran from https://github.com/termux/ndk-toolchain-clang-with-flang.

T-Dynamos avatar Mar 27 '25 15:03 T-Dynamos

SDL does not builds with ndk r27c because of https://github.com/libsdl-org/SDL/issues/9792. So we need to wait for #3125.

CC: @misl6

T-Dynamos avatar Mar 27 '25 17:03 T-Dynamos

Will continue working on the SDL3 recipes this weekend, however we are going to support both SDL2 and SDL3 for a long time, as only Kivy 3.0.0 will support SDL3, and will not be ready quite soon.

Maybe we can use NDK 27 only for some specific recipes? (or a previous one for SDL2)

misl6 avatar Mar 27 '25 20:03 misl6

Just saw fixed was also pushed for SDL2, so bumped version to include that fix. :)

T-Dynamos avatar Mar 27 '25 21:03 T-Dynamos

Should I document this things or add warnings in code:

 Can't build on macos
 Only 64 bit arches supported

T-Dynamos avatar Apr 20 '25 16:04 T-Dynamos

Should I document this things or add warnings in code:

 Can't build on macos
 Only 64 bit arches supported

Yeah maybe failing early with an explicit message would do it, what do you think? Which makes me think, maybe in the future we want a consistent behavior for this case and have something like "unsupported_arch" attribute in a recipe (empty by default) that would make the build fail fast with an explicit message.

AndreMiras avatar Apr 21 '25 08:04 AndreMiras

@cclauss This PR depends on https://github.com/kivy/python-for-android/pull/3180, I will continue this after that is merged.

T-Dynamos avatar Oct 08 '25 09:10 T-Dynamos

This PR is ready for review.

CI fails due to out of storage?

I was able to build it on my computer with only single arch, .buildozer folder was around 7GB.

Anyways it's only made to build on linux computer with 64bit android arches.

T-Dynamos avatar Oct 11 '25 17:10 T-Dynamos

Ping @AndreMiras ! What to do with the failing tests? Just ignore them and merge? I have personally tested this PR.

kuzeyron avatar Oct 14 '25 14:10 kuzeyron

Ping @AndreMiras ! What to do with the failing tests? Just ignore them and merge? I have personally tested this PR.

Thanks for giving it a try and thank you T-Dynamos for the huge effort getting that one to work :1st_place_medal: I've just left a comment so that the CI wouldn't fail, other than that I'm good for a merge

AndreMiras avatar Oct 14 '25 15:10 AndreMiras