fix sdist source inclusion, enable pypi test builld

Author: reneSchm

.github/workflows/pypi.yml

@@ -6,7 +6,7 @@ on:
       - published
   ### Uncomment the "push:" together with the "repository-url:" field of action pypa/gh-action-pypi-publish
   ### to make a test release. Building this is rather expensive so do not merge this uncommented!
-  #push:
+  push:
   ### @Reviewer: MAKE SURE THE LINE ABOVE IS COMMENTED OUT BEFORE MERGE! ###
 
 jobs:
@@ -73,4 +73,4 @@ jobs:
         with:
           skip-existing: true
           ### To test uploads to TestPyPI in a PR, uncomment the following, as well as the "push:" line at the top:
-          #repository-url: https://test.pypi.org/legacy/
+          repository-url: https://test.pypi.org/legacy/

pycode/memilio-simulation/CMakeLists.txt

@@ -1,6 +1,16 @@
 cmake_minimum_required(VERSION 3.14)
 project(memilio-python)
 
+# copy cpp files for building sdist's, then exit immediately
+# Note: ${SKBUILD_STATE} is "wheel" while building from an sdist
+if(SKBUILD_STATE STREQUAL "sdist")
+    message(STATUS "Copying current C++ source tree for distribution")
+    file(COPY ${CMAKE_CURRENT_SOURCE_DIR}/../../cpp DESTINATION ${CMAKE_CURRENT_SOURCE_DIR}
+         PATTERN "build" EXCLUDE
+         PATTERN ".cache" EXCLUDE)
+    return()
+endif()
+
 set(CMAKE_CXX_STANDARD "20")
 set(CMAKE_CXX_STANDARD_REQUIRED "20")
 
@@ -55,7 +65,14 @@ else()
 endif()
 
 # add in C++ library
-add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/../../cpp ${CMAKE_CURRENT_BINARY_DIR}/cpp EXCLUDE_FROM_ALL)
+if(EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/cpp)
+    # this is a sdist build, hence we use a copy C++ source tree, so that it is not out-of-tree wrt. pyproject.toml
+    message(WARNING "Using ${CMAKE_CURRENT_SOURCE_DIR}/cpp as C++ source tree. This is intentional for sdist builds.")
+    add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/cpp ${CMAKE_CURRENT_BINARY_DIR}/cpp EXCLUDE_FROM_ALL)
+else()
+    # normal include using the main project tree
+    add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}../../cpp ${CMAKE_CURRENT_BINARY_DIR}/cpp EXCLUDE_FROM_ALL)
+endif()
 
 # a list of all "LINKED_LIBRARIES" that are given to add_pymio_module. will contain duplicates
 # used for wheel installation

pycode/memilio-simulation/pyproject.toml

@@ -9,11 +9,11 @@ build-backend = "scikit_build_core.build"
 
 [project]
 name = "memilio-simulation"
-dynamic = ["version"] 
+dynamic = ["version"]
 description = "Part of MEmilio project, Python bindings to the C++ libraries that contain the models and simulations."
 readme = "README.md"
 requires-python = ">=3.8"
-license = "Apache-2.0" 
+license = "Apache-2.0"
 authors = [{ name = "MEmilio Team" }]
 maintainers = [
   { email = "martin.kuehn@dlr.de" }
@@ -56,10 +56,12 @@ cmake.version = ">=3.13"
 cmake.args = ["-DMEMILIO_BUILD_SHARED_LIBS:BOOL=ON"]
 wheel.packages = ["memilio"]
 wheel.install-dir = "memilio/simulation"
+sdist.cmake = true
+sdist.exclude = ["dist"]
 build-dir = "../build/memilio-simulation"
 metadata.version.provider = "scikit_build_core.metadata.setuptools_scm"
 
-[tool.setuptools_scm]  
+[tool.setuptools_scm]
 local_scheme = "no-local-version"
 # point scm to the git project root
 root = "../.."