Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hdf4 hdfi.h include goes away in the next release #70

Closed
schwehr opened this issue Feb 25, 2024 · 2 comments · Fixed by #72
Closed

hdf4 hdfi.h include goes away in the next release #70

schwehr opened this issue Feb 25, 2024 · 2 comments · Fixed by #72

Comments

@schwehr
Copy link
Contributor

schwehr commented Feb 25, 2024

The hdfi.h include here

#include "hdfi.h" /* declares int32, float32, etc */
goes away in hdf4 here: HDFGroup/hdf4@ec211cd

For my build, I was able to just remove that line and things built and tested fine.

Note that HDF4 has been undergoing a ton of cleanup recently. It's worth taking a careful look at pyhdf to coordinate with hdf4 before its next release

@derobins

@derobins
Copy link

HDF4 has historically had a very poor separation of public vs. private interfaces. In 4.3.0, which will release at the end of February, we've finally stopped deploying internal header files like hdfi.h. For people who are using the public API, as defined in the reference manual, user guide, etc., you should no longer need hdfi.h. FWIW, you didn't need to include that file at any time, since it was pulled in by hdf.h.

Once you've stopped including hdfi.h, ping us over at https://github.com/HDFGroup/hdf4 and we'll create a CI action to make sure pyhdf doesn't break as we change things.

@sebastic
Copy link
Contributor

sebastic commented Apr 4, 2024

HDF 4.3.0 has been released, with which pyhdf fails to build as reported in Debian Bug #1068384.

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 a pull request may close this issue.

3 participants