[dnstap] [notifications at github.com: [dnstap/dnstap.pb] Add the syntax version (#4)]
Robert Edmonds
edmonds at mycre.ws
Tue Nov 22 17:07:02 UTC 2016
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 <notifications at github.com> -----
Date: Sat, 19 Nov 2016 02:39:46 -0800
From: Frank Denis <notifications at github.com>
To: "dnstap/dnstap.pb" <dnstap.pb at noreply.github.com>
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
More information about the dnstap
mailing list