Opened 8 years ago

Closed 8 years ago

#49810 closed defect (fixed)

hdf5 @1.8.16 broken configure with thread safe variant

Reported by: dolfimeth@… Owned by: mamoll (Mark Moll)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: Cc: tenomoto (Takeshi Enomoto)
Port: hdf5

Description

I think that the latest hdf5 port update broke (at least) the threadsafe variant.

This is the output of the log

checking for thread safe support... configure: error: The thread-safe library is incompatible with the high-level library. --disable-hl can be used to prevent building the high-level library (recommended). Alternatively, --enable-unsupported will allow building the high-level library, though this configuration is not supported by The HDF Group.

As advised by the message I added --disable-hl in the Portfile, attached a patch.

The problem is that this will afterwards break netcdf, because the netcdf4 port (enabled by default) requires the high-level interface that just got disabled.

My solution to that was:

sudo port deactivate -f netcdf@4.3.3.1_4+dap+netcdf4
sudo port install netcdf @4.3.3.1_4+dap-netcdf4

Attachments (2)

hdf5.patch (163 bytes) - added by dolfimeth@… 8 years ago.
hdf5_hl_variant.patch (687 bytes) - added by dolfimeth@… 8 years ago.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by dolfimeth@…

Attachment: hdf5.patch added

comment:1 Changed 8 years ago by mf2k (Frank Schima)

Cc: takeshi@… added
Keywords: hdf5 netcdf threadsafe removed
Owner: changed from macports-tickets@… to mmoll@…
Port: netcdf removed

In the future, please Cc the port maintainers (port info --maintainers hdf5 netcdf), if any.

comment:2 Changed 8 years ago by mamoll (Mark Moll)

Resolution: fixed
Status: newclosed

Committed in r142879.

Changed 8 years ago by dolfimeth@…

Attachment: hdf5_hl_variant.patch added

comment:3 Changed 8 years ago by dolfimeth@…

I realized later that a better approach might be introducing an hl variant which is enabled by default and if the user explicitly enables both it will fall back in the experimental section with --enable-unsupported.

Attached is a new patch doing that.

comment:4 Changed 8 years ago by dolfimeth@…

Resolution: fixed
Status: closedreopened

comment:5 Changed 8 years ago by mamoll (Mark Moll)

Resolution: fixed
Status: reopenedclosed

Committed in r142935.

Note: See TracTickets for help on using tickets.