Open VitaliKaiser opened 6 years ago
Can you provide the output of bazel version
and java -version
?
I'm guessing this is an issue with rules_closure, maybe this one: https://github.com/bazelbuild/rules_closure/issues/275
Since TensorBoard builds today, it's possible this is fixed by updating rules_closure. Could you try, in your version of the plugin-example repo's WORKSPACE file, replacing the rules_closure http_archive here https://github.com/tensorflow/tensorboard-plugin-example/blob/f0408974bf76faaed006bcc8e4ca7433af1154fc/WORKSPACE#L14-L22
with the TensorBoard version here: https://github.com/tensorflow/tensorboard/blob/2370c4803f5d5e68f7c3190267316259e259d3bf/WORKSPACE#L3-L11
Hi,
sorry for the late response! I tried to to replace "io_bazel_rules_closure" but getting a new error :see_no_evil:
ERROR: /external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl:25:1: file '@io_bazel_rules_closure//closure/private:defs.bzl' does not contain symbol 'CLOSURE_LIBRARY_DEPS_ATTR' (did you mean 'CLOSURE_LIBRARY_BASE_ATTR'?)
ERROR: /external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl:418:34: Traceback (most recent call last):
File "/external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl", line 382
rule(implementation = _tf_web_library, <3 more arguments>)
File "/external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl", line 385, in rule
dict((CLUTZ_ATTRIBUTES.items() + {"pa...()))
File "external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl", line 385, in dict
{"path": attr.string(), "srcs": attr.label_list(allow_files = True), "deps": attr.label_list(aspects = [web_aspect, clutz_aspect, legacy_js]), "exports": attr.label_list(), "data": attr.label_list(cfg = "data", allow_files = True), "suppress": attr.string_list(), "strip_prefix": attr.string(), "external_assets": attr.string_dict(default = {"/_/runfiles": "."}), "clutz_entry_points": attr.string_list(), "_execrooter": attr.label(default = Label("//tensorboard/scripts:execrooter"), executable = True, cfg = "host"), "_tsc": attr.label(default = Label("@com_microsoft_typescript//:tsc"), allow_files = True, executable = True, cfg = "host"), "_default_typings": attr.label(default = Label("//tensorboard:tf_web_library_default_typings"), allow_files = True), "_WebfilesServer": attr.label(default = Label("@io_bazel_rules_closure//java/io/bazel/rules/closure/webfiles/server:WebfilesServer"), executable = True, cfg = "host"), "_ClosureWorker": _CLOSURE_WORKER, "_closure_library_base": CLOSURE_LIBRARY_BASE_ATTR, "_closure_library_deps": CLOSURE_LIBRARY_DEPS_ATTR}.items()
File "./external/org_tensorflow_tensorboard/tensorboard/defs/web.bzl", line 418, in {"path": attr.string
CLOSURE_LIBRARY_DEPS_ATTR
name 'CLOSURE_LIBRARY_DEPS_ATTR' is not defined (did you mean 'CLOSURE_LIBRARY_BASE_ATTR'?)
ERROR: Skipping 'greeter_tensorboard': error loading package 'greeter_tensorboard': Extension file 'tensorboard/defs/web.bzl' has errors
WARNING: Target pattern parsing failed.
bazel version:
Build label: 0.17.1
java -version
java version "1.8.0_162"
Best, Vitali
To me the suggestion from @nfelt did it. Here's the patch I used:
--- WORKSPACE 2018-12-15 15:28:23.319897000 +0000
+++ ../WORKSPACE 2018-12-15 15:28:17.178279000 +0000
@@ -13,11 +13,11 @@
http_archive(
name = "io_bazel_rules_closure",
- sha256 = "a80acb69c63d5f6437b099c111480a4493bad4592015af2127a2f49fb7512d8d",
- strip_prefix = "rules_closure-0.7.0",
+ sha256 = "b29a8bc2cb10513c864cb1084d6f38613ef14a143797cea0af0f91cd385f5e8c",
+ strip_prefix = "rules_closure-0.8.0",
urls = [
- "https://mirror.bazel.build/github.com/bazelbuild/rules_closure/archive/0.7.0.tar.gz",
- "https://github.com/bazelbuild/rules_closure/archive/0.7.0.tar.gz", # 2018-05-09
+ "https://mirror.bazel.build/github.com/bazelbuild/rules_closure/archive/0.8.0.tar.gz",
+ "https://github.com/bazelbuild/rules_closure/archive/0.8.0.tar.gz", # 2018-08-03
],
)
Hi,
when I run
bazel build greeter_tensorboard
I get the following error:
compiling the current tensorboard source works out of the box.
I dont have to much experience with bazel, so maybe you could give me a hint?
Best, Vitali