Opened 17 years ago

Closed 17 years ago

#27 closed bug (WorksForMe)

change to mod_ncparam.F broke my ROMS_APPLICATION ?= WAKE case

Reported by: rsignell Owned by: arango
Priority: critical Milestone: Release ROMS/TOMS 3.1
Component: Nonlinear Version: 3.1
Keywords: Cc:

Description

I've been running with the makefile

ROMS_APPLICATION ?= WAKE

and then defining MY_HEADER_DIR ?= /home/rsignell/p/wake/roms_sed/run004 MY_ANALYTICAL_DIR ?= /home/rsignell/p/wake/roms_sed/run004

in which my wake.h and ana_xxx.h files are kept.

The change from ROMS_HEADER to HEADER: https://www.myroms.org/projects/src/browser/trunk/ROMS/Modules/mod_ncparam.F?rev=53

broke this case. Reverting back to ROMS_HEADER worked.

Change History (1)

comment:1 by arango, 17 years ago

Resolution: worksforme
Status: newclosed

Your are using the wrong makefile. Please update. The application header file now needs a full path. Notice that we have the following logic in the makefile:

 ifdef MY_HEADER_DIR
  CPPFLAGS += -D'ROMS_HEADER="$(MY_HEADER_DIR)/$(HEADER)"'
 else
  CPPFLAGS += -D'ROMS_HEADER="$(HEADER)"'
 endif

This allows the users to modify the provided test cases in a different directory. For example, you can have you own copy of upwelling.h in a different directory than ROMS/Include. This is one of the strategies that users follow when learning ROMS.

Note: See TracTickets for help on using tickets.