[mpas-developers] keeping around .f90 files

[Michael Duda]

Hi, Developers.

I have in mind a change to the makefiles that would keep
the pre-processed Fortran source files (i.e., the .f90
files) when building the MPAS code, and only remove them
when 'make clean' is run. The major benefit that I see 
would be in debugging code, where compile-time errors are
reported with respect to line numbers in .f90 files and
debuggers generally need the .f90 files when displaying 
source. What opinions do you all have on making this change
to the repository trunk? The changes are essentially those
made in revision 250 to a branch, and are summarized below.
Is this something that we'd like to do in the trunk as well?

Cheers,
Michael


Change all Makefiles to keep the pre-processed Fortran<br>
source files (i.e., the .f90 files), and to remove these <br>
files under the 'clean' target.<br>
<br>
<br>
M    src/core_hyd_atmos/Makefile<br>
M    src/core_sw/Makefile<br>
M    src/driver/Makefile<br>
M    src/driver_cam_interface/Makefile<br>
M    src/core_ocean/Makefile<br>
M    src/framework/Makefile<br>
M    src/operators/Makefile<br>
</p><hr noshade><pre><font color="gray">Modified: branches/mpas_cam_coupling/src/core_hyd_atmos/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/core_hyd_atmos/Makefile      2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/core_hyd_atmos/Makefile      2010-05-05 16:59:19 UTC (rev 250)
@@ -19,10 +19,9 @@
 mpas_interface.o: module_advection.o module_test_cases.o module_time_integration.o

 clean:
-       $(RM) *.o *.mod libdycore.a
+       $(RM) *.o *.mod *.f90 libdycore.a

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework -I../operators
-       $(RM) $*.f90

Modified: branches/mpas_cam_coupling/src/core_ocean/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/core_ocean/Makefile  2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/core_ocean/Makefile  2010-05-05 16:59:19 UTC (rev 250)
@@ -19,10 +19,9 @@
 mpas_interface.o: module_global_diagnostics.o module_test_cases.o module_time_integration.o

 clean:
-       $(RM) *.o *.mod libdycore.a
+       $(RM) *.o *.mod *.f90 libdycore.a

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework -I../operators
-       $(RM) $*.f90

Modified: branches/mpas_cam_coupling/src/core_sw/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/core_sw/Makefile     2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/core_sw/Makefile     2010-05-05 16:59:19 UTC (rev 250)
@@ -16,10 +16,9 @@
 mpas_interface.o: module_test_cases.o module_time_integration.o

 clean:
-       $(RM) *.o *.mod libdycore.a
+       $(RM) *.o *.mod *.f90 libdycore.a

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework -I../operators
-       $(RM) $*.f90

Modified: branches/mpas_cam_coupling/src/driver/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/driver/Makefile      2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/driver/Makefile      2010-05-05 16:59:19 UTC (rev 250)
@@ -10,10 +10,9 @@
 mpas.o: module_subdriver.o

 clean:
-       $(RM) *.o *.mod
+       $(RM) *.o *.mod *.f90

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework -I../core_$(CORE)
-       $(RM) $*.f90

Modified: branches/mpas_cam_coupling/src/driver_cam_interface/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/driver_cam_interface/Makefile        2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/driver_cam_interface/Makefile        2010-05-05 16:59:19 UTC (rev 250)
@@ -13,10 +13,9 @@
 test_driver.o: module_mpas_cam_interface.o

 clean:
-       $(RM) *.o *.mod
+       $(RM) *.o *.mod *.f90

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework -I../core_$(CORE)
-       $(RM) $*.f90

Modified: branches/mpas_cam_coupling/src/framework/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/framework/Makefile   2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/framework/Makefile   2010-05-05 16:59:19 UTC (rev 250)
@@ -35,13 +35,12 @@
 module_io_output.o: module_grid_types.o module_dmpar.o module_sort.o module_configure.o

 clean:
-       $(RM) *.o *.mod libframework.a
+       $(RM) *.o *.mod *.f90 libframework.a

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES)
-       $(RM) $*.f90

 .c.o:
        $(CC) $(CFLAGS) $(CPPFLAGS) $(CPPINCLUDES) -c $&lt;

Modified: branches/mpas_cam_coupling/src/operators/Makefile
===================================================================
--- branches/mpas_cam_coupling/src/operators/Makefile   2010-05-05 09:41:20 UTC (rev 249)
+++ branches/mpas_cam_coupling/src/operators/Makefile   2010-05-05 16:59:19 UTC (rev 250)
@@ -10,10 +10,9 @@
 module_vector_reconstruction.o:

 clean:
-       $(RM) *.o *.mod libops.a
+       $(RM) *.o *.mod *.f90 libops.a

 .F.o:
        $(RM) $@ $*.mod
        $(CPP) $(CPPFLAGS) $(CPPINCLUDES) $&lt; &gt; $*.f90
        $(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework
-       $(RM) $*.f90