Opened 10 years ago

Closed 10 years ago

#399 closed enhancement (fixed)

Remove references to HDF5 f90 libs from configure.ac file

Reported by: Ian Culverwell Owned by: Ian Culverwell
Priority: normal Milestone: 8.0
Component: ropp_io Version: 7.1
Keywords: build Cc:

Description

ROPP8.0 beta reviewer Dave Offiler (UKMO) recommends:

b) ROPP_IO: remove references to the two HDF5 Fortran libraries from 
the configure.ac file (test CM_CHECK_LIB(hdf5, nf_inq_ncid,...) 
line) and regenerate the configure script.

Very uneasy about changing something that works OK. Probably won't fix.

Change history (4)

comment:1 by Ian Culverwell, 10 years ago

Component: ROPP (all)ropp_io
Milestone: 9.08.0

Bring forward to ROPP8.0.

Make change -- replacing

CM_CHECK_LIB(hdf5, nf_inq_ncid, -lnetcdff -lnetcdf -lhdf5hl_fortran -lhdf5_hl -lhdf5_fortran -lhdf5 -lz)

by

CM_CHECK_LIB(hdf5, nf_inq_ncid, -lnetcdff -lnetcdf -lhdf5_hl -lhdf5 -lz)

in Sec 7.3 of ropp_io/config.ac -- at r4444.

comment:2 by Ian Culverwell, 10 years ago

Need to do likewise for the ropp_pp, ropp_fm and ropp_1dvar configure.ac files, which also refer to HDF5 fortran libs. (Why do they refer to HDF5 at all? Future-proofing, I suppose.)

Done (to trunk) at r4453.

comment:3 by Ian Culverwell, 10 years ago

Actually, this affects the configure files as well, so these have been updated (on trunk) at r4454.

comment:4 by Ian Culverwell, 10 years ago

Resolution: fixed
Status: newclosed

Works OK on RHEL6, HPC and Cygwin, so closing ticket.

Note: See TracTickets for help on using tickets.