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

[BUG] ANDROID_ARM_MODE behavior change between legacy and new toolchain file #1656

Closed
DanAlbert opened this issue Jan 18, 2022 · 1 comment
Closed
Assignees
Labels
Projects

Comments

@DanAlbert
Copy link
Member

Description

Mirroring https://gitlab.kitware.com/cmake/cmake/-/issues/23108 for tracking purposes.

The legacy toolchain would ignore the -DANDROID_ARM_MODE argument if it was passed for any ABI other than armeabi-v7a. The new toolchain converts that to CMAKE_ANDROID_ARM_MODE, and CMake 3.22 emits an error if that is passed for ABIs other than armeabi-v7a. We should preserve the old behavior.

@hhb has already uploaded a fix for us (thanks!): https://android-review.googlesource.com/c/platform/ndk/+/1951676

Affected versions

r23, r24

Canary version

No response

Host OS

Linux, Mac, Windows

Host OS version

All

Affected ABIs

arm64-v8a, x86, x86_64

Build system

CMake

Other build system

No response

minSdkVersion

Any

Device API level

No response

@DanAlbert DanAlbert added the bug label Jan 18, 2022
@DanAlbert DanAlbert added this to Triaged in r23c via automation Jan 18, 2022
@DanAlbert DanAlbert added this to Triaged in r24 via automation Jan 18, 2022
@DanAlbert DanAlbert removed this from Triaged in r24 Feb 22, 2022
@DanAlbert DanAlbert added this to Triaged in r24b via automation Feb 22, 2022
@DanAlbert DanAlbert removed this from Triaged in r24b Feb 22, 2022
@DanAlbert DanAlbert added this to Triaged in r24 via automation Feb 22, 2022
@DanAlbert DanAlbert moved this from Triaged to Merged in r24 Feb 28, 2022
r23c automation moved this from Triaged to Merged Feb 28, 2022
@DanAlbert DanAlbert moved this from Merged to Needs cherry-pick in r23c Apr 22, 2022
@DanAlbert DanAlbert moved this from Needs cherry-pick to Merged in r23c Apr 22, 2022
@DanAlbert
Copy link
Member Author

Should be fixed in r23 build 8486889.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
r23c
  
Merged
r24
  
Merged
Development

No branches or pull requests

2 participants