mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <and...@schwartzmeyer.com>
Subject Re: Review Request 65721: Windows: Specified byproducts of `sasl2` imported target.
Date Tue, 06 Mar 2018 20:05:47 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65721/
-----------------------------------------------------------

(Updated March 6, 2018, 12:05 p.m.)


Review request for mesos, Akash Gupta, Benjamin Bannier, Jeff Coffler, and Joseph Wu.


Changes
-------

Rebased.


Bugs: MESOS-8599
    https://issues.apache.org/jira/browse/MESOS-8599


Repository: mesos


Description
-------

This is necessary to enable the Ninja build. This was not previously
fixed because we have not yet used Ninja on Windows. The same pattern
is used for other imported targets.


Diffs (updated)
-----

  3rdparty/CMakeLists.txt da605707b89bbe9b3db9e60bc0b0a26dac46e56e 


Diff: https://reviews.apache.org/r/65721/diff/2/

Changes: https://reviews.apache.org/r/65721/diff/1-2/


Testing
-------

Built with Ninja on Windows in 17 minutes, versus 21 minutes using the VS generator (both
builds completely from scratch).

Note that this isn't perfect; Ninja generates some warnings because the imported libraries
have both debug and release configurations pointing to the same location. We can ignore this
for now, as the build works, but it's ugly.


Thanks,

Andrew Schwartzmeyer


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