Yes we are indeed using protoc version 3.0.0. I'll try out using protoc v2. Thanks.
Meanwhile, is there an issue or conversation about upgrading Mesos to protoc v3? Not sure whether there's actual benefit. On Mon, Jun 13, 2016 at 2:51 PM, Joseph Wu <[email protected]> wrote: > Looks like that is a warning in v3, see [1]. The same code in v2.6.1 is > [2], and does not have that warning. > > [1] > > https://github.com/google/protobuf/blob/088c5c491e7a1c95c7b8eb55f119a8a999c81dc1/src/google/protobuf/compiler/parser.cc#L547-L550 > [2] > > https://github.com/google/protobuf/blob/bba83652e1be610bdb7ee1566ad18346d98b843c/src/google/protobuf/compiler/parser.cc#L436-L444 > > On Mon, Jun 13, 2016 at 2:46 PM, Vinod Kone <[email protected]> wrote: > > > Are you are using a v3 protoc compiler? Looks like the "syntax" keyword > was > > introduced in > > https://github.com/google/protobuf/releases/tag/v3.0.0-alpha-2. > > Not sure if v2 protoc compilers recognize that keyword. > > > > On Mon, Jun 13, 2016 at 5:42 PM, Zhitao Li <[email protected]> > wrote: > > > > > Hi, > > > > > > We are experimenting with HTTP API and tried to compile Mesos's .proto > > > files with our repo (with make commands like "protoc > > --proto_path=protobuf > > > --go_out=.gen protobuf/mesos/v1/scheduler/scheduler.proto", but we see > > > warnings like > > > > > > ``` > > > [libprotobuf WARNING google/protobuf/compiler/parser.cc:547] No syntax > > > specified for the proto file: mesos/v1/scheduler/scheduler.proto. > > > Please use 'syntax > > > = "proto2";' or 'syntax = "proto3";' to specify a syntax version. > > > (Defaulted > > > to proto2 syntax.) > > > ``` > > > > > > Should we have proper `syntax` clauses in various .proto files? If not, > > > what's the reason? > > > > > > Thanks. > > > > > > -- > > > Cheers, > > > > > > Zhitao Li > > > > > > -- Cheers, Zhitao Li
