Skip to content

build: Prefer binary installs in cibuildwheel options#67

Draft
matthewfeickert wants to merge 1 commit intothaler-lab:masterfrom
matthewfeickert:build/update-cibuildwheel-options
Draft

build: Prefer binary installs in cibuildwheel options#67
matthewfeickert wants to merge 1 commit intothaler-lab:masterfrom
matthewfeickert:build/update-cibuildwheel-options

Conversation

@matthewfeickert
Copy link
Contributor

Resolves #66

  • Enable PIP_PREFER_BINARY to ensure wheels are used if possible.
  • Use 'build[uv]' as build frontend to speed up installs when possible.
  • Revise the test command to explicitly target the tests/ directory.

* Enable PIP_PREFER_BINARY to ensure wheels are used if possible.
* Use 'build[uv]' as build frontend to speed up installs when possible.
* Revise the test command to explicitly target the tests/ directory.
@matthewfeickert matthewfeickert force-pushed the build/update-cibuildwheel-options branch from 87fc85c to 1c67f56 Compare November 11, 2024 22:01
@matthewfeickert
Copy link
Contributor Author

matthewfeickert commented Nov 12, 2024

Ah, so the reason why I could get wheels for h5py for Python 3.7 in #66 (comment) but @j-s-ashley saw cibuildwheel building them from the sdist and then failing from lack of libhdf5 was because the failing builds were from cp38-manylinux_i686 and there are no h5py wheels for manylinux_i686 so it has to build them there.

So the question is it is possible to do something like

[tool.cibuildwheel.linux]
before-all = "yum install -y hdf5"

but only on certain architectures? I think the answer is no given that the cibuildwheel CIBW_BEFORE_ALL option will

Execute a shell command on the build system before any wheels are built.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EnergyFlow HDF5 dependency causes build test fails on Windows and Linux

1 participant