-
Notifications
You must be signed in to change notification settings - Fork 466
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
turbojpeg-sys fails to build with cc 1.0.84 with Android NDK 21 #903
Comments
cmake uses @tyilo Can you try compiling with cc@=1.0.83 and get cmake to print the compiler it is used? I suspect it might be using |
@NobodyXu There is no gcc compiler included in the Android NDK, only clang, so that would be impossible I think (especially when the build completes successfully). Anyway here is the values of
Here is the same when using
Seems like the difference is that |
hmmm @tyilo Can you do a bisect on v1.0.83..v1.0.84 to find out which commits caused this error? |
@NobodyXu Done:
|
Thanks! I will investigate this, would be great if you can also get the build commands run by |
Seems to be the following with at commit 53564e0:
and using the previous commit 755e4fe:
Again, the only difference seems to be the |
Seems that 53564e0 actually adds |
IIRC, recent clang wants an API version in the target, as in |
That's unfortunate. Is there an environment variable or something we can use to get that value (similar to |
@tyilo Can you try cc 1.0.87 to see if it fixes your issue? |
I still get the same error:
|
Same thing happens with |
|
I have a project depending on
turbojpeg-sys
which I need to compile for thearmv7-linux-androideabi
target for Android API level 17, meaning that I need to use Android NDK v21.Compiling a project with the latest version of
turbojpeg-sys
with version 1.0.83 of cc works:However, if I upgrade
cc
to version 1.0.84 the build fails:Error:
The text was updated successfully, but these errors were encountered: