Correct cmake path hint

flatironinstitute · Feb 13, 2024 · 7bf919c · 7bf919c
1 parent e6cb776
commit 7bf919c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -37,6 +37,7 @@ jobs:
         libclang-15-dev
         libblas-dev
         libomp-15-dev
+        clang-format-15
         python3
         python3-dev
         python3-numpy

diff --git a/share/cmake/Modules/clang_detection.cmake b/share/cmake/Modules/clang_detection.cmake
@@ -10,7 +10,9 @@ set(LLVM_ROOT_DIR "${LLVM_ROOT_DIR_DEFAULT}") #CACHE PATH "LLVM installation dir
 
 # Add the location of ClangConfig.cmake to CMake search paths (so that
 # find_package can locate it)
-list(PREPEND CMAKE_PREFIX_PATH "${LLVM_ROOT_DIR}/lib/cmake/clang/")
+list(PREPEND CMAKE_PREFIX_PATH "${LLVM_ROOT_DIR}/lib/cmake/clang/")  # For OS X
+list(PREPEND CMAKE_PREFIX_PATH "${LLVM_ROOT_DIR}/cmake/clang/")      # For Ubuntu
+# We must find the proper LLVM config
 
 find_package(LLVM REQUIRED CONFIG)
 find_package(Clang REQUIRED CONFIG)