Changes between Version 3 and Version 4 of ticket/0848/Review

Timestamp:

2011-10-03T14:47:30+02:00 (13 years ago)

Author:

sga

Comment:

--

ticket/0848/Review

@@ -34 +34 @@
 
 Testing Comments
+  I, the reviewer have not yet managed to get this code to compile.
+  Problems encountered so far:
+    the name of the C compiler (cc) is hard-coded into NEMOGCM/EXTERNAL/AGRIF/LIB/Makefile;
+    one cpp command I'm using doesn't like a space between '-I' and the include directory name;
+  Obscure build errors occur on two different systems I've tried so far, so I will persevere.
+  However, it seems a bit delicateand may therefore present support issues.
 
 Add specific testing comments here
@@ -77 +83 @@
 
   NEMOGCM/NEMO/NST_SRC/agrif_user.F90
+    Agrif_InitValues has a commented out call to Agrif_InitValues_cont_lim2
+    it would seem best either to delete this or add comments as to when it would be required.
+
   NEMOGCM/NEMO/NST_SRC/agrif_ice.F90
   NEMOGCM/NEMO/NST_SRC/agrif_opa_update.F90