search

SyneRBI / SIRF

Main repository for the CCP SynerBI software
http://www.ccpsynerbi.ac.uk
Other
57 stars 29 forks source link

fix STIR HDF5 #1230

Closed KrisThielemans closed 5 months ago

KrisThielemans commented 5 months ago

We're using #ifdef HAVE_HDF5 but it should be STIR_built_with_HDF5. On my local machine, HAVE_HDF5 is defined, although I cannot find where, so it was enabled, but on GHA it probably isn't.

I'll test that first. Then replace.

KrisThielemans commented 5 months ago

HAVE_HDF5 is set in STIR/config.h. STIR_built_with_HDF5 is a CMake variable.

So this was all ok. Sorry