Custom Query (964 matches)

Following on from the problems with netcdf.mod & typesize.mod reported in #50 and the proposed solutions, I have come up with a simpler and (I think) better scheme for creating the scratch directory and populating it with the files it needs initially. Since the changes go beyond the scope ot ticket #50, I'm making this a separate ticket.

Modified versions of makefile and Compilers/UNICOS-sn-f90.mk are attached. Changes are:

• File names for netCDF F90 module files are specified via variables NETCDF_MODFILE and TYPESIZES_MODFILE. Default values (netcdf.mod & typesizes.mod) are specified in makefile; these are overridden if necessary in the machine-dependent make file (eg. NETCDF.mod & TYPESIZES.mod for the T3E).

• The netCDF module files are now prerequisites for $(SCRATCH_DIR)/MakeDepend, meaning they will be created at the start of the build process. Rules have been added to make them by copying the originals from $(NETCDF_INCDIR).

• Targets $(SCRATCH_DIR)/MakeDepend, $(SCRATCH_DIR)/$(NETCDF_MODFILE), $(SCRATCH_DIR)/$(TYPESIZES_MODFILE) include $(SCRATCH_DIR) as an order-only prerequisite. Order-only prerequisites are indicated by the pipe ("|") symbol and explained in ​http://www.gnu.org/software/make/manual/make.html#Prerequisite-Types. If $(SCRATCH_DIR) is made an ordinary prerequisite, then files like $(SCRATCH_DIR)/MakeDepend get re-made every time the directory changes, which is definitely undesirable.

I think the result is quite a bit cleaner than what it replaces.

With SVN version 79, there is a compile error when USE_DEBUG is defined when building with Compilers/CYGWIN-gfortran.mk. The problem is that FFLAGS is so long it exceeds 132 characters, the default in GFORTRAN when -ffree-form is specified.

I fixed the problem by adding the compile flag "-ffree-line-length-none" to this rule in CYGWIN-gfortran.mk for mod_strings:

Before:

$(SCRATCH_DIR)/mod_strings.o: FFLAGS += -ffree-form 

After:

$(SCRATCH_DIR)/mod_strings.o: FFLAGS += -ffree-form -ffree-line-length-none

Should this additional flag be added for the compilation of mod_ncparam as well?

I imagine the other gfortran .mk files should be modified as well, not just the gfortran one.

Thanks, Rich

As we know, ROMS will eventually move to using netCDF 4, allowing various enhancements like parallel I/O and compression. In the meantime, I have been experimenting with using netCDF 4 libraries as a direct substitute for netCDF 3 libraries. It turns out that the only change required in ROMS is to add some linker flags to link to the underlying HDF5 and Zlib libraries. For the current versions of netCDF 4 and HDF5 the flags required are:

-lnetcdf -lhdf5_hl -lhdf5 -lz

I suggest the following changes to support this:

• In makefile, add a variable USE_NETCDF4 (similar to the other USE_* variables).

• In the platform-specific makefiles, add an "ifdef USE_NETCDF4" block to set the flags.

Files are attached.

In future I expect USE_NETCDF4 will also be used to set preprocessor flags to support netCDF 4 features in the code.