----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38539/#review99788 -----------------------------------------------------------
Ship it! Definitely looks a lot cleaner with the NOOP macro :) 3rdparty/libprocess/cmake/macros/VsBuildCommand.bat (line 19) <https://reviews.apache.org/r/38539/#comment156761> Period at the end. 3rdparty/libprocess/cmake/macros/VsBuildCommand.bat (line 52) <https://reviews.apache.org/r/38539/#comment156760> s/not/does not/ s/current/the current/ And a period at the end. - Joseph Wu On Sept. 20, 2015, 7:40 p.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38539/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2015, 7:40 p.m.) > > > Review request for mesos, haosdent huang, Artem Harutyunyan, Joris Van > Remoortere, and Joseph Wu. > > > Bugs: MESOS-3434 > https://issues.apache.org/jira/browse/MESOS-3434 > > > Repository: mesos > > > Description > ------- > > Add CMake macro VsBuildCommand in libprocess. > > This commit is the product of two reviews; the first one[1], by > haosdent, was lightly rebased by hausdorff to target a later HEAD, and > then augmented to fix the broken Ubuntu 14.04 build that this commit > caused. Other than those things, the review is functionally identical to > the original, and all credit for the work should go to haosdent. > > [1] Original review: https://reviews.apache.org/r/37273 > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/CMakeLists.txt > b9c9fae7d448906e9c9f5ab0ee3fe138a0171a7d > 3rdparty/libprocess/cmake/macros/VsBuildCommand.bat PRE-CREATION > 3rdparty/libprocess/cmake/macros/VsBuildCommand.cmake PRE-CREATION > > Diff: https://reviews.apache.org/r/38539/diff/ > > > Testing > ------- > > Compiled and ran made sure libprocess and stout tests ran and passed on the > following platforms: > > * OS X 10.10 > * Windows 10 > * Ubuntu 14.04.2 > > > Thanks, > > Alex Clemmer > >