Skip to content

Commit

Permalink
build: Use build-system pybind11 over Git submodule (#313)
Browse files Browse the repository at this point in the history
build: Add support for NumPy 2.0 wheels

* Update lower bound on build-system pybind11 to v2.12.0 to add support for
  compiling for NumPy 2.0.
   - c.f. https://github.com/pybind/pybind11/releases/tag/v2.12.0
* Remove pybind11 submodule to use build-system instead.

ci: List Python packages installed before testing

* Add an easy to parse log of the environment used for each test, which is useful
  for debugging differences between CI jobs seperated in time.
  • Loading branch information
matthewfeickert authored Sep 16, 2024
1 parent 5d42a0e commit 1a4f68c
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 6 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,9 @@ jobs:
echo $PATH
python -m pip install '.[test]' -v
- name: List installed Python packages
run: python -m pip list

- name: Test package
run: python -m pytest -vv -rs -Wd

Expand Down
3 changes: 0 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
[submodule "fastjet"]
path = extern/fastjet-core
url = https://gitlab.com/fastjet/fastjet.git
[submodule "pybind11"]
path = pybind11
url = https://github.com/pybind/pybind11.git
[submodule "fastjet-contrib"]
path = extern/fastjet-contrib
url = https://github.com/cms-externals/fastjet-contrib.git
1 change: 0 additions & 1 deletion MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
graft src
graft tests
graft extern
graft pybind11
global-exclude .git .gitmodules
include LICENSE README.md pyproject.toml setup.py setup.cfg patch_clustersequence.txt
exclude .cirrus.yml
1 change: 0 additions & 1 deletion pybind11
Submodule pybind11 deleted from 8a099e
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
requires = [
"setuptools>=42",
"setuptools_scm[toml]>=3.4",
"pybind11>=2.6.1",
"pybind11>=2.12.0",
]
build-backend = "setuptools.build_meta"

Expand Down

0 comments on commit 1a4f68c

Please sign in to comment.