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

flag expr_proto_desc_sets unrecognized in few versions of bazel #174

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

basavaraj29
Copy link

@basavaraj29 basavaraj29 commented Sep 18, 2020

the flag, experimental_proto_descriptor_sets_include_source_info, isn't supported in few versions of Bazel, which leads to the following issue while trying to import and use go_proto_library targets from the repo.

$ bazel build @bazel_remote_apis//build/bazel/remote/execution/v2:remote_execution_go_proto
ERROR: /home/basav/.cache/bazel/_bazel_basav/<>/external/bazel_remote_apis/build/bazel/remote/execution/v2/BUILD:43:1: GoCompilePkg external/bazel_remote_apis/build/bazel/remote/execution/v2/linux_amd64_stripped/remote_execution_go_proto%/github.com/bazelbuild/remote-apis/build/bazel/remote/execution/v2.a failed (Exit 1) builder failed: error executing command bazel-out/host/bin/external/go_sdk/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -src ... (remaining 51 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
compilepkg: missing strict dependencies:
	/home/basav/.cache/bazel/_bazel_basav/<>/sandbox/processwrapper-sandbox/767/execroot/<>/bazel-out/k8-fastbuild/bin/external/bazel_remote_apis/build/bazel/remote/execution/v2/linux_amd64_stripped/remote_execution_go_proto%/github.com/bazelbuild/remote-apis/build/bazel/remote/execution/v2/remote_execution.pb.go: import of "build/bazel/semver"
No dependencies were provided.
Check that imports in Go sources match importpath attributes in deps.
Target @bazel_remote_apis//build/bazel/remote/execution/v2:remote_execution_go_proto failed to build

Bazel Version: 1.0.0

@googlebot googlebot added the cla: yes Pull requests whose authors are covered by a CLA with Google. label Sep 18, 2020
@EdSchouten
Copy link
Collaborator

Out of interest, what are the versions of Bazel that we want to support in this repository? Why hasn't CI caught this issue? Would we want to set up a CI build for the oldest version of Bazel that we do support?

@bergsieker bergsieker added the BUILD-Codegen Related to BUILD, Bazel, Code Generation, etc. label Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUILD-Codegen Related to BUILD, Bazel, Code Generation, etc. cla: yes Pull requests whose authors are covered by a CLA with Google.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants