Make ninja smart console builds more pretty
authorDavid Zarzycki <dave@znu.io>
Mon, 22 Jun 2020 10:34:49 +0000 (06:34 -0400)
committerDavid Zarzycki <dave@znu.io>
Mon, 22 Jun 2020 10:35:13 +0000 (06:35 -0400)
Summary: CMake's `find_package` outputs to the console on success, which confuses the smart console mode of the `ninja` build system. Let's quiet the success message and manually warn instead.

Reviewers: tstellar, phosek, mehdi_amini

Reviewed By: mehdi_amini

Subscribers: mgorny, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82276

llvm/cmake/modules/VersionFromVCS.cmake

index 1b6519b..18edbea 100644 (file)
@@ -4,7 +4,7 @@
 # extra argument, otherwise uses CMAKE_CURRENT_SOURCE_DIR).
 
 function(get_source_info path revision repository)
-  find_package(Git)
+  find_package(Git QUIET)
   if(GIT_FOUND)
     execute_process(COMMAND ${GIT_EXECUTABLE} rev-parse --git-dir
       WORKING_DIRECTORY ${path}
@@ -45,5 +45,7 @@ function(get_source_info path revision repository)
         set(${repository} ${path} PARENT_SCOPE)
       endif()
     endif()
+  else()
+    message(WARNING "Git not found. Version cannot be determined.")
   endif()
 endfunction()