-
Notifications
You must be signed in to change notification settings - Fork 343
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
third_party/Android.mk enhancements: Use LOCAL_CPPFLAGS #281
Comments
Hello, I'd like to help with this if possible. I'm pretty new to Android, is this just a case of going through the code and switching any use of LOCAL_CXXFLAGS to LOCAL_CPPFLAGS or is there a bit more to it than that? Thanks |
Yes, it's just making that change in the one Android.mk file. :-) Real work is in verifying the change. I'd like to test it with the NDK versions r10e and r13. Here's an example of how I test the build, customized to NDK location and especially how the tree is checked out. (Actually it's a bit of a wonky structure but it works).
If you can make the Android.mk change and verify with the two versions of the NDK that would be grand! |
Thanks for the information, I've updated the one Android.mk file, but I think the testing is a little out of my depth, so haven't submitted a PR just yet. If there's anything a little more beginner level I can work on, I'd love to help out! |
@taml I hear you. :-) If you're willing to help on another project, I have something in mind in KhronosGroup/SPIRV-Tools. I haven't written it down yet, but it's a nice isolated piece that can be verified on any platform. I'm taking this opportunity to formalize the "help wanted" process and task list in that project. Stay tuned. |
@dneto0 Yeah sure, I'd definitely be interested in helping out! |
Take a look at KhronosGroup/SPIRV-Tools#502 |
Use LOCAL_CPPFLAGS instead of LOCAL_CXXFLAGS.
LOCAL_CPPFLAGS is newer and recommended.
The text was updated successfully, but these errors were encountered: