-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Explicitly decalre functions used in TorService.c to build with NDK 26+ #6
Conversation
@eighthave thoughts? |
If you stick to using NDK 25.2.9519653 which the CI uses and the scripts in tor-android recommends this won't be a problem (you just get compiler warnings), when working on code changes in tor-android I used a newer NDK by accident (27.0.11718014) and realized that tor no longer compiles due to NDK upgrading to clang 16: https://www.redhat.com/en/blog/new-warnings-and-errors-clang-16 Of course, this code still compiles with the older clang/ndk that guardian project uses to release tor-android. |
Specifically clang throws out guardianproject/tor-android#57 but for that matter, both were introduced to stop gap fix a crash tor had on Android with comments in guardian project's fork of tor indicating that this code was to be removed once an upstream fix was introduced in tor itself. I've only kind of cursorily researched the issue tracker on the |
The commit fixes the build issue with Tor and NDK 26 Although, I still can't figure out why the jni in Tor is not included in libtor.so after updating to NDK 26. |
…ain 26 uses a newer version of clang that treats these implicitly invoked function calls as errors instead of warngings
I've successfully finished a build of |
Android NDK toolchain 26 and 27 uses a newer version of clang that treats these implicitly invoked function calls as errors instead of warnings. since 25.2.9519653 is deprecated this is one of the things we need todo in order to use a supported NDK toolchain