pybamm-team#2047 force windows build to not use python casadi

anmolg3 · May 9, 2022 · 3e1ab1f · 3e1ab1f
1 parent 889735f
commit 3e1ab1f
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/CMakeBuild.py b/CMakeBuild.py
@@ -42,10 +42,6 @@ def initialize_options(self):
         build_ext.initialize_options(self)
         self.suitesparse_root = None
         self.sundials_root = None
-        if system() == "Windows":
-            self.use_python_casadi = False
-        else:
-            self.use_python_casadi = True
 
     def finalize_options(self):
         build_ext.finalize_options(self)
@@ -82,9 +78,13 @@ def run(self):
         if not self.extensions:
             return
 
+        if system() == "Windows":
+            use_python_casadi = False
+        else:
+            use_python_casadi = True
         cmake_args = [
             "-DPYTHON_EXECUTABLE={}".format(sys.executable),
-            "-DUSE_PYTHON_CASADI={}".format(self.use_python_casadi),
+            "-DUSE_PYTHON_CASADI={}".format("TRUE" if use_python_casadi else "FALSE"),
         ]
         if self.suitesparse_root:
             cmake_args.append(