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

Update Bazel dependencies on GRPC and protoc_gen_validate #7115

Merged
merged 3 commits into from
May 31, 2019

Conversation

laurentlb
Copy link
Contributor

They include fixes for future Bazel changes. In particular, the
repository can now build with --incompatible_depset_is_not_iterable,
which will be included in Bazel 0.27.

Description:
Risk Level: low
Testing: bazel build --nobuild ... --incompatible_depset_is_not_iterable

#6995

@laurentlb
Copy link
Contributor Author

cc @keith

@moderation
Copy link
Contributor

Rather than add com_envoyproxy_protoc_gen_validate in addition to com_lyft_protoc_gen_validate it would be good to replace the old name with the new one everywhere - https://github.com/envoyproxy/envoy/search?utf8=✓&q="com_lyft_protoc_gen_validate"&type=

When will a new version of PGV be cut with the commit c15f2c24fb27b136e722fa912accddd0c8db9dfa @rodaine? Would allow us to continue referencing a release vs. a commit.

They include fixes for future Bazel changes. In particular, the
repository can now build with `--incompatible_depset_is_not_iterable`,
which will be included in Bazel 0.27.

Signed-off-by: Laurent Le Brun <laurentlb@gmail.com>
…date

Signed-off-by: Laurent Le Brun <laurentlb@gmail.com>
It looks like more recent commits in PGV cause a failure.

Signed-off-by: Laurent Le Brun <laurentlb@gmail.com>
@keith
Copy link
Member

keith commented May 30, 2019

I asked about a new release on that repo but it's blocked on bufbuild/protoc-gen-validate#191

Copy link
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

Thanks!

@htuch htuch merged commit 0739cd6 into envoyproxy:master May 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants