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

Fix Android CI #3900

Merged
merged 2 commits into from
May 24, 2021
Merged

Fix Android CI #3900

merged 2 commits into from
May 24, 2021

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented May 23, 2021

The binary_libtorchvision_ops_android job is failing due to:

2021-05-23T10:53:46.946+0000 [INFO] [com.android.build.gradle.internal.cxx.logging.IssueReporterLoggingEnvironment] /home/circleci/project/android/ops/CMakeLists.txt : C/C++ debug|armeabi-v7a : {"cookie":"","inReplyTo":"configure","message":"CMake Error: The following variables are used in this project, but they are set to NOTFOUND.\nPlease set them or make sure they are set and tested correctly in the CMake files:\nPYTORCH_LIBRARY\n    linked by target \"torchvision_ops\" in directory /home/circleci/project/android/ops\n","title":"Error","type":"message"}

Checking the contents of the library we see that pytorch_jni was renamed to pytorch_jni_lite:

$ ls -lh /home/circleci/project/android/ops/build/pytorch_android-1.9.0-SNAPSHOT.aar/jni/armeabi-v7a
total 28M
-rw-r--r-- 1 circleci circleci 562K May 23 10:53 libc++_shared.so
-rw-r--r-- 1 circleci circleci 122K May 23 10:53 libfbjni.so
-rw-r--r-- 1 circleci circleci  27M May 23 10:53 libpytorch_jni_lite.so

The PR fixes the issue by changing the expected name of the library.

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

stamping

@datumbox datumbox merged commit c89cd06 into pytorch:master May 24, 2021
@datumbox datumbox deleted the ci/fix_android branch May 24, 2021 09:27
facebook-github-bot pushed a commit that referenced this pull request May 25, 2021
Reviewed By: vincentqb, cpuhrsch

Differential Revision: D28679972

fbshipit-source-id: a54ff7d736b0747fc645845048ac1bbd18c3114f
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.

3 participants