[Debian-med-packaging] Bug#841959: Problems with your ismrmrd/1.3.2-4.1 NMU

[Ghislain Vaillant]

Hi Gilles,

On 01/11/16 17:50, Gilles Filippini wrote:
>> Could you give me more insight of the compatibility problem with HDF5 1.10?
>
> The most important change in the HDF5 1.10 release is that the hid_t
> type was changed from 'int' to 'long long' [1]. Assuming that hid_t is
> int isn't true anymore. Thus the change to the ISMRMRD_Dataset
> structure, because the fileid field is actually used as an hid_t. See
> for example the ismrmrd_open_dataset() function in libsrc/dataset.c:
>
> int ismrmrd_open_dataset(ISMRMRD_Dataset *dset, const bool
> create_if_needed) {
>     /* TODO add a mode for clobbering the dataset if it exists. */
>     hid_t fileid;
>     ...
>     fileid = H5Fopen(dset->filename, H5F_ACC_RDWR, H5P_DEFAULT);
>
>     if (fileid > 0) {
>         dset->fileid = fileid;
>     }
>     ...

I submitted your patch to a PR upstream for review:

   https://github.com/ismrmrd/ismrmrd/pull/74

Since there were no DEP3 header attached to the patch, I used your 
Debian name and email for the commit authorship. I hope this is ok.

Ghis