[ET Trac] [Einstein Toolkit] #669: Rewrite users of deprecated HDF5 C++ API
Einstein Toolkit
trac-noreply at einsteintoolkit.org
Mon Sep 3 08:06:43 CDT 2012
#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.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/669#comment:6>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
More information about the Trac
mailing list