Add support to build MOM5 with an external FMS and Generic Tracers using Spack

[harshula]

• Ideally we should use a better build system. However, this appears to be the
  least intrusive and simple way to introduce this feature.

[harshula]

FYI, I decided not to include this commit in this PR for the sake of simplicity:

commit a830e525565902b89faeb4d5fde65d187f7a3139 (dev_backup)
Author: Harshula Jayasuriya <harshula.jayasuriya@anu.edu.au>
Date:   Fri Nov 22 17:37:37 2024 +1100

    Ready for review

diff --git a/exp/MOM_compile.csh b/exp/MOM_compile.csh
index d72f477..8de451b 100755
--- a/exp/MOM_compile.csh
+++ b/exp/MOM_compile.csh
@@ -158,12 +158,14 @@ endif
 
 if ( ! $?SPACK_GTRACERS_EXTERNAL ) then
     echo "Building type=$type with internal FMS"
+    # Build FMS.
     source ./FMS_compile.csh
-    set includes = "-I$code_dir/shared/include -I$executable:h:h/lib_FMS -I$executable:h:h/lib_ocean"
+    set includes = "-I$executable:h:h/lib_FMS"
 else
     echo "Building type=$type with external FMS"
-    set includes = "-I$code_dir/shared/include -I$executable:h:h/lib_ocean"
+    set includes = ""
 endif
+set includes = "$includes -I$code_dir/shared/include -I$executable:h:h/lib_ocean"
 
 if ( $cosima_version ) then
     set includes = "$includes -I$executable:h:h/lib_version/"
diff --git a/exp/ocean_compile.csh b/exp/ocean_compile.csh
index 5eb000a..58f7887 100644
--- a/exp/ocean_compile.csh
+++ b/exp/ocean_compile.csh
@@ -9,13 +9,10 @@ if( $type == ACCESS-OM || $type == ACCESS-CM || $type == ACCESS-OM-BGC || $type
     if( $type ==  ACCESS-OM-BGC ) then
         set srcList = ( $srcList mom5/ocean_csiro_bgc )
     else if ( $type ==  ACCESS-OM || $type == ACCESS-ESM ) then
-
+        set srcList = ( $srcList mom5/ocean_bgc )
         if ( ! $?SPACK_GTRACERS_EXTERNAL ) then
-            set srcList = ( $srcList mom5/ocean_bgc access/generic_tracers/generic_tracers access/generic_tracers/mocsy/src )
-        else
-            set srcList = ( $srcList mom5/ocean_bgc )
+            set srcList = ( $srcList access/generic_tracers/generic_tracers access/generic_tracers/mocsy/src )
         endif
-
     endif
     mkdir -p $executable:h:h/$type/$lib_name
     cd $executable:h:h/$type/$lib_name

[aidanheerdegen]

Line 150 sets lib_include_dirs which is used in all the component build scripts

set lib_include_dirs = "$root/include $code_dir/shared/include $code_dir/shared/mpp/include"

I think it should also be bracketed with logic to test for external FMS.

[dougiesquire]

What's holding up getting this merged @harshula? Is it just waiting for final review/approval?

[harshula]

Hi @dougiesquire , Originally it was a hack in-order to test the generic tracers. I'm not happy with the changes that I made. Can you and @aidanheerdegen work on this and come up with a neater solution? I'll drop my commits in the development branch.