mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Clemmer" <>
Subject Re: Review Request 38539: [VIA HAOSDENT] [1/2]Add CMake macro VsBuildCommand in libprocess.
Date Fri, 25 Sep 2015 22:08:11 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Sept. 25, 2015, 10:08 p.m.)

Review request for mesos, haosdent huang, Artem Harutyunyan, Joris Van Remoortere, and Joseph

Bugs: MESOS-3434

Repository: mesos


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:

Diffs (updated)

  3rdparty/libprocess/3rdparty/CMakeLists.txt b9c9fae7d448906e9c9f5ab0ee3fe138a0171a7d 
  3rdparty/libprocess/cmake/macros/VsBuildCommand.bat PRE-CREATION 
  3rdparty/libprocess/cmake/macros/VsBuildCommand.cmake PRE-CREATION 



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


Alex Clemmer

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message