> On April 12, 2018, 4:41 p.m., Andrew Schwartzmeyer wrote: > > 3rdparty/CMakeLists.txt > > Line 143 (original), 144-145 (patched) > > <https://reviews.apache.org/r/61096/diff/6/?file=1996892#file1996892line144> > > > > Seems reasonable. Could alternatively setup `ENABLE_GRPC` to set > > `ENABLE_SSL` (and therefore also `libevent`?)
As discussed, now I only build `gpr`, `grpc_cpp_plugin`, and `grpc++_unsecure` and `grpc_unsecure` or `grpc++` and `grpc` depending on whether `ENABLE_SSL` is on. > On April 12, 2018, 4:41 p.m., Andrew Schwartzmeyer wrote: > > 3rdparty/CMakeLists.txt > > Lines 1012-1015 (patched) > > <https://reviews.apache.org/r/61096/diff/6/?file=1996892#file1996892line1014> > > > > As stated above, the probably won't work on Windows yet. Ths VS > > solutions generally stick everything under `Debug` or `Release` folders, > > and it's done at build time, not configuration time. It's super annoying. Temporarily disable the WIN32 build. Can you help me on this since I don't have an available Windows machine to test the build? :( > On April 12, 2018, 4:41 p.m., Andrew Schwartzmeyer wrote: > > 3rdparty/CMakeLists.txt > > Lines 1022-1027 (patched) > > <https://reviews.apache.org/r/61096/diff/6/?file=1996892#file1996892line1024> > > > > Do you want the `C` or `CXX` flags forwarded too? Thanks for pointing this out. Actually it's not only the flags, but the compiler themselves should be forwarded as well. - Chun-Hung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61096/#review201007 ----------------------------------------------------------- On April 12, 2018, 3:22 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61096/ > ----------------------------------------------------------- > > (Updated April 12, 2018, 3:22 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Jie Yu, and > Joseph Wu. > > > Bugs: MESOS-7881 > https://issues.apache.org/jira/browse/MESOS-7881 > > > Repository: mesos > > > Description > ------- > > This patch adds an `ENABLE_GRPC` option and builds the bundled gRPC > 3rdparty library in CMake. > > > Diffs > ----- > > 3rdparty/CMakeLists.txt 488e906486a583e74faceb44d906cee5036a8b99 > 3rdparty/cmake/Versions.cmake 93f0322c1ac926bcfdcd4c1cfd9ba9f22bcf7099 > cmake/CompilationConfigure.cmake 3cb072ddcd286c0e40d44eaeba210ddf1796975c > src/cmake/MesosProtobuf.cmake dde034fe4d4251dd7c7397d422136bfa3c9c3ebc > > > Diff: https://reviews.apache.org/r/61096/diff/6/ > > > Testing > ------- > > `sudo make check` with CMake > > NOTE: Testing on Windowns is not done yet. > > > Thanks, > > Chun-Hung Hsiao > >