[m-rev.] for review: remove RM_C

Simon Taylor stayl at cs.mu.OZ.AU
Fri May 18 20:28:37 AEST 2001


Estimated hours taken: 0.5

Remove the RM_C mmake variable, which controlled whether the intermediate
`.c' files were removed (now they never are).

The implementation was buggy (it didn't work with parallel makes), and 
made it difficult to avoid always recompiling the `.c' file with smart
recompilation.

Mmake.common.in:
configure.in:
README.AIX:
bindist/bindist.build_vars.in:
bindist/bindist.configure.in:
scripts/Mmake.vars.in:
scripts/Mmake.rules:
compiler/modules.m:
tools/bootcheck:
tools/binary_step:
*/Mmakefile:
	Remove references to RM_C, DEFAULT_RM_C and LIBRARY_RM_C.

compiler/modules.m:
	The `.o' and `.pic_o' file now depends only on the `.c' file,
	not on everthing the `.c' file depends on. The extra dependencies
	were only needed because the intermediate `.c' file could
	be removed by RM_C. This change is needed to avoid recompiling
	unchanged `.c' files with smart recompilation.


Index: Mmake.common.in
===================================================================
RCS file: /home/mercury1/repository/mercury/Mmake.common.in,v
retrieving revision 1.53
diff -u -u -r1.53 Mmake.common.in
--- Mmake.common.in	2001/02/13 08:28:23	1.53
+++ Mmake.common.in	2001/05/18 07:30:09
@@ -51,13 +51,6 @@
 # just reuse the .o files.
 EXT_FOR_PIC_OBJECTS	= @EXT_FOR_PIC_OBJECTS@
 
-# Set LIBRARY_RM_C as `:' if EXT_FOR_PIC_OBJECTS is `pic_o', or `rm'
-# otherwise.  This is used to ensure that in cases when we're building
-# both `.o' and `.pic_o' files, we don't remove the intermediate `.c' files.
-# This avoids compiling things to C twice, and also avoids problems when
-# bootstrapping from the source distribution.
-LIBRARY_RM_C		= @LIBRARY_RM_C@
-
 # This specifies the path to the so_locations file (or its equivalent),
 # which is used by the linker to help it to map different shared objects
 # to different virtual memory addresses at static link time, so as to avoid 
Index: README.AIX
===================================================================
RCS file: /home/mercury1/repository/mercury/README.AIX,v
retrieving revision 1.6
diff -u -u -r1.6 README.AIX
--- README.AIX	2001/02/28 13:13:17	1.6
+++ README.AIX	2001/05/18 10:17:13
@@ -21,15 +21,8 @@
     This works around a fixed limit on the size of the TOC (Table of Contents)
     in the RS/6000 object file format.
 
-2.  Also add the following line to Mmake.params:
+2.  Follow the instructions in the INSTALL file.
 
-    	RM_C=:
-
-    This prevents the intermediate C files from being removed,
-    which will make it easier to debug things later.
-
-3.  Follow the instructions in the INSTALL file.
-
 The last version of Mercury which was known to work on AIX was
 Mercury 0.6 -- actually a version just prior to 0.6, namely
 the 30Apr1996 development snapshot (the sources for this version
@@ -39,7 +32,7 @@
 an error message from the assembler, so the following additional
 step was needed.
 
-4. Change directory to the `compiler' directory, and run the command
+3. Change directory to the `compiler' directory, and run the command
 
 	sh ../scripts/rs6000_hack
 
Index: configure.in
===================================================================
RCS file: /home/mercury1/repository/mercury/configure.in,v
retrieving revision 1.254
diff -u -u -r1.254 configure.in
--- configure.in	2001/05/17 09:05:34	1.254
+++ configure.in	2001/05/18 06:31:26
@@ -2412,33 +2412,6 @@
 		;;
 esac
 
-#
-# This sets the default value of RM_C, which should be either `rm' to
-# remove the intermediate C files, or `:' to keep them.
-#
-# If RM_C is set to `rm', then unfortunately Mmake needs to invoke
-# make recursively, and this causes several problems:
-#	(1) it causes performance problems, particularly on MS-Windows
-#	(2) it can break parallel makes, because there are race
-#           conditions involved when you invoke make recursively
-#	    in a parallel make
-#
-# Therefore by default we currently keep the intermediate C files.
-#
-DEFAULT_RM_C=:
-
-case $EXT_FOR_PIC_OBJECTS in
-	o)
-		LIBRARY_RM_C=$DEFAULT_RM_C
-		;;
-	*)
-		# If we're going to be building both `.pic_o' files
-		# and `.o' files, then we should not remove the generated `.c'
-		# files in the library directory after building the `.o' files.
-		LIBRARY_RM_C=:
-		;;
-esac
-
 #-----------------------------------------------------------------------------#
 
 case "$host" in
@@ -2467,8 +2440,6 @@
 AC_SUBST(USE_DLLS)
 AC_SUBST(SHARED_LIBS)
 AC_SUBST(SHARED_LIBS_SH)
-AC_SUBST(DEFAULT_RM_C)
-AC_SUBST(LIBRARY_RM_C)
 if test $USE_DLLS = "yes"; then
 	AC_DEFINE(USE_DLLS)
 fi
Index: bindist/bindist.build_vars.in
===================================================================
RCS file: /home/mercury1/repository/mercury/bindist/bindist.build_vars.in,v
retrieving revision 1.16
diff -u -u -r1.16 bindist.build_vars.in
--- bindist/bindist.build_vars.in	2000/09/24 08:52:05	1.16
+++ bindist/bindist.build_vars.in	2001/05/18 07:30:23
@@ -33,7 +33,6 @@
 SHARED_LIBS='@SHARED_LIBS@'
 SHARED_LIBS_SH='@SHARED_LIBS_SH@'
 HAVE_DELAY_SLOT="@HAVE_DELAY_SLOT@"
-DEFAULT_RM_C="@DEFAULT_RM_C@"
 ERROR_UNDEFINED="@ERROR_UNDEFINED@"
 ALLOW_UNDEFINED="@ALLOW_UNDEFINED@"
 DL_LIBRARY="@DL_LIBRARY@"
Index: bindist/bindist.configure.in
===================================================================
RCS file: /home/mercury1/repository/mercury/bindist/bindist.configure.in,v
retrieving revision 1.31
diff -u -u -r1.31 bindist.configure.in
--- bindist/bindist.configure.in	2001/03/15 08:08:13	1.31
+++ bindist/bindist.configure.in	2001/05/18 07:30:32
@@ -61,7 +61,6 @@
 AC_SUBST(EXT_FOR_SHARED_LIB)
 AC_SUBST(SHARED_LIBS)
 AC_SUBST(HAVE_DELAY_SLOT)
-AC_SUBST(DEFAULT_RM_C)
 AC_SUBST(ALLOW_UNDEFINED)
 AC_SUBST(ERROR_UNDEFINED)
 AC_SUBST(DL_LIBRARY)
Index: browser/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/browser/Mmakefile,v
retrieving revision 1.15
diff -u -u -r1.15 Mmakefile
--- browser/Mmakefile	2001/03/16 01:40:36	1.15
+++ browser/Mmakefile	2001/05/18 10:15:05
@@ -24,12 +24,6 @@
 
 #-----------------------------------------------------------------------------#
 
-# If we're going to generate both `.o' files and `.pic_o' files, then
-# don't remove the intermediate `.c' files.
-RM_C	=	$(LIBRARY_RM_C)
-
-#-----------------------------------------------------------------------------#
-
 # Specify which compilers to use to compile the library.
 # Don't change these without good reason - if you want to
 # do a temporary change, change ../Mmake.params.
Index: compiler/modules.m
===================================================================
RCS file: /home/mercury1/repository/mercury/compiler/modules.m,v
retrieving revision 1.164
diff -u -u -r1.164 modules.m
--- compiler/modules.m	2001/05/12 05:03:34	1.164
+++ compiler/modules.m	2001/05/18 08:19:02
@@ -1710,8 +1710,6 @@
 			AsmFileName, " ",
 			PicAsmFileName, " ",
 			ErrFileName, " ",
-			PicObjFileName, " ",
-			ObjFileName, " ",
 			SplitObjPattern, " ",
 			RLOFileName, " ",
 			ILFileName, " : ",
@@ -1745,8 +1743,6 @@
 				CFileName, " ",
 				TransOptDateFileName, " ",
 				ErrFileName, " ", 
-				PicObjFileName, " ",
-				ObjFileName, " ",
 				SplitObjPattern, " :"
 			]),
 
@@ -1775,8 +1771,6 @@
 					"\n\n", 
 					CFileName, " ",
 					ErrFileName, " ", 
-					PicObjFileName, " ", 
-					ObjFileName, " ",
 					SplitObjPattern, " :"
 				]),
 				write_dependencies_list(TransOptDeps,
@@ -1931,28 +1925,6 @@
 				"\trm -f ", CFileName, "\n",
 				"\t$(MCG) $(ALL_GRADEFLAGS) $(ALL_MCGFLAGS) ",
 					"$< > ", ErrFileName, " 2>&1\n",
-				"ifneq ($(RM_C),:)\n",
-				ObjFileName, " : ", SourceFileName, "\n",
-				"\t$(MMAKE_MAKE_CMD) $(MFLAGS) ",
-					"MC=""$(MC)"" ",
-					"ALL_MCFLAGS=""$(ALL_MCFLAGS)"" ",
-					"ALL_GRADEFLAGS=""$(ALL_GRADEFLAGS)"" ",
-					CFileName, "\n",
-				"\t$(MGNUC) $(ALL_GRADEFLAGS) ",
-					"$(ALL_MGNUCFLAGS) -c ", CFileName,
-					" -o $@\n",
-				"\t$(RM_C) ", CFileName, "\n",
-				PicObjFileName, " : ", SourceFileName, "\n",
-				"\t$(MMAKE_MAKE_CMD) $(MFLAGS) ",
-					"MC=""$(MC)"" ",
-					"ALL_MCFLAGS=""$(ALL_MCFLAGS)"" ",
-					"ALL_GRADEFLAGS=""$(ALL_GRADEFLAGS)"" ",
-					CFileName, "\n",
-				"\t$(MGNUC) $(ALL_GRADEFLAGS) ",
-					"$(ALL_MGNUCFLAGS) $(CFLAGS_FOR_PIC) ",
-					"\\\n",
-				"\t\t-c ", CFileName, " -o $@\n",
-				"endif # RM_C != :\n",
 				"ifeq ($(TARGET_ASM),yes)\n",
 				AsmFileName, " : ", SourceFileName, "\n",
 				"\trm -f ", AsmFileName, "\n",
@@ -2754,6 +2726,24 @@
 	write_extra_link_dependencies_list(ExtraLinkObjs, ".$O", DepStream),
 	io__write_string(DepStream, "\n"),
 
+	%
+	% $(foo.cs_or_ss) contains the names of the generated intermediate
+	% files between `.m' and `.o' files. This is used in foo.dep
+	% to make sure the intermediate files are generated before the
+	% object files, so that errors are reported as soon as possible.
+	% 
+	% If TARGET_ASM=yes, we define $(foo.cs_or_ss) to be $(foo.ss),
+	% otherwise it is defined to be $(foo.cs).
+	%
+	io__write_strings(DepStream, [
+		"ifeq ($(TARGET_ASM),yes)\n",
+		MakeVarName, ".cs_or_ss =$(", MakeVarName, ".ss)\n",
+		"else\n",
+		MakeVarName, ".cs_or_ss =$(", MakeVarName, ".cs)\n",
+		"endif\n\n"
+	]),
+
+
 	io__write_string(DepStream, MakeVarName),
 	io__write_string(DepStream, ".rlos = "),
 	write_compact_dependencies_list(Modules, "$(rlos_subdir)", ".rlo",
@@ -2992,35 +2982,14 @@
 	% rather than first spending time compiling C files to .$O,
 	% which could be a waste of time if the program contains errors.
 	%
-	% But we can only do this if we don't remove the .c files,
-	% i.e. if RM_C=:
-	% So we define $(foo.maybe_cs) here and use it in the rules below.
-	% This needs to be defined here in the .dep file rather than
-	% in the .dv file since it depends on the setting of the $(RM_C) file
-	% which can be overridden by the user's Mmakefile.
-	%
 	% When compiling to assembler, we want to do the same kind of
 	% thing, for the same reason, but with the `.s' files rather
-	% than the `.c' files.  So if TARGET_ASM=yes, we define
-	% $(foo.maybe_cs) to be $(foo.ss).
-	% XXX The name `.maybe_cs' is a bit misleading in this case.
-	%     Perhaps we should change it.
+	% than the `.c' files.
 	%
-	module_name_to_file_name(SourceModuleName, "", no, ExeFileName),
-	io__write_strings(DepStream, [
-		"ifeq ($(TARGET_ASM),yes)\n",
-		MakeVarName, ".maybe_cs=$(", MakeVarName, ".ss)\n",
-		"else\n",
-		"  ifeq ($(RM_C),:)\n",
-		MakeVarName, ".maybe_cs=$(", MakeVarName, ".cs)\n",
-		"  else\n",
-		MakeVarName, ".maybe_cs=\n",
-		"  endif\n\n",
-		"endif\n\n"
-	]),
 
+	module_name_to_file_name(SourceModuleName, "", no, ExeFileName),
 	io__write_strings(DepStream, [
-		ExeFileName, " : $(", MakeVarName, ".maybe_cs) ",
+		ExeFileName, " : $(", MakeVarName, ".cs_or_ss) ",
 			"$(", MakeVarName, ".os) ",
 			InitObjFileName, " $(MLOBJS) ", All_MLLibsDepString,
 			"\n",
@@ -3084,7 +3053,7 @@
 	]),
 
 	io__write_strings(DepStream, [
-		SharedLibFileName, " : $(", MakeVarName, ".maybe_cs) ",
+		SharedLibFileName, " : $(", MakeVarName, ".cs_or_ss) ",
 			"$(", MakeVarName, ".pic_os) ",
 			"$(MLPICOBJS) ", All_MLLibsDepString, "\n",
 		"\t$(ML) --make-shared-lib $(ALL_GRADEFLAGS) $(ALL_MLFLAGS) ",
@@ -3094,7 +3063,7 @@
 	]),
 
 	io__write_strings(DepStream, [
-		LibFileName, " : $(", MakeVarName, ".maybe_cs) ",
+		LibFileName, " : $(", MakeVarName, ".cs_or_ss) ",
 			"$(", MakeVarName, ".os) $(MLOBJS)\n",
 		"\trm -f ", LibFileName, "\n",
 		"\t$(AR) $(ALL_ARFLAGS) $(AR_LIBFILE_OPT)", LibFileName, " ",
Index: extras/graphics/mercury_opengl/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/extras/graphics/mercury_opengl/Mmakefile,v
retrieving revision 1.2
diff -u -u -r1.2 Mmakefile
--- extras/graphics/mercury_opengl/Mmakefile	1999/07/20 03:39:23	1.2
+++ extras/graphics/mercury_opengl/Mmakefile	2001/05/18 10:15:17
@@ -1,6 +1,4 @@
 
-#RM_C=:
-
 MAIN_TARGET	= all
 
 MLOBJS		= togl.o 
Index: extras/morphine/non-regression-tests/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/extras/morphine/non-regression-tests/Mmakefile,v
retrieving revision 1.3
diff -u -u -r1.3 Mmakefile
--- extras/morphine/non-regression-tests/Mmakefile	2000/01/04 02:32:54	1.3
+++ extras/morphine/non-regression-tests/Mmakefile	2001/05/18 10:15:27
@@ -6,8 +6,6 @@
 # This file is copied from the Mercury tests repository
 include ./Mmake.common
 
-
-RM_C=:
 #-----------------------------------------------------------------------------#
 
 MORPHINE = ../scripts/morphine  --no-banner
Index: extras/trailed_update/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/extras/trailed_update/Mmakefile,v
retrieving revision 1.11
diff -u -u -r1.11 Mmakefile
--- extras/trailed_update/Mmakefile	2000/11/21 23:52:33	1.11
+++ extras/trailed_update/Mmakefile	2001/05/18 10:18:21
@@ -7,7 +7,6 @@
 GRADEFLAGS += --use-trail
 
 # enable C debugging
-RM_C = :
 MGNUCFLAGS = -g
 MLFLAGS = -g
 
Index: extras/trailed_update/samples/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/extras/trailed_update/samples/Mmakefile,v
retrieving revision 1.13
diff -u -u -r1.13 Mmakefile
--- extras/trailed_update/samples/Mmakefile	2001/02/28 13:46:06	1.13
+++ extras/trailed_update/samples/Mmakefile	2001/05/18 10:18:28
@@ -13,7 +13,6 @@
 
 # Enable C debugging
 CFLAGS=-g
-RM_C=:
 MLFLAGS=-g
 
 # Link in the trailed_update library from ..
Index: extras/trailed_update/tests/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/extras/trailed_update/tests/Mmakefile,v
retrieving revision 1.5
diff -u -u -r1.5 Mmakefile
--- extras/trailed_update/tests/Mmakefile	2000/04/20 06:35:30	1.5
+++ extras/trailed_update/tests/Mmakefile	2001/05/18 10:18:36
@@ -12,7 +12,6 @@
 MCFLAGS = --infer-all
 
 MGNUCFLAGS=-g -I..
-RM_C=:
 MLFLAGS=-g
 
 # Link in the trailed_update library from ..
Index: library/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/library/Mmakefile,v
retrieving revision 1.67
diff -u -u -r1.67 Mmakefile
--- library/Mmakefile	2001/05/17 09:36:11	1.67
+++ library/Mmakefile	2001/05/18 10:15:36
@@ -34,12 +34,6 @@
 
 #-----------------------------------------------------------------------------#
 
-# If we're going to generate both `.$O' files and `.pic_o' files, then
-# don't remove the intermediate `.c' files.
-RM_C	=	$(LIBRARY_RM_C)
-
-#-----------------------------------------------------------------------------#
-
 # Specify which options to use to compile the library.
 # Don't change these without good reason - if you want to
 # do a temporary change, change ../Mmake.params.
Index: runtime/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/runtime/Mmakefile,v
retrieving revision 1.70
diff -u -u -r1.70 Mmakefile
--- runtime/Mmakefile	2001/03/16 01:40:39	1.70
+++ runtime/Mmakefile	2001/05/18 10:15:47
@@ -342,12 +342,6 @@
 
 endif
 
-
-#-----------------------------------------------------------------------------#
-
-# prevent Mmake from removing C files
-RM_C=:
-
 #-----------------------------------------------------------------------------#
 
 clean_local: clean_o clean_check
Index: samples/c_interface/c_calls_mercury/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/samples/c_interface/c_calls_mercury/Mmakefile,v
retrieving revision 1.5
diff -u -u -r1.5 Mmakefile
--- samples/c_interface/c_calls_mercury/Mmakefile	2001/01/24 13:48:09	1.5
+++ samples/c_interface/c_calls_mercury/Mmakefile	2001/05/18 10:15:59
@@ -28,5 +28,3 @@
 	rm -f c_main.o mercury_lib.h
 
 #-----------------------------------------------------------------------------#
-
-RM_C=:
Index: samples/c_interface/cplusplus_calls_mercury/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/samples/c_interface/cplusplus_calls_mercury/Mmakefile,v
retrieving revision 1.4
diff -u -u -r1.4 Mmakefile
--- samples/c_interface/cplusplus_calls_mercury/Mmakefile	2001/02/05 09:35:45	1.4
+++ samples/c_interface/cplusplus_calls_mercury/Mmakefile	2001/05/18 10:16:13
@@ -30,9 +30,6 @@
 MGNUCFLAGS=-g
 MLFLAGS=-g
 
-# tell Mmake not to remove the `.c' files, so we can look at them (e.g. in gdb)
-RM_C=:
-
 # tell the Mercury linker to link in cpp_main.o and mercury_lib.o
 MLLIBS=cpp_main.o mercury_lib.o
 
Index: samples/c_interface/simpler_c_calls_mercury/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/samples/c_interface/simpler_c_calls_mercury/Mmakefile,v
retrieving revision 1.5
diff -u -u -r1.5 Mmakefile
--- samples/c_interface/simpler_c_calls_mercury/Mmakefile	2001/01/24 13:48:12	1.5
+++ samples/c_interface/simpler_c_calls_mercury/Mmakefile	2001/05/18 10:16:49
@@ -23,9 +23,6 @@
 # not a main program
 C2INITFLAGS=--library
 
-# keep the `.c' files around, for debugging
-RM_C=:
-
 #-----------------------------------------------------------------------------#
 
 c_main.o: mercury_lib.h c_main.c
Index: samples/c_interface/simpler_cplusplus_calls_mercury/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/samples/c_interface/simpler_cplusplus_calls_mercury/Mmakefile,v
retrieving revision 1.4
diff -u -u -r1.4 Mmakefile
--- samples/c_interface/simpler_cplusplus_calls_mercury/Mmakefile	2001/02/05 09:35:53	1.4
+++ samples/c_interface/simpler_cplusplus_calls_mercury/Mmakefile	2001/05/18 10:16:29
@@ -35,9 +35,6 @@
 # not a main program
 C2INITFLAGS=--library
 
-# tell Mmake not to remove the `.c' files, so we can look at them (e.g. in gdb)
-RM_C=:
-
 OBJECTS = cpp_main.o mercury_lib_init.o $(mercury_lib.os)
 
 #-----------------------------------------------------------------------------#
Index: scripts/Mmake.rules
===================================================================
RCS file: /home/mercury1/repository/mercury/scripts/Mmake.rules,v
retrieving revision 1.93
diff -u -u -r1.93 Mmake.rules
--- scripts/Mmake.rules	2001/04/25 15:16:49	1.93
+++ scripts/Mmake.rules	2001/05/18 07:35:53
@@ -224,29 +224,6 @@
 	$(MS_ILASM) $(ALL_MS_ILASMFLAGS) /dll /quiet /OUT=$@ $<
 endif
 
-# If we are removing the .c files, we need to tell Make that we're
-# generating the .$O files directly from the .m files, but
-# in order to avoid remaking the .c files if they're already there,
-# we need to invoke mmake recursively for each .c file.
-# This can be pretty inefficient.
-
-ifneq ($(RM_C),:)
-
-$(os_subdir)%.$O : %.m
-	$(MMAKE_MAKE_CMD) $(MFLAGS) MC="$(MC)" ALL_MCFLAGS="$(ALL_MCFLAGS)" \
-		ALL_GRADEFLAGS="$(ALL_GRADEFLAGS)" $(cs_subdir)$*.c
-	$(MGNUC) $(ALL_GRADEFLAGS) $(ALL_MGNUCFLAGS) -c $(cs_subdir)$*.c \
-		$(OBJFILE_OPT)$@
-	$(RM_C) $(cs_subdir)$*.c
-
-$(os_subdir)%.pic_o : %.m
-	$(MMAKE_MAKE_CMD) $(MFLAGS) MC="$(MC)" ALL_MCFLAGS="$(ALL_MCFLAGS)" \
-		ALL_GRADEFLAGS="$(ALL_GRADEFLAGS)" $(cs_subdir)$*.c
-	$(MGNUC) $(ALL_GRADEFLAGS) $(ALL_MGNUCFLAGS) $(CFLAGS_FOR_PIC) \
-		-c $(cs_subdir)$*.c $(OBJFILE_OPT)$@
-
-endif # RM_C != :
-
 # For --split-c-files, we generate the .$O files directly from the .m files.
 # (One reason for this is that there's no easy of telling Make how many
 # `.$O' files it should make, since the number of them depends on what's
Index: scripts/Mmake.vars.in
===================================================================
RCS file: /home/mercury1/repository/mercury/scripts/Mmake.vars.in,v
retrieving revision 1.41
diff -u -u -r1.41 Mmake.vars.in
--- scripts/Mmake.vars.in	2001/04/25 15:16:49	1.41
+++ scripts/Mmake.vars.in	2001/05/18 07:35:22
@@ -256,11 +256,6 @@
 # shared libraries.
 EXT_FOR_SHARED_LIB	= @EXT_FOR_SHARED_LIB@
 
-# Set `RM_C' to `:' if you want to keep the intermediate `.c' files,
-# or to `rm' if you want Mmake to remove them.  Beware that RM_C=rm
-# causes some potential race conditions if you use parallel make.
-RM_C		= @DEFAULT_RM_C@
-
 # lists of targets that depend on $(MLOBJS) and $(MLPICOBJS)
 MLOBJS_DEPS = 
 MLPICOBJS_DEPS = 
Index: tests/debugger/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/tests/debugger/Mmakefile,v
retrieving revision 1.51
diff -u -u -r1.51 Mmakefile
--- tests/debugger/Mmakefile	2001/05/08 03:05:01	1.51
+++ tests/debugger/Mmakefile	2001/05/18 10:18:48
@@ -5,8 +5,6 @@
 include ../Mmake.common
 -include ../Mmake.params
 
-RM_C=:
-
 #-----------------------------------------------------------------------------#
 
 # We suppress the printing of the banner, because different workspaces
Index: tests/debugger/declarative/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/tests/debugger/declarative/Mmakefile,v
retrieving revision 1.29
diff -u -u -r1.29 Mmakefile
--- tests/debugger/declarative/Mmakefile	2001/05/03 07:26:04	1.29
+++ tests/debugger/declarative/Mmakefile	2001/05/18 10:18:56
@@ -5,8 +5,6 @@
 include ../../Mmake.common
 -include ../../Mmake.params
 
-RM_C=:
-
 #-----------------------------------------------------------------------------#
 
 # We suppress the printing of the banner, because different workspaces
Index: tools/binary_step
===================================================================
RCS file: /home/mercury1/repository/mercury/tools/binary_step,v
retrieving revision 1.14
diff -u -u -r1.14 binary_step
--- tools/binary_step	2000/07/19 03:45:11	1.14
+++ tools/binary_step	2001/05/18 10:14:21
@@ -203,9 +203,7 @@
 
 set -x
 
-# the `RM_C=:' ensures that the `.c' files do not get deleted
-
-if (cd stage2/library ; mmake $mmake_opts $jfactor RM_C=: )
+if (cd stage2/library ; mmake $mmake_opts $jfactor)
 then
 	echo "building of stage 2 library successful"
 else
@@ -214,7 +212,7 @@
 	exit 1
 fi
 
-if (cd stage2/compiler ; mmake $mmake_opts $jfactor RM_C=: mercury_compile)
+if (cd stage2/compiler ; mmake $mmake_opts $jfactor mercury_compile)
 then
 	echo "building of stage 2 compiler successful"
 else
Index: tools/bootcheck
===================================================================
RCS file: /home/mercury1/repository/mercury/tools/bootcheck,v
retrieving revision 1.117
diff -u -u -r1.117 bootcheck
--- tools/bootcheck	2001/05/15 12:20:45	1.117
+++ tools/bootcheck	2001/05/18 10:14:37
@@ -530,9 +530,7 @@
 		exit 1
 	fi
 
-	# the `RM_C=:' ensures that the `.c' files do not get deleted
-
-	if (cd stage2/library && $MMAKE $mmake_opts $jfactor RM_C=: mercury)
+	if (cd stage2/library && $MMAKE $mmake_opts $jfactor mercury)
 	then
 		echo "building of stage 2 library successful"
 	else
@@ -540,7 +538,7 @@
 		exit 1
 	fi
 
-	if (cd stage2/browser && $MMAKE $mmake_opts $jfactor RM_C=: library)
+	if (cd stage2/browser && $MMAKE $mmake_opts $jfactor library)
 	then
 		echo "building of stage 2 browser successful"
 	else
@@ -556,8 +554,7 @@
 		exit 1
 	fi
 
-	if (cd stage2/compiler && $MMAKE $mmake_opts $jfactor RM_C=: \
-		mercury_compile)
+	if (cd stage2/compiler && $MMAKE $mmake_opts $jfactor mercury_compile)
 	then
 		echo "building of stage 2 compiler successful"
 	else
Index: trace/Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/trace/Mmakefile,v
retrieving revision 1.21
diff -u -u -r1.21 Mmakefile
--- trace/Mmakefile	2001/03/16 01:40:43	1.21
+++ trace/Mmakefile	2001/05/18 10:14:46
@@ -210,11 +210,6 @@
 
 #-----------------------------------------------------------------------------#
 
-# prevent Mmake from removing C files
-RM_C=:
-
-#-----------------------------------------------------------------------------#
-
 clean_local: clean_o clean_check
 
 .PHONY: clean_o
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list