Custom Query (986 matches)

The ROMS makefile is very powerful and complex. All the ROMS algorithms are distributed in several sub-directories. Each sub-directory has it own library, which is used during the linking stage. We need to account for the circularity of these objects in the libraries. For instance, the ROMS/Utility codes (libUTIL.a) uses modules located in other sub-directories. Therefore, in the makefile we have the libUTIL.a at the beginning and near the end of the libraries list token. For example, in nonlinear model algorithms we have:

  libraries  := $(SCRATCH_DIR)/libUTIL.a
                $(SCRATCH_DIR)/libNLM.a
                $(SCRATCH_DIR)/libNLM_bio.a
                $(SCRATCH_DIR)/libNLM_sed.a
                $(SCRATCH_DIR)/libANA.a
                $(SCRATCH_DIR)/libUTIL.a
                $(SCRATCH_DIR)/libMODS.a

It turns out that with some compiler versions and computers operating systems we also need circularity in the modules token for the ROMS/Nonlinear directory modules:

 modules  +=    ROMS/Nonlinear \
                ROMS/Nonlinear/Biology \
                ROMS/Nonlinear/Sediment \
                ROMS/Functionals \
                ROMS/Utility \
                ROMS/Nonlinear \
                ROMS/Modules

Many thanks to Özgür Gürses for ​reporting this problem and to Kate for providing a family.

Corrected couple of bug in wrt_his.F introduced recently with the output of eastward and northward velocity components. This only affect 2D applications (shallow water). The use of module uv_rotate2d was included in the wrong place of the CPP directives. We need to have instead:

      USE uv_rotate_mod, ONLY : uv_rotate2d
#ifdef SOLVE3D
      USE uv_rotate_mod, ONLY : uv_rotate3d
#endif

outside of another SOLVE3D loop directives.

Many thanks to Mark Hadfield for ​bringing this to my attention.

In ticket ​src:ticket:541, the makefile was updated to include cyclic dependencies on the modules target by including ROMS/Nonlinear twice. This solve the issues about the problems that a particular user was having and we were not able to reproduce. However, this added a lot of warnings during compilations:

ROMS/Nonlinear/Module.mk:13: warning: overriding commands for target `/Users/arango/ocean/repository/test/upwelling/Build/libNLM.a'
ROMS/Nonlinear/Module.mk:13: warning: ignoring old commands for target `/Users/arango/ocean/repository/test/upwelling/Build/libNLM.a'
...

Many users are panicking about this and always like the clean compilation. Therefore, we are reserving the cycling change in the makefile. We need to look for alternative solution.

I also updated all the input NetCDF file for BIO_TOY in the code repository directories Data/ROMS/* to agree with the files in the test repository:

https://www.myroms.org/svn/src/test

Many thanks to John Wilkin for bringing this to my attention.