#1158: ExternalLibraries/HDF5 should change the defaults for HDF5_ENABLE_CXX and HDF5_ENABLE_FORTRAN to "no". ------------------------------------+--------------------------------------- Reporter: knarf | Owner: Type: defect | Status: confirmed Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: | Keywords: ------------------------------------+---------------------------------------
Comment (by knarf):
Replying to [comment:22 eschnett]:
In Frank's example, the cost is 140 seconds for a full build, which is a
rare occasion.
140 seconds does not sound that much, but that is on a 16 core workstation. If looked at it in another light, it is a 50% increase in build time for the whole ET.
For example, MacPorts provides HDF5 with Fortran bindings if the
+Fortran variant is chosen; Ubuntu provides Fortran bindings by default, etc. I also assume that most people only install HDF5 on their laptops/workstations because the ET install instructions say so. Couldn't we just ask for the Fortran bindings as well? Which systems don't provide a system HDF5 with Fortran?
Actually, the Debian hdf5 packages also support Fortran (now, they didn't use to). The Cactus build triggered because of the missing C++ interface. If the latter requirement is dropped, including Fortran should be good there and does in fact use the system libraries.
So, I think we arrived at a viable solution for the default options in simfactory: requiring HDF5 interfaces for Fortran, but not C++ - any objections?