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

ANDROID_NDK_ROOT environment variable should be preset #2426

Closed
7 tasks done
dmitry-shibanov opened this issue Jan 12, 2021 · 3 comments
Closed
7 tasks done

ANDROID_NDK_ROOT environment variable should be preset #2426

dmitry-shibanov opened this issue Jan 12, 2021 · 3 comments
Assignees
Labels
awaiting-deployment Code complete; awaiting deployment and/or deployment in progress feature request OS: macOS OS: Ubuntu OS: Windows

Comments

@dmitry-shibanov
Copy link
Contributor

dmitry-shibanov commented Jan 12, 2021

Description
Add ANDROID_NDK_HOME and ANDROID_NDK_ROOT to the all images. According to discussion: ANDROID_NDK_HOME and ANDROID_NDK_ROOT are widely used by many other softwares. In most cases, If ANDROID_NDK_HOME is not defined, the value in ANDROID_NDK_ROOT is used. but some software only use ANDROID_NDK_HOME, others only use ANDROID_NDK_ROOT. so, both are preset is the best.

References:

Question, Bug, or Feature?:
Feature request

Virtual environments affected

  • Ubuntu 16.04
  • Ubuntu 18.04
  • Ubuntu 20.04
  • macOS 10.15
  • macOS 11.0
  • Windows Server 2016 R2
  • Windows Server 2019
@ystefinko
Copy link

vote for this feature!

@nikita-bykov
Copy link
Contributor

The PR has been merged. The new environment variables will be available after the next vm rollout.

@miketimofeev miketimofeev added OS: Ubuntu awaiting-deployment Code complete; awaiting deployment and/or deployment in progress and removed OS: Ubuntu labels Jan 29, 2021
@miketimofeev
Copy link
Contributor

ANDROID_NDK_ROOT variable is now available across all the images

nappairam pushed a commit to nappairam/liboqs-rust that referenced this issue Aug 3, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.
nappairam pushed a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.
nappairam pushed a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.
nappairam added a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.

Signed-off-by: Mariappan Ramasamy <[email protected]>
nappairam added a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.

Signed-off-by: Mariappan Ramasamy <[email protected]>
nappairam added a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.

Signed-off-by: Mariappan Ramasamy <[email protected]>
nappairam added a commit to nappairam/liboqs-rust that referenced this issue Aug 6, 2024
Based on the cmake manual, cross compiling for Android,
https://cmake.org/cmake/help/latest/manual/cmake-toolchains.7.html#cross-compiling-for-android-with-the-ndk

CMAKE_SYSTEM_NAME **must** be set to Android.

Additionally, set CMAKE_ANDROID_NDK if corresponding env var is available.
Android ecosystem uses both ANDROID_NDK_ROOT and ANDROID_NDK_HOME to refer to
NDK's path. For example, github runners has both variables set:
actions/runner-images#2426

These two cmake variables are sufficient to build for Android with cargo-ndk tool.

Signed-off-by: Mariappan Ramasamy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-deployment Code complete; awaiting deployment and/or deployment in progress feature request OS: macOS OS: Ubuntu OS: Windows
Projects
None yet
Development

No branches or pull requests

5 participants