NetCDF 4 c++ headers.

[f.baart@…]

Starting from netCDF 4.1.1 netcdf has a new c++ interface. This can be build using:

 ./configure --enable-cxx-4 [plus other options]

Can you add this option to the netCDF portfile. I think the following should be added:

variant cxx-4 description {enable netCDF-4 c++ interface} {
    configure.args-delete   --disable-cxx-4
    configure.args-append   --enable-cxx-4
}

Or maybe it makes more sense to add it to the netcdf4 variant. For details about the new c++ interface, see: ​http://www.unidata.ucar.edu/software/netcdf/docs/cxx4/

[f.baart@…]

Cc Me!

[ryandesign@…]

Is there any reason not to enable this all the time? Having fewer variants is better.

[f.baart@…]

I checked a build with and without --enable-cxx-4 and the old header files are always there, so it's not likely to break anything build against the old header files. It is not useful if netcdf-4 is not used, because in the configure.ac enable-cxx is disabled if netcdf-4 is not enabled:

if test "x$enable_netcdf_4" = xno; then
   enable_cxx_4=no
fi

So the --enable-cxx-4 flag can be set when the netcdf-4 variant is used. No need for an extra variant.

[takeshi@…]

I included --enable-cxx-4 in netcdf4 variant. Committed in r80405.

[f.baart@…]

I tested it and it works fine, thanks.