[mpas-developers] keeping around .f90 files
Michael Duda
duda at ucar.edu
Wed May 5 11:11:41 MDT 2010
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) $< > $*.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) $< > $*.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) $< > $*.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) $< > $*.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) $< > $*.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) $< > $*.f90
$(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES)
- $(RM) $*.f90
.c.o:
$(CC) $(CFLAGS) $(CPPFLAGS) $(CPPINCLUDES) -c $<
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) $< > $*.f90
$(FC) $(FFLAGS) -c $*.f90 $(FCINCLUDES) -I../framework
- $(RM) $*.f90
More information about the mpas-developers
mailing list