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

Also download Android NDK in update_deps.py #1183

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yukawa
Copy link
Collaborator

@yukawa yukawa commented Feb 14, 2025

Description

This is a preparation before removing the dependency on Docker from our Android build instructions (#1181).

With this commit running the update_deps.py also downloads Android NDK for macOS and Linux then extract it under third_party/ndk/.

This would enable us to fully control which version of Android NDK should be used to build Android Library (libmozc.so) in a subsequent commit.

At this moment only the observable difference is that the Android NDK will be extracted under third_party/ndk/. There must be no difference in the final artifact yet.

Issue IDs

Steps to test new behaviors (if any)

  • OS: All
  • Steps:
    1. Confirm CI is still passing in GitHub Actions

This is a preparation before removing the dependency on Docker from our
Android build instructions (google#1181).

With this commit running the 'update_deps.py' also downloads Android NDK
for macOS and Linux then extract it under third_party/ndk/.

This would enable us to fully control which version of Android NDK
should be used to build Android Library ('libmozc.so') in a subsequent
commit.

At this moment only the observable difference is that the Android NDK
will be extracted under third_party/ndk/. There must be no difference
in the final artifact yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant