When will grpc move to protobuf 3.1.0?

464 views
Skip to first unread message

Zachary Deretsky

unread,
Oct 10, 2016, 9:28:56 PM10/10/16
to grpc.io
Protobuf 3.1.0 fixed Json related issues.

I replaced the grpc/third_party/protobuf directory with the untarred protobuf-cpp-3.1.0.tar.gz but the compilation fails because of incompatibility:
include/grpc++/ext/reflection.pb.h:51:2: 
#error This file was generated by an older version of protoc which is incompatible with your Protocol Buffer headers. Please regenerate this file with a newer version of protoc.


Is there plan to move grpc to protobuf 3.1.0 so that recommended build procedure (see below) works?

 $ git clone -b $(curl -L http://grpc.io/release) https://github.com/grpc/grpc
 $ cd grpc
 $ git submodule update --init
 $ make
 $ [sudo] make install

Nicolas Noble

unread,
Oct 13, 2016, 4:08:09 PM10/13/16
to Zachary Deretsky, grpc.io
The presence of pre-generated headers in the repository was a mistake. We'll try to address this so that the code is agnostic towards whichever protobuf version we have on the system.

--
You received this message because you are subscribed to the Google Groups "grpc.io" group.
To unsubscribe from this group and stop receiving emails from it, send an email to grpc-io+unsubscribe@googlegroups.com.
To post to this group, send email to grp...@googlegroups.com.
Visit this group at https://groups.google.com/group/grpc-io.
To view this discussion on the web visit https://groups.google.com/d/msgid/grpc-io/7e5519ae-813e-46dd-a4f1-2df3be7332c8%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Zachary Deretsky

unread,
Oct 13, 2016, 4:31:52 PM10/13/16
to grpc.io
Please post it here once the issue is resolved. Thanks, Zach.

benyu...@gmail.com

unread,
Nov 1, 2016, 1:32:11 AM11/1/16
to grpc.io
 I have same problem. how to fix it?

在 2016年10月14日星期五 UTC+8上午4:31:52,Zachary Deretsky写道:

Carl Mastrangelo

unread,
Nov 2, 2016, 2:49:59 PM11/2/16
to grpc.io
FYI Java made this change recently, it will be in the next release:



On Monday, October 10, 2016 at 6:28:56 PM UTC-7, Zachary Deretsky wrote:

nelson....@admobilize.com

unread,
Dec 12, 2016, 9:48:10 AM12/12/16
to grpc.io
Is there a workaround? I think I saw a way to tell Make to use the GRPC that is inside the build tree.
Reply all
Reply to author
Forward
0 new messages