Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Updated grpc and protobuf for newer bazel versions #137

Merged
merged 4 commits into from
Oct 10, 2019

Conversation

kwek20
Copy link
Contributor

@kwek20 kwek20 commented Sep 10, 2019

Fixes #135
Tested on bazel 0.28.1

Updated docker-rules from v0.5.1 to v0.9.0
Updated GRPC from v1.17.2 to v1.23.0
Updated protobuf from v3.6.1.3 to v3.8.0

@nelsonchan1996
Copy link

still can't run it:

root@vultr:~/compass# bazel run //docker:layers_calculator
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:69:1: file '//container:layer.bzl' does not contain symbol 'LayerInfo' (did you mean 'layer'?)
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:396:15: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 396
dict(((((_layer.attrs.items() + {"bas...()))
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 396, in dict
_layer
global variable '_layer' is referenced before assignment.
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:438:17: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 438
dict(_layer.outputs)
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 438, in dict
_layer
global variable '_layer' is referenced before assignment.
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:440:1: global variable '_outputs' is referenced before assignment.
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:443:13: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 442
struct(attrs = _attrs, outputs = _outputs, ...)
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 443, in struct
_attrs
global variable '_attrs' is referenced before assignment.
ERROR: /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl:449:13: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 448
rule(attrs = _attrs, executable = True, o..., ...)
File "/root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/image.bzl", line 449, in rule
_attrs
global variable '_attrs' is referenced before assignment.
ERROR: error loading package '': in /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/container.bzl: Extension file 'container/image.bzl' has errors
ERROR: error loading package '': in /root/.cache/bazel/_bazel_root/506ac1cd9efd19d09d738427a1de2bb4/external/io_bazel_rules_docker/container/container.bzl: Extension file 'container/image.bzl' has errors
INFO: Elapsed time: 0.070s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
FAILED: Build did NOT complete successfully (0 packages loaded)

@kwek20
Copy link
Contributor Author

kwek20 commented Sep 11, 2019

@nelsonchan1996 Could you try and start with a fresh environment? i tested it on your exact bazel version, and it seems to work!

@kwek20
Copy link
Contributor Author

kwek20 commented Sep 23, 2019

@nelsonchan1996 I confirmed on a new machine this is not yet solved. Ill take a look at it again later

Copy link
Contributor

@karimodm karimodm left a comment

Choose a reason for hiding this comment

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

Build breaks instantly with the errors reported by @nelsonchan1996

@karimodm
Copy link
Contributor

Builds for me now, merging.

@karimodm karimodm merged commit 1264e19 into iotaledger:master Oct 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error at Compass Instalation
3 participants