mbox series

[bug#68171,0/2] Enable tests for python-h5py

Message ID 875y0einz5.fsf@troyfigiel.com
Headers show
Series Enable tests for python-h5py | expand

Message

Troy Figiel Dec. 31, 2023, 11:52 a.m. UTC 
While writing a different package definition, I noticed python-h5py does
not have any of its tests enabled. To make the tests pass, only two new
packages needed to be added to the native inputs.

A total of 130 dependent packages would need to be rebuilt. Although it
seems unlikely that only adding tests would break dependent builds, I
checked that locally all dependent packages that could be built before
this patch series, can still be built when these patches are applied (at
least for x86_64-linux).

Troy Figiel (2):
  gnu: python-h5py: Enable tests.
  gnu: python-h5py: Reformat with guix style.

 gnu/packages/python-xyz.scm | 30 +++++++++++++-----------------
 1 file changed, 13 insertions(+), 17 deletions(-)


base-commit: 2a242e86379ebddbdddf2927f26e5e27a98fc605

Comments

Mathieu Othacehe Jan. 9, 2024, 9:09 a.m. UTC | #1 
> While writing a different package definition, I noticed python-h5py does
> not have any of its tests enabled. To make the tests pass, only two new
> packages needed to be added to the native inputs.
>
> A total of 130 dependent packages would need to be rebuilt. Although it
> seems unlikely that only adding tests would break dependent builds, I
> checked that locally all dependent packages that could be built before
> this patch series, can still be built when these patches are applied (at
> least for x86_64-linux).

Agreed. Applied, thanks!

Mathieu