mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <>
Subject Re: Review Request 61365: Added `get_byproducts()` function to fix Ninja generator.
Date Wed, 02 Aug 2017 01:03:51 GMT

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

(Updated Aug. 1, 2017, 6:03 p.m.)

Review request for mesos and Joseph Wu.


Add Joseph and bugs.

Bugs: MESOS-3576, MESOS-5656 and MESOS-6794

Repository: mesos


CMake also supports Ninja as a generator, but Ninja requires that each
use of `ExternalProject_Add` specifies its `BUILD_BYPRODUCTS`.
Interestingly, it doesn't care about interface libraries, and
furthermore, it can't derive the dependency from other
information (despite it being available).


  3rdparty/CMakeLists.txt f4feaf90b4750338cfa2127524a5e6556a86d063 


Testing (updated)

This is the last patch in a long chain of CMake refactoring.

This has been tested on Windows 10, CentOS 7, and macOS. While I've worked on it extensively,
it could still use yet more testing by others to ensure all configurations and environments
still work properly.


Andrew Schwartzmeyer

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