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

bazel-0.4.4 android dep. error back for tensorflow #2478

Closed
interestingdev opened this issue Feb 3, 2017 · 7 comments
Closed

bazel-0.4.4 android dep. error back for tensorflow #2478

interestingdev opened this issue Feb 3, 2017 · 7 comments
Assignees
Labels
P1 I'll work on this now. (Assignee required) type: bug

Comments

@interestingdev
Copy link

Hi,
I can see you rolled back the commit before and made below issue gone. But now it's back with latest release 0.4.4. Please help provide solution, thanks.

When I run .configure under syntaxnet/tensorflow/, following error appears:

ERROR: /root/.cache/bazel/_bazel_root/18eb17b8355c7d22a446de6d75221628/external/bazel_tools/src/tools/android/java/com/google/devtools/build/android/dexer/BUILD:3:1: no such target '//external:android/dx_jar_import': target 'android/dx_jar_import' not declared in package 'external' defined by /root/development/models/syntaxnet/tensorflow/WORKSPACE and referenced by '@bazel_tools//src/tools/android/java/com/google/devtools/build/android/dexer:dexer'.
ERROR: /root/.cache/bazel/_bazel_root/18eb17b8355c7d22a446de6d75221628/external/bazel_tools/src/tools/android/java/com/google/devtools/build/android/dexer/BUILD:3:1: no such target '//external:android/dx_jar_import': target 'android/dx_jar_import' not declared in package 'external' defined by /root/development/models/syntaxnet/tensorflow/WORKSPACE and referenced by '@bazel_tools//src/tools/android/java/com/google/devtools/build/android/dexer:dexer'.
ERROR: Evaluation of query "deps(//tensorflow/...)" failed: errors were encountered while computing transitive closure.
root@ubuntu-512mb-vpn-live-snapshot-512mb-nyc3-01:/development/models/syntaxnet/tensorflow# vim root/.cache/bazel/_bazel_root/18eb17b8355c7d22a446de6d75221628/external/bazel_tools/src/tools/android/java/com/google/devtools/build/android/dexer/BUILD
root@ubuntu-512mb-vpn-live-snapshot-512mb-nyc3-01:
/development/models/syntaxnet/tensorflow# vim /root/.cache/bazel/_bazel_root/18eb17b8355c7d22a446de6d75221628/external/bazel_tools/src/tools/android/java/com/google/devtools/build/android/dexer/BUILD

Please provide the following information. The more we know about your system and use case, the more easily and likely we can help.

Environment info

ubuntu

  • Bazel version (output of bazel info release):
    0.4.4

Java:
java version "1.8.0_121"
Java(TM) SE Runtime Environment (build 1.8.0_121-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.121-b13, mixed mode)

@meteorcloudy meteorcloudy added category: rules > android P1 I'll work on this now. (Assignee required) type: bug labels Feb 3, 2017
@meteorcloudy
Copy link
Member

@aj-michael Any idea how to fix this?

@aj-michael
Copy link
Contributor

Hmmm, I cleaned this up in the tensorflow repo in tensorflow/tensorflow#6225.

@interestingdev can you clarify as to what git repository you are building (is it https://github.com/tensorflow/tensorflow or something else?) and what commit you are at (run git rev-parse HEAD)

@aj-michael
Copy link
Contributor

Ok, the problem is that https://github.com/tensorflow/models has https://github.com/tensorflow/tensorflow as a submodule, but is using a commit which does not include my fix.

The solution is for the tensorflow/models project to update their submodule. A temporary fix would be git submodule update --remote -f syntaxnet/tensorflow which will update their submodule in your local repository to HEAD. (Although I know nothing about tensorflow's release and versioning process so I don't know if that would cause other problems)

@interestingdev
Copy link
Author

Thanks for your response, now I'm taking bazel-0.4.3 to have tensorflow(syntaxnet) build passed.

@aj-michael aj-michael self-assigned this Feb 8, 2017
@aj-michael
Copy link
Contributor

I'm going to close this because the README.md for models/syntaxnet explicitly says to use bazel 0.4.3: https://github.com/tensorflow/models/blob/master/syntaxnet/README.md. Of course this is less than ideal, their repo should not be breaking on bazel minor version updates, but this particular problem is probably worth filing an issue to their repo about updating their tensorflow version.

@jsamson008
Copy link

Can you please highlight this in the README file. Red text would be best. I've had dozens of people ask about this problem. And with reasonably good Google foo.

@kchodorow
Copy link
Contributor

Could you report this on the syntaxnet repo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 I'll work on this now. (Assignee required) type: bug
Projects
None yet
Development

No branches or pull requests

5 participants