pybamm-team#3049 Remove improper CMake message, add for macOS

agriyakhetarpal · Sep 1, 2023 · 13ed52d · 13ed52d
1 parent 82197eb
commit 13ed52d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/test_on_push.yml b/.github/workflows/test_on_push.yml
@@ -76,7 +76,7 @@ jobs:
         run: |
           brew analytics off
           brew update
-          brew install graphviz openblas
+          brew install graphviz openblas cmake
 
       - name: Install Windows system dependencies
         if: matrix.os == 'windows-latest'
@@ -217,7 +217,7 @@ jobs:
         run: |
           brew analytics off
           brew update
-          brew install graphviz openblas
+          brew install graphviz openblas cmake
 
       - name: Install Windows system dependencies
         if: matrix.os == 'windows-latest'

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -65,7 +65,6 @@ if (CASADI_DIR)
   file(TO_CMAKE_PATH ${CASADI_DIR} CASADI_DIR)
   message("Found python casadi path: ${CASADI_DIR}")
 endif()
-message("Could not find python casadi path")
 
 if(${USE_PYTHON_CASADI})
   message("Trying to link against python casadi package")

diff --git a/pyproject.toml b/pyproject.toml
@@ -16,7 +16,7 @@ requires = [
     "setuptools",
     "wheel",
     "casadi>=3.6.0; platform_system!='Windows'",
-    "cmake; platform_system=='Linux'",
+    "cmake; platform_system!='Windows'",
     ]
 build-backend = "setuptools.build_meta"