From edmonds at mycre.ws Tue Nov 22 17:07:02 2016 From: edmonds at mycre.ws (Robert Edmonds) Date: Tue, 22 Nov 2016 12:07:02 -0500 Subject: [dnstap] [notifications@github.com: [dnstap/dnstap.pb] Add the syntax version (#4)] Message-ID: <20161122170702.5vzps7tzmkvnia44@mycre.ws> Hi, Frank Denis submitted the below pull request to the dnstap protobuf schema repository on GitHub; forwarding it to the list to get feedback. I think it is OK to merge, but I suspect it will probably require a protobuf-c linked with protobuf 3 in order to support the 'syntax = "proto2";' directive. ----- Forwarded message from Frank Denis ----- Date: Sat, 19 Nov 2016 02:39:46 -0800 From: Frank Denis To: "dnstap/dnstap.pb" Subject: [dnstap/dnstap.pb] Add the syntax version (#4) ...so that Protobuf 3 doesn't complain about it. Also fix a typo (`authorative` -> `authoritative`). You can view, comment on, or merge this pull request online at: https://github.com/dnstap/dnstap.pb/pull/4 -- Commit Summary -- * Add the syntax version -- File Changes -- M dnstap.proto (3) -- Patch Links -- https://github.com/dnstap/dnstap.pb/pull/4.patch https://github.com/dnstap/dnstap.pb/pull/4.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/dnstap/dnstap.pb/pull/4 ----- End forwarded message ----- -- Robert Edmonds