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

depends: android_ndk: update to 18b #9182

Merged
merged 1 commit into from
Feb 24, 2024

Conversation

tobtoht
Copy link
Collaborator

@tobtoht tobtoht commented Feb 19, 2024

Splitting this out of #9162, since it also resolves an Android depends build issue here: seraphis-migration#16.

See related CI run: https://github.com/tobtoht/monero/commits/pr-16/

@tobtoht
Copy link
Collaborator Author

tobtoht commented Feb 19, 2024

@0xFFFC0000

I would like to request a new "build system" tag for (future) PRs that touch contrib/{depends,gitian,guix} or CMakeLists.txt

@0xFFFC0000
Copy link
Collaborator

@0xFFFC0000

I would like to request a new "build system" tag for (future) PRs that touch contrib/{depends,gitian,guix} or CMakeLists.txt

I agree. We do have "cmake" tag, but for this kind of PR that does not cmake, and related to build system, I am always worried to assign a tag. I will ask core about this. Thanks.

Copy link
Contributor

@jeffro256 jeffro256 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that building https://github.com/UkoeHB/monero/tree/seraphis_lib for Android ARMv8 doesn't work before this commit, but does work after this commit. Excellent work!

@tobtoht tobtoht mentioned this pull request Feb 20, 2024
@luigi1111 luigi1111 merged commit 7c22754 into monero-project:master Feb 24, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants