[PATCH] D82276: Make ninja smart console builds more pretty
    Aaron Puchert via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Aug 17 17:36:18 PDT 2020
    
    
  
aaronpuchert added inline comments.
================
Comment at: llvm/cmake/modules/VersionFromVCS.cmake:49
+  else()
+    message(WARNING "Git not found. Version cannot be determined.")
   endif()
----------------
Why a `WARNING` and not just a `STATUS`? When `git` can't be found the probability is high that this isn't a repo checkout but just a build from tarballs.
Alternatively we could try to detect if a `.git` directory is there and not warn if it isn't.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82276/new/
https://reviews.llvm.org/D82276
    
    
More information about the llvm-commits
mailing list