bazel-contrib / bazel-contrib/rules_go

remove dependency on go-genproto

Open
#2,845 9 comments 5 reactions 0 assignees View on GitHub
code-health proto
Dominant language
Go
Stars
1.5k
Forks
760
Avg merge
1d 11h
Merged PRs (30d)
12

Description

### What version of rules_go are you using?

v0.26.0

### What version of gazelle are you using?

v0.23.0

### What version of Bazel are you using?

4.0.0

### Does this issue reproduce with the latest releases of all the above?

Yes

### What operating system and processor architecture are you using?

Linux / MacOS, AMD64.

### What did you do?

Used rules_go with [cloud.google.com/go](https://github.com/googleapis/google-cloud-go).

### What did you expect to see?

A build that compiled.

### What did you see instead?

The build failed due to rules_go declaring an older version of [go-genproto](https://github.com/googleapis/go-genproto). It is possible to get everything working by using the override instructions but that is not super intuitive. Unless there is a need for the genproto to be bundled I suggest removing the dependency on it so users of rules_go can more easily use the latest version of cloud.google.com/go without the need for overriding dependencies.

Contributor guide

Open the contributing guide

Research direction

Start by examining rules_go's dependency declarations and the override instructions referenced in the issue, then reproduce the cloud.google.com/go build with the stated rules_go, Gazelle, and Bazel versions. Done means the build compiles without requiring users to override the go-genproto dependency.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.