-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62176/
-----------------------------------------------------------
(Updated Sept. 8, 2017, 10:35 p.m.)
Review request for mesos, Andrew Schwartzmeyer, Jeff Coffler, and Joseph Wu.
Changes
-------
Addressed suggested comments
Bugs: MESOS-3110
https://issues.apache.org/jira/browse/MESOS-3110
Repository: mesos
Description
-------
Added cmake dependency check for libsasl2 on non-Windows platforms.
Diffs (updated)
-----
3rdparty/CMakeLists.txt 2a183a4a46bf8bc97455fe02648407ba561b38e7
Diff: https://reviews.apache.org/r/62176/diff/2/
Changes: https://reviews.apache.org/r/62176/diff/1-2/
Testing
-------
I tested this on my Ubuntu 16.04 system. When the libsasl2 library doesn't
exist, it fails the cmake configure/build. I did not test this on any other
platform, but this code is in a "if (NOT WIN32)" block and won't affect a
Windows build. I'm uncertain if there is much support for other kinds of
builds (like Mac OS), but this should be platform independent.
Thanks,
John Kordich
