[ESI Runtime] Fix python build

llvm · Jun 24, 2024 · 91dd18e · 91dd18e
1 parent 34c73c3
commit 91dd18e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/Dialect/ESI/runtime/cosim/CMakeLists.txt b/lib/Dialect/ESI/runtime/cosim/CMakeLists.txt
@@ -55,7 +55,7 @@ protobuf_generate(
 
 include_directories("${CMAKE_CURRENT_SOURCE_DIR}/include")
 
-add_library(CosimRpcServer
+add_library(CosimRpcServer SHARED
   lib/RpcServer.cpp
 )
 target_link_libraries(CosimRpcServer

diff --git a/lib/Dialect/ESI/runtime/setup.py b/lib/Dialect/ESI/runtime/setup.py
@@ -11,7 +11,7 @@
 #
 # It is recommended to build with Ninja and ccache. To do so, set environment
 # variables by prefixing to above invocations:
-#   CMAKE_GENERATOR=Ninja CMAKE_CXX_COMPILER=clang++
+#   CC=clang CXX=clang++
 #
 # On CIs, it is often advantageous to re-use/control the CMake build directory.
 # This can be set with the PYCDE_CMAKE_BUILD_DIR env var.
@@ -67,9 +67,9 @@ def run(self):
     cfg = "Release"
     cmake_args = [
         "-DCMAKE_BUILD_TYPE={}".format(cfg),  # not used on MSVC, but no harm
-        "-DCAPNP_PATH={}".format(os.getenv("CAPNP_PATH")),
         "-DPython3_EXECUTABLE={}".format(sys.executable.replace("\\", "/")),
         "-DWHEEL_BUILD=ON",
+        "-GNinja",
     ]
     cxx = os.getenv("CXX")
     if cxx is not None: