mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <>
Subject Review Request 59773: Removed `ENABLE_DEBUG` CMake option.
Date Sat, 03 Jun 2017 18:49:13 GMT

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

Review request for mesos, Jeff Coffler, Joseph Wu, and Li Li.

Repository: mesos


This option was added as a mapping from the Autotools `--enable-debug`
flag. However, it does not work as expected, since some CMake generators
are multi-configuration. That is, CMake does not necessarily configure a
particular Debug or Release configuration, and instead (for some
generators, such as Visual Studio) it produces build files where the
congfiguration can be changed at build time.

The CMake documentation states that the variable `CMAKE_BUILD_TYPE`
should never be set in a CMake file (although it can be referenced, as
CMake is automatically invoked during builds, where it may be defined).
Thus multi-configuration generators did not respect what a user intended
with `-DENABLE_DEBUG=1` nor with `-DENABLE_DEBUG=0`, and the default
configuration was meaningless.


  cmake/CompilationConfigure.cmake a0032a35d5c02cc6e7a5e90c187dae4954879f95 




Andrew Schwartzmeyer

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