55 pdi packages fail to build on jammy trixie

[jmorice91]

!!!INSERT YOUR DESCRIPTION HERE!!!

List of things to check before making a PR

Before merging your code, please check the following:

• [ x] you have added a line describing your changes to the Changelog;
• [ x] you have added unit tests for any new or improved feature;
• [ x] In case you updated dependencies, you have checked pdi/docs/CheckList.md
• you have checked your code format:
  • [ x] you have checked that you respect all conventions specified in CONTRIBUTING.md;
  • [ x] you have checked that the indentation and formatting conforms to the .clang-format;
  • [ x] you have documented with doxygen any new or changed function / class;
• you have correctly updated the copyright headers:
  • [ x] your institution is in the copyright header of every file you (substantially) modified;
  • [ x] you have checked that the end-year of the copyright there is the current one;
• you have updated the AUTHORS file:
  • [ x] you have added yourself to the AUTHORS file;
  • [ x] if this is a new contribution, you have added it to the AUTHORS file;
• you have added everything to the user documentation:
  • [ x] any new CMake configuration option;
  • [ x] any change in the yaml config;
  • [ x] any change to the public or plugin API;
  • [ x] any other new or changed user-facing feature;
  • [ x] any change to the dependencies;
• you have correctly linked your MR to one or more issues:
  • [ x] your MR solves an identified issue;
  • [ x] your commit contain the Fix #issue keyword to autoclose the issue when merged.

[benoitmartin88]

I'm only asking for a tiny change :)

[jbigot]

This is a great fix! Can you rebase it on top of v1.8 so that we can publish a bugfix release. Could you also add an issue about adding a test for sequential hdf5

[jbigot]

you have updated the AUTHORS file:

[ x] you have added yourself to the AUTHORS file;
[ x] if this is a new contribution, you have added it to the AUTHORS file

Have you?

[benoitmartin88]

LGTM

[jmorice91]

This is a great fix! Can you rebase it on top of v1.8 so that we can publish a bugfix release. Could you also add an issue about adding a test for sequential hdf5

The merge of "#502" introduce this bug in the main repo. This pull request is not merged yet in v.1.8.1. So there is no bugfix release.

For adding tests on sequential hdf5, the issue #530 is created

[jbigot]

Sorry, my bad, I didn't realize. Then if this is a fix for something that didn't make it to any release, it shouldn't be listed in the changelog. This is transparent for our users

[jbigot]

However, it would be nice to add the contribution to the authors file.

[jmorice91]

I added a comment for this issue in AUTHORS

[jbigot]

Perfect, thanks 🙏