Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#30120 closed enhancement (fixed)

NetCDF 4 c++ headers.

Reported by: SiggyF Owned by: tenomoto (Takeshi Enomoto)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: netcdf

Description (last modified by mf2k (Frank Schima))

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/

Change History (7)

comment:1 Changed 13 years ago by SiggyF

Cc: f.baart@… added

Cc Me!

comment:2 Changed 13 years ago by mf2k (Frank Schima)

Cc: f.baart@… removed
Description: modified (diff)
Keywords: c++ netcdf removed
Owner: changed from macports-tickets@… to takeshi@…
Version: 1.9.2

comment:3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

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

comment:4 Changed 13 years ago by SiggyF

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.

comment:5 Changed 13 years ago by tenomoto (Takeshi Enomoto)

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

comment:6 Changed 13 years ago by tenomoto (Takeshi Enomoto)

Resolution: fixed
Status: newclosed

comment:7 Changed 13 years ago by SiggyF

I tested it and it works fine, thanks.

Note: See TracTickets for help on using tickets.