fix: improve build system, packaging, and cross-platform support

Author: peterkchung

.github/workflows/release.yml

@@ -0,0 +1,40 @@
+name: Release
+
+on:
+  push:
+    tags: ['v*']
+
+jobs:
+  build-release:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install dependencies
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y cmake ninja-build
+
+      - name: Configure
+        run: >
+          cmake -B build -G Ninja
+          -DCMAKE_BUILD_TYPE=Release
+          -DSIMFORGE_BUILD_TESTS=ON
+
+      - name: Build
+        run: cmake --build build -j$(nproc)
+
+      - name: Test
+        run: cd build && ctest --output-on-failure
+
+      - name: Package
+        run: |
+          cmake --install build --prefix staging
+          tar czf simforge-linux-x86_64.tar.gz -C staging .
+
+      - name: Create Release
+        uses: softprops/action-gh-release@v2
+        with:
+          files: simforge-linux-x86_64.tar.gz
+          generate_release_notes: true

CMakeLists.txt

@@ -106,8 +106,10 @@ endif()
 if(SIMFORGE_USE_COACD)
     find_package(coacd QUIET)
     if(NOT coacd_FOUND)
+        if(CMAKE_VERSION VERSION_LESS "3.24")
+            message(FATAL_ERROR "SIMFORGE_USE_COACD with FetchContent requires CMake >= 3.24")
+        endif()
         message(STATUS "CoACD not found, fetching from source...")
-        cmake_minimum_required(VERSION 3.24)
         FetchContent_Declare(
             coacd
             GIT_REPOSITORY https://github.com/SarahWeiii/CoACD.git
@@ -141,3 +143,4 @@ if(SIMFORGE_BUILD_TESTS)
     enable_testing()
     add_subdirectory(tests)
 endif()
+

pyproject.toml

@@ -0,0 +1,15 @@
+[build-system]
+requires = ["scikit-build-core>=0.9"]
+build-backend = "scikit_build_core.build"
+
+[project]
+name = "simforge"
+version = "0.1.0"
+requires-python = ">=3.9"
+description = "Asset pipeline for robotics simulation"
+license = {file = "LICENSE"}
+
+[tool.scikit-build]
+cmake.build-type = "Release"
+cmake.args = ["-DSIMFORGE_BUILD_PYTHON=ON", "-DSIMFORGE_BUILD_TESTS=OFF"]
+wheel.packages = []

python/CMakeLists.txt

@@ -14,7 +14,15 @@ target_link_libraries(simforge_python PRIVATE simforge_core)
 
 # Ensure all symbols from the static library are included so that
 # adapter and stage registration works correctly.
-if(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
+if(APPLE)
+    target_link_options(simforge_python PRIVATE
+        "-Wl,-force_load,$<TARGET_FILE:simforge_core>"
+    )
+elseif(MSVC)
+    target_link_options(simforge_python PRIVATE
+        "/WHOLEARCHIVE:$<TARGET_FILE:simforge_core>"
+    )
+elseif(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
     target_link_options(simforge_python PRIVATE
         "LINKER:--whole-archive"
         $<TARGET_FILE:simforge_core>

src/CMakeLists.txt

@@ -30,12 +30,14 @@ target_include_directories(simforge_core PUBLIC
     ${CMAKE_BINARY_DIR}/include
 )
 
-target_link_libraries(simforge_core PUBLIC
-    yaml-cpp::yaml-cpp
-    spdlog::spdlog
-    nlohmann_json::nlohmann_json
-    tinyxml2::tinyxml2
-    meshoptimizer
+target_link_libraries(simforge_core
+    PUBLIC
+        yaml-cpp::yaml-cpp
+        nlohmann_json::nlohmann_json
+    PRIVATE
+        spdlog::spdlog
+        tinyxml2::tinyxml2
+        meshoptimizer
 )
 
 target_include_directories(simforge_core PRIVATE
@@ -57,8 +59,13 @@ add_executable(simforge cli/main.cpp)
 target_link_libraries(simforge PRIVATE
     simforge_core
     CLI11::CLI11
+    spdlog::spdlog
 )
 
-install(TARGETS simforge RUNTIME DESTINATION bin)
-install(TARGETS simforge_core ARCHIVE DESTINATION lib)
-install(DIRECTORY ${CMAKE_SOURCE_DIR}/include/ DESTINATION include)
+include(GNUInstallDirs)
+
+install(TARGETS simforge RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
+install(TARGETS simforge_core ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR})
+install(DIRECTORY ${CMAKE_SOURCE_DIR}/include/ DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
+install(FILES ${CMAKE_SOURCE_DIR}/data/materials.yaml
+        DESTINATION ${CMAKE_INSTALL_DATADIR}/simforge)

tests/CMakeLists.txt

@@ -25,6 +25,7 @@ add_executable(simforge_tests
 target_link_libraries(simforge_tests PRIVATE
     simforge_core
     Catch2::Catch2WithMain
+    spdlog::spdlog
     tinyxml2::tinyxml2
 )