Merge pull request #1287 from glotzerlab/cmake-color

Enable cmake color output for clang/ninja

CMakeLists.txt

@@ -63,6 +63,17 @@ if(WIN32)
   add_compile_options(/DNOMINMAX)
 endif()
 
+# Enable diagnostic colors for ninja
+if(CMAKE_COMPILER_IS_GNUCXX AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER 5.0)
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fdiagnostics-color=always")
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fdiagnostics-color=always")
+endif()
+
+if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fcolor-diagnostics")
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fcolor-diagnostics")
+endif()
+
 include_directories(
   ${PROJECT_SOURCE_DIR}/cpp/util ${PROJECT_SOURCE_DIR}/cpp/locality
   ${PROJECT_SOURCE_DIR}/cpp/box)

doc/source/reference/credits.rst

@@ -403,6 +403,12 @@ Philipp Schönhöfer
 
 * Contributed code, design, documentation, and testing for ``freud.locality.FilterRAD`` class.
 
+
+Jen Bradley
+
+* Updated CMake to color error messages for build systems that buffer output.
+
+
 Source code
 -----------