[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