search

farsightsec / nmsg

network message encapsulation library
Apache License 2.0
30 stars 8 forks source link

get rid of the protobuf version warnings #69

Closed reedjc closed 4 years ago

reedjc commented 5 years ago

Explicitly specify the protocol version?

GEN nmsg/nmsg.pb-c.c [libprotobuf WARNING google/protobuf/compiler/parser.cc:547] No syntax specified for the proto file: nmsg/nmsg.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.) ... GEN nmsg/base/dnstap.pb-c.c [libprotobuf WARNING google/protobuf/compiler/parser.cc:547] No syntax specified for the proto file: nmsg/base/dnstap.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)

morrowc commented 5 years ago

This is done, acutally in: https://github.com/farsightsec/nmsg/pull/78

so, maybe read and accept that PR? :)

morrowc commented 4 years ago

this is fixed now, I believe.