mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Coffler <j...@taltos.com>
Subject Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
Date Tue, 18 Apr 2017 20:30:52 GMT

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



Big change, but important for us.

I'd like you to comment on using cmake for zookeeper, specifically in terms of rolling that
back to Zookeeper themselves for them to maintain it.

Have you looked at contributing this to the Zookeeper folks for them to use and maintain?
What was their response?

This feels like something that is easily broken (mentioning specific sources, etc) if they
aren't maintaining it, and I'd rather not own this for life. If the Zookeeper folks don't
want to maintain this, what do they say about VS 2017 support? Even if it's not cmake, but
a hand-generated solution for V/S, at least they maintain it, not us.

- Jeff Coffler


On April 14, 2017, 4:12 a.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58448/
> -----------------------------------------------------------
> 
> (Updated April 14, 2017, 4:12 a.m.)
> 
> 
> Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This unblocks us from building exclusively with VS 2017. The previous
> patch to ZooKeeper only added VS 2015 support. This patch replaces it
> with a CMake build system that will generate whichever solution we need
> for Windows (and can replace the Autotools system on Linux).
> 
> We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a
> source tarball, and so missing the necessary generated files. The most
> currently used version was based off a random commit. 3.5.2-alpha is the
> latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is
> semi-stable, in comparison to 3.5.3 which is in RC).
> 
> 
> Diffs
> -----
> 
>   3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 
>   3rdparty/cmake/Mesos3rdpartyConfigure.cmake c60652688a23f8628f133b7890ff39e38fc8ae94

>   3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be 
>   3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/58448/diff/1/
> 
> 
> Testing
> -------
> 
> cmake and make check on Linux
> stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. (Note: my
machine updated to the Creators Update today, disabling long path support; the previous way
to enable it doesn't seem to work, so I re-ran the failed tests on a machine that hadn't updated
and still had long path support, they passed).
> 
> Also tested the build on a clean machine with nothing but the VS 2017 build tools (no
IDE, no prior VS installations); mesos-tests builds no problem.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>


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