Skip to content

Android openssl atomic bugfix with ndk update#4751

Merged
AenBleidd merged 1 commit intoBOINC:masterfrom
talregev:TalR/test_android_openssl_fix_ndk
May 20, 2022
Merged

Android openssl atomic bugfix with ndk update#4751
AenBleidd merged 1 commit intoBOINC:masterfrom
talregev:TalR/test_android_openssl_fix_ndk

Conversation

@talregev
Copy link
Copy Markdown
Contributor

@talregev talregev commented May 20, 2022

Android openssl atomic bugfix with ndk update.
Fixes microsoft/vcpkg#24021
Fixes #4696

@talregev talregev force-pushed the TalR/test_android_openssl_fix_ndk branch 2 times, most recently from 0d38e24 to 68524dc Compare May 20, 2022 18:39
@talregev
Copy link
Copy Markdown
Contributor Author

talregev commented May 20, 2022

@AenBleidd This PR is fix the openssl atomic problem with update of clang (update ndk). I think this fix is better and we should cancel your PR to vcpkg.

@talregev talregev force-pushed the TalR/test_android_openssl_fix_ndk branch from 68524dc to a8883c8 Compare May 20, 2022 18:51
@talregev
Copy link
Copy Markdown
Contributor Author

talregev commented May 20, 2022

@AenBleidd This fix interduce a new problem. That vcpkg compile arm with neon, even if the neon set to False.
I want to investigate this problem. It ok if you put your vcpkg PR on draft?

I choose another ndk without this problem.

@talregev talregev changed the title [Test] android openssl fix with ndk update Android openssl fix with ndk update May 20, 2022
@talregev talregev changed the title Android openssl fix with ndk update Android openssl atomic bugfix with ndk update May 20, 2022
@talregev talregev marked this pull request as ready for review May 20, 2022 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project
Archived in project

Development

Successfully merging this pull request may close these issues.

[Android][vcpkg] Remove vcpkg pinning [openssl] build failure on Android x86

2 participants