On Wed, Jun 12, 2024 at 6:10 AM Reinhard Tartler <siret...@gmail.com> wrote: > > On Mon, Jun 10, 2024 at 8:19 AM Shengjing Zhu <z...@debian.org> wrote: >> >> On Mon, Jun 10, 2024 at 6:45 PM Reinhard Tartler <siret...@gmail.com> wrote: >> > On Mon, Jun 10, 2024 at 2:48 AM Shengjing Zhu <z...@debian.org> wrote: >> >> >> Though removing src:golang-github-golang-protobuf-1-3 probably will >> >> >> not be easy. >> >> > >> >> > Can you please elaborate on where you see the difficulties? >> >> >> >> Only pb.go files generated by protoc-gen-go-1-3 are compatible with >> >> gogo protobuf. That's where the protobuf transition stuck... >> > >> > Well, I suppose that means either protoc-gen-go-1-3 or protoc-gen-gogo >> > (from gogoprotobuf). >> > >> > Can you please point out examples of packages with "pb.go" files that are >> > generated with protoc-gen-go-1-3 and then used with gogo protobuf? >> > >> >> Hmm, I thought you already saw that. You mentioned >> https://github.com/containerd/ttrpc/issues/62#issuecomment-686768932 >> in another thread. >> This is caused by using gogo protobuf (at runtime) with pb.go files >> (from golang-google-genproto-dev) generated by protoc-gen-go-1-5 (or >> newer). > > > Thanks for the reminder. > > I've started looking at ttrpc. This is an interesting package. Basically, > they have implemented a "lightweight" grpc implementation, by replacing the > framing mechanism to make it use less memory. That's cool. Their use of > gogoproto is certainly problematic. Also, they provide a code generator that > does not appear to be used by any package in Debian. I see no good coming > from keeping this package in Debian, so I took the liberty of dropping that > binary package. Please do let me know of any breakages, adding it back is > easy and quick at this point. >
The ttrpc generator is used by containerd to generate code, but currently due to lacking protobuild in Debian, this is not implemented in containerd packaging. But I see you start to package protobuild, the ttrpc generator could be kept. -- Shengjing Zhu