buildGo.protos doesn't work with multiple related .proto files

#221
Opened by flokli at 2022-11-19T20·52+00

If you have multiple .proto files that import each other, there's no way to express this with buildGo.protos.

  1. IMO, buildGo.protos was a mistake and should be removed. We should instead just generate the protos in a derivation and use those as the source in a normal buildGo call.

    tazjin at 2022-12-03T13·27+00

  2. I'm still more in favor of having the generated .pb.go files in the repository, and a linter ensuring consistency, but let's see how this all works out.

    c/7536 removes buildGo.proto and buildGo.grpc as proposed here.

    flokli at 2022-12-06T15·16+00

  3. Turns out we can't use this just yet, as depot-scanner still uses this.

    @tazjin should we move this to generated .pb.go files?

    flokli at 2022-12-06T15·20+00

  4. We've deleted this.

    tazjin at 2023-01-08T11·23+00

  5. tazjin closed this issue at 2023-01-08T11·23+00