Use `protoc` of a specific version #79

New issue

Open

opened 2025-02-05 11:44:40 +00:00 by a-savchuk · 3 comments

a-savchuk commented

2025-02-05 11:44:40 +00:00

Member

I've encountered an error while regenerating the documentation:

$ protoc --version
libprotoc 3.21.12
$ make doc 2>&1 | head
⇒ Documentation for accounting
[libprotobuf WARNING google/protobuf/compiler/parser.cc:646] No syntax specified for the proto file: accounting/service.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)
accounting/service.proto:1:1: Expected top-level statement (e.g. "message").
accounting/service.proto:36:5: Expected "required", "optional", or "repeated".
accounting/service.proto:39:3: Expected "required", "optional", or "repeated".
accounting/service.proto:43:3: Expected "required", "optional", or "repeated".
accounting/service.proto:48:3: Expected "required", "optional", or "repeated".
accounting/service.proto:58:5: Expected "required", "optional", or "repeated".
accounting/service.proto:61:3: Expected "required", "optional", or "repeated".
accounting/service.proto:65:3: Expected "required", "optional", or "repeated".

It would be helpful to have a Makefile target for installing protoc of a specific version, similar to how it's done in frostfs/sdk-go.

I've encountered an error while regenerating the documentation: ```console $ protoc --version libprotoc 3.21.12 $ make doc 2>&1 | head ⇒ Documentation for accounting [libprotobuf WARNING google/protobuf/compiler/parser.cc:646] No syntax specified for the proto file: accounting/service.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.) accounting/service.proto:1:1: Expected top-level statement (e.g. "message"). accounting/service.proto:36:5: Expected "required", "optional", or "repeated". accounting/service.proto:39:3: Expected "required", "optional", or "repeated". accounting/service.proto:43:3: Expected "required", "optional", or "repeated". accounting/service.proto:48:3: Expected "required", "optional", or "repeated". accounting/service.proto:58:5: Expected "required", "optional", or "repeated". accounting/service.proto:61:3: Expected "required", "optional", or "repeated". accounting/service.proto:65:3: Expected "required", "optional", or "repeated". ``` It would be helpful to have a Makefile target for installing `protoc` of a specific version, similar to how it's done in [frostfs/sdk-go](https://git.frostfs.info/TrueCloudLab/frostfs-sdk-go/src/commit/5a35fa4353c1584c7ea4770de41ba86c30ef990a/Makefile).

👀 1