Improve sdist-building process (#675)

* Exclude wheelhouse dir from sdists

So that we don't accidentally include a previously generated sdist when
creating a new one (i.e., finufft after generating cufinufft and vice
versa).

* Fix sdist-building scripts

Copy instead of move (makes script idempotent) and assume we're
executing from the root of the repository, not its parent.

Author: janden

python/cufinufft/pyproject.toml

@@ -66,7 +66,8 @@ sdist.exclude = [".*",
                  "tutorial",
                  "python/finufft",
                  "python/cufinufft/tests",
-                 "src"]
+                 "src",
+                 "wheelhouse"]
 sdist.include = ["src/cuda"]
 
 [tool.scikit-build.metadata.version]

python/finufft/pyproject.toml

@@ -66,7 +66,8 @@ sdist.exclude = [".*",
                  "tutorial",
                  "include/cufinufft*",
                  "python/cufinufft",
-                 "src/cuda"]
+                 "src/cuda",
+                 "wheelhouse"]
 
 [tool.scikit-build.metadata.version]
 # Instead of hardcoding the version here, extract it from the source files.

tools/cufinufft/build-sdist.sh

@@ -3,15 +3,15 @@
 set -e -x
 
 # Move pyproject.toml to root (otherwise no way to include C++ sources in sdist).
-mv finufft/python/cufinufft/pyproject.toml finufft/
+cp python/cufinufft/pyproject.toml .
 
 # Fix paths in pyproject.toml to reflect the new directory structure.
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.cmake.source-dir "."
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.wheel.packages --to-array "[\"python/cufinufft/cufinufft\"]"
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.metadata.version.input "python/cufinufft/cufinufft/__init__.py"
 
 # Package the sdist.
-python3 -m build --verbose --sdist --outdir wheelhouse finufft
+python3 -m build --verbose --sdist --outdir wheelhouse .

tools/finufft/build-sdist.sh

@@ -3,15 +3,15 @@
 set -e -x
 
 # Move pyproject.toml to root (otherwise no way to include C++ sources in sdist).
-mv finufft/python/finufft/pyproject.toml finufft/
+cp python/finufft/pyproject.toml .
 
 # Fix paths in pyproject.toml to reflect the new directory structure.
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.cmake.source-dir "."
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.wheel.packages --to-array "[\"python/finufft/finufft\"]"
-toml set --toml-path finufft/pyproject.toml \
+toml set --toml-path pyproject.toml \
          tool.scikit-build.metadata.version.input "python/finufft/finufft/__init__.py"
 
 # Package the sdist.
-python3 -m build --verbose --sdist --outdir wheelhouse finufft
+python3 -m build --verbose --sdist --outdir wheelhouse .