----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65997/#review199226 -----------------------------------------------------------
Let me apply these patches on Windows and make sure everything is G2G before a ship it. Just a few minutes... - Andrew Schwartzmeyer On March 14, 2018, 4:05 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65997/ > ----------------------------------------------------------- > > (Updated March 14, 2018, 4:05 p.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Jie Yu, and > Joseph Wu. > > > Bugs: MESOS-8657 > https://issues.apache.org/jira/browse/MESOS-8657 > > > Repository: mesos > > > Description > ------- > > PROTOC_GENERATE now has the following new features: > (1) With the `LIB` option, compile .proto files found in a third-party > specification library. > (2) Provides the `GRPC` option that, once we support gRPC in cmake, > will generate the `.grpc.pb.h` and `.grpc.pb.cc` files. > (3) With the `LIB` option, append to list variable `PUBLIC_PROTO_PATH` > or `INTERNAL_PROTO_PATH` the fully qualified path to the library's > include directory, and append to list variable > `PUBLIC_PROTOBUF_INCLUDE_DIR` or `INTERNAL_PROTOBUF_INCLUDE_DIR` > the fully qualified path to the directory where the generated > `.pb.h` files are placed. > > > Diffs > ----- > > src/CMakeLists.txt 0c135034982a6a4844509cd0e9b2cc41804333ef > src/cmake/MesosProtobuf.cmake ef90c15a6f2afaedaf7559c4a411098924528505 > > > Diff: https://reviews.apache.org/r/65997/diff/4/ > > > Testing > ------- > > `make check` with cmake. > > > Thanks, > > Chun-Hung Hsiao > >
