Skip to content
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

ndk version up problem #10323

Closed
ksh8281 opened this issue Nov 7, 2013 · 3 comments
Closed

ndk version up problem #10323

ksh8281 opened this issue Nov 7, 2013 · 3 comments
Labels
O-android Operating system: Android

Comments

@ksh8281
Copy link
Contributor

ksh8281 commented Nov 7, 2013

i found diffrent thing in ndk r9b.
until ndk ~r9, function tgammaf is not defined at math.h.
so rust has fake tgammaf function in src/rt/rust_android_dummy.cpp
now, ndk r9b has tgammaf that is defined at sysroot/usr/include/math.h.(android-18)
so, it causes complie problem

/home/ksh8281/rust_android/rust/src/rt/rust_android_dummy.cpp: In function 'void tgammaf()':
/home/ksh8281/rust_android/rust/src/rt/rust_android_dummy.cpp:63:25: error: declaration of C function 'void tgammaf()' conflicts with
/home/ksh8281/rust_android/ndk/bin/../sysroot/usr/include/math.h:349:7: error: previous declaration 'float tgammaf(float)' here
make: *** [arm-linux-androideabi/rt/stage2/rust_android_dummy.o] Error 1

i tried to find prefer solution, but i cannot found best solution yet.(i think deleteing our tgammaf causes compatibility problem in order version of ndk)

@thestinger
Copy link
Contributor

Is compatibility with old versions of the NDK useful?

@yichoi
Copy link
Contributor

yichoi commented Nov 7, 2013

I think it is not useful.
If we want to maintain compatibility with old NDK, many variation of compile option should be guaranteed.
Let's make official NDK version which Rust basically require for android as r9b. then we can remove dummy function of tgammaf().

@thestinger
Copy link
Contributor

+1 for removing it, we'll need the android buildbot updated though

bors added a commit that referenced this issue Nov 7, 2013
update for ndk r9b (#10323)

````
Android NDK, Revision 9b (October 2013)
Important changes:
Updated include/android/*h and math.h for all Android API levels up to 18, 
including the addition of levels 13, 15, 16 and 17. 
For information on added APIs, 
see commit messages for Changes 68012 and 68014. (Issues 47150, 58528, and 38423)
````

https://android-review.googlesource.com/#/c/68014/
@ksh8281 ksh8281 closed this as completed Nov 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-android Operating system: Android
Projects
None yet
Development

No branches or pull requests

3 participants