----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55030/#review161306 -----------------------------------------------------------
3rdparty/libprocess/cmake/ProcessConfigure.cmake (lines 59 - 72) <https://reviews.apache.org/r/55030/#comment232546> s/Process/Libprocess/ ^ Not in the path, only in the first argument of `source_group`. Even if the "lib" part is a unix semantic, the full name of this 3rdparty library is "libprocess". 3rdparty/libprocess/src/CMakeLists.txt (lines 72 - 78) <https://reviews.apache.org/r/55030/#comment232537> There are two other SSL .cpp files: * openssl.cpp * libevent_ssl_socket.cpp Note that these are not in the `ssl` subfolder. 3rdparty/libprocess/src/CMakeLists.txt (line 140) <https://reviews.apache.org/r/55030/#comment232548> s/Process/Libprocess/ 3rdparty/libprocess/src/tests/CMakeLists.txt (lines 80 - 81) <https://reviews.apache.org/r/55030/#comment232552> What's the effect of adding these `source_groups` in multiple CMake lists? Is the only effect that each subproject in MSVC will show the files in an organized way? - Joseph Wu On Dec. 24, 2016, 9:39 p.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55030/ > ----------------------------------------------------------- > > (Updated Dec. 24, 2016, 9:39 p.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, John Kordich, > and Joseph Wu. > > > Bugs: MESOS-3542 > https://issues.apache.org/jira/browse/MESOS-3542 > > > Repository: mesos > > > Description > ------- > > Currently in IDEs such as XCode and Visual Studio, all mesos source > files are grouped into one huge folder, with no attempt made to reflect > the hierarchy of folders that exist on disk. > > This commit groups libprocess files into relevant source groups so that > they appear in a sensible directory structure in these IDEs. > > In addition to beginning to directly address MESOS-3542, this is also > (indirectly) the first step towards addressing the problem of breaking > libmesos up into smaller binaries (MESOS-3542). > > > Diffs > ----- > > 3rdparty/libprocess/cmake/ProcessConfigure.cmake > 873f41d844faa0d442f77411e94314a89be5f046 > 3rdparty/libprocess/src/CMakeLists.txt > 60f0e76dfd237d9a12a366b413802d1a96892b55 > 3rdparty/libprocess/src/tests/CMakeLists.txt > 4b80c397381ca2c869cd6eb7507bb9df94ce3623 > > Diff: https://reviews.apache.org/r/55030/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >