#669: Rewrite users of deprecated HDF5 C++ API -----------------------+---------------------------------------------------- Reporter: eschnett | Owner: Type: task | Status: reopened Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: -----------------------+---------------------------------------------------- Changes (by hinder):
* status: review => reopened
Comment:
I agree. Let's disable the use of the C++ API by default.
Looking at HDF5/configure.sh, it looks like we should just change
{{{ if [ "${HDF5_ENABLE_CXX:=yes}" = 'yes' ]; then }}} to {{{ if [ "${HDF5_ENABLE_CXX:=no}" = 'yes' ]; then }}}
However, the script looks like it might not be quite right, as later on this variable is used without a default being set. Additionally, the script does not treat "yes/no" in a caseless way like the other configuration scripts do.
The options supported by a configuration script are listed in configuration.ccl, but their default values are not - this is left up to the configuration script, so every configuration script has to contain programmatic code to manage defaults. I have amended #1042 to address this.
Changing status of this ticket from Review to "reopened" as there is no patch as yet.