Skip to content

Commit 2a7632b

Browse files
author
Ken Harris
committed
Add protobuf syntax version
The protocol buffer spec suggests that this is required. I've tried two protobuf libraries, and one errors without this, and the other prints a warning. There's no documentation in OLA I can find that it's proto2, but it works this way, and it makes sense. Both files use "required", which was removed from proto3.
1 parent 1a68747 commit 2a7632b

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

common/protocol/Ola.proto

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@
1818
* Copyright (C) 2005 Simon Newton
1919
*/
2020

21+
syntax = "proto2";
22+
2123
package ola.proto;
2224

2325
option cc_generic_services = false;

common/rpc/Rpc.proto

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@
1818
* Copyright (C) 2005 Simon Newton
1919
*/
2020

21+
syntax = "proto2";
22+
2123
/*
2224
* Based on, but not guaranteed to be the same as the specification here:
2325
* http://protorpc.likbilen.com/Protorpcdocprotobuf.html

0 commit comments

Comments
 (0)