-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[CI] Fix build android rpc failure in CI #12216
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks!
@@ -122,7 +122,7 @@ jobs: | |||
- name: Build android_rpc | |||
working-directory: apps/android_rpc | |||
run: | | |||
export PATH="${ANDROID_NDK_HOME}:$PATH" | |||
export PATH="${ANDROID_NDK_LATEST_HOME}:$PATH" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add set -eux
to all the inline scripts so we get errors about unbound variables if something changes in the future?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
This reverts commit e7e29b4.
This reverts commit e7e29b4.
Reverts #12216 Addition to my previous changes. After the previous changes, the android camera build failed because by default `PYTHONPATH` is empty, and after `set -eux` it is fails: https://github.com/apache/tvm/blob/759a648cd5237885a8205b1ee4508dabcc3af2d5/.github/workflows/main.yml#L152-L156 This error was not noticed because the flag `continue-on-error` is true.
android_rpc build problem: apache#12191 The problem with the build appeared due to the fact that the `ANDROID_NDK_HOME` environment variable was removed in the current version of github actions. https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-Readme.md But this variable is used here: https://github.com/apache/tvm/blob/ee319d9d23c80091da9c4fb764b1e6d49d462714/.github/workflows/main.yml#L122-L127 Now only `ANDROID_NDK_LATEST_HOME` is available for ndk.
Reverts apache#12216 Addition to my previous changes. After the previous changes, the android camera build failed because by default `PYTHONPATH` is empty, and after `set -eux` it is fails: https://github.com/apache/tvm/blob/759a648cd5237885a8205b1ee4508dabcc3af2d5/.github/workflows/main.yml#L152-L156 This error was not noticed because the flag `continue-on-error` is true.
android_rpc build problem: #12191
The problem with the build appeared due to the fact that the
ANDROID_NDK_HOME
environment variable was removed in the current version of github actions.https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-Readme.md
But this variable is used here:
tvm/.github/workflows/main.yml
Lines 122 to 127 in ee319d9
Now only
ANDROID_NDK_LATEST_HOME
is available for ndk.cc @Mousius @areusch @driazati