[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