fhs / pyhdf

Python wrapper around the NCSA HDF version 4 library

Home Page:https://pypi.org/project/pyhdf/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

hdf4 hdfi.h include goes away in the next release

schwehr opened this issue · comments

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

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.

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