New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

android.rake toolchain outdated #3208

Closed
felixjones opened this Issue Sep 14, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@felixjones
Contributor

felixjones commented Sep 14, 2016

The android.rake toolchain needs updating.

Issues with the current version;

  • Does not work for Windows x64 NDK as it searches for "windows" rather than "windowsx-86_64" in the NDK selected toolchain's prebuilts
  • No real clang support (can only target a single architecture, which is unspecified)
  • Default compiler is still GCC, Android NDK now defaults to clang
  • Defaults to Android platform 'android-14', even if it's not installed
  • NDK no longer uses llvm-ar (GCC's is used), causing clang to fail as the executable no longer exists

I've seen discussions that the build system may be replaced, is this still the case? Is it worth updating the current provided toolchains?

Is it worth removing the android.rake if keeping it up to date with the NDK is too much work?

Not a desperate issue for myself as I have already written a new android.rake for my own Android projects, but having the build system not work for the current NDK platform is quite a discouragement.

@matz

This comment has been minimized.

Show comment
Hide comment
@matz

matz Sep 14, 2016

Member

I still want to replace the build system, but currently we have no concrete plan.
So please just update the Android rules.

Member

matz commented Sep 14, 2016

I still want to replace the build system, but currently we have no concrete plan.
So please just update the Android rules.

@felixjones

This comment has been minimized.

Show comment
Hide comment
@felixjones

felixjones Oct 7, 2016

Contributor

With mips support being added #3222 this issue is "resolved" as the Android rules are now updated, however Android GCC support would still be a 'nice thing to have' until GCC is removed from the Android platform.

https://android.googlesource.com/platform/ndk.git/+/master/CHANGELOG.md
GCC for Android is currently deprecated, with its removal being likely once libc++ matures;

The lost Android GCC support should be its own issue at this point (the goal of updating Android's rules so they are no-longer outdated is met).

Contributor

felixjones commented Oct 7, 2016

With mips support being added #3222 this issue is "resolved" as the Android rules are now updated, however Android GCC support would still be a 'nice thing to have' until GCC is removed from the Android platform.

https://android.googlesource.com/platform/ndk.git/+/master/CHANGELOG.md
GCC for Android is currently deprecated, with its removal being likely once libc++ matures;

The lost Android GCC support should be its own issue at this point (the goal of updating Android's rules so they are no-longer outdated is met).

@felixjones felixjones closed this Oct 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment