[m-rev.] uninstall and Emacs Lisp (was: deep profiler installation)

[Zoltan Somogyi]

On 04-Jul-2001, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> > BTW, does anyone know the answer to the question I added in a comment in
> > scripts/Mmakefile, which is why the scripts uninstall action does not remove
> > the Emacs Lisp file installed by that same Mmakefile?
> 
> The `uninstall' rule in the top-level makefile does `rm -r' on the parent
> of the directory that the Emacs Lisp file normally gets installed into.
> 
> >From Mmakefile:
> 
> 	.PHONY: uninstall
> 	uninstall:
> 		cd scripts; $(SUBDIR_MMAKE) uninstall
> 		cd util; $(SUBDIR_MMAKE) uninstall
> 		cd doc; $(SUBDIR_MMAKE) uninstall
> 		-rm -r $(INSTALL_LIBDIR)
> 
> >From scripts/Mmake.vars.in:
> 
> 	INSTALL_LIBDIR          = $(INSTALL_PREFIX)/lib/mercury
> 	INSTALL_ELISP_DIR 	= $(INSTALL_PREFIX)/lib/mercury/elisp

How about this then?

scripts/Mmakefile:
	Document why we don't uninstall $(EMACS_SCRIPTS) here.

Zoltan.

cvs diff: Diffing .
Index: Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/scripts/Mmakefile,v
retrieving revision 1.23
diff -u -b -r1.23 Mmakefile
--- Mmakefile	2001/07/09 04:58:52	1.23
+++ Mmakefile	2001/07/13 07:28:22
@@ -107,7 +107,12 @@
 	-rm -r $(INSTALL_LIBDIR)/mmake
 	-cd $(INSTALL_BINDIR) && rm $(SCRIPTS) 
 	-cd $(INSTALL_LIBDIR)/mdb && rm $(DEBUGGER_SCRIPTS)
-	# Is there any reason why we don't uninstall $(EMACS_SCRIPTS) here?
+	# The reason why we don't uninstall $(EMACS_SCRIPTS) specifically here
+	# is that the uninstall rule in the top-level Mmakefile does an "rm -r"
+	# on $(INSTALL_LIBDIR), and the directory into which $(EMACS_SCRIPTS)
+	# is installed, $(INSTALL_ELISP_DIR), is configured to be a
+	# subdirectory of $(INSTALL_LIBDIR).
+	#
 	# We try to uninstall $(CGI_SCRIPTS), but failure to do so is not an
 	# error for two reasons: because we may simply lack permission to
 	# update the directory, and because the deep profiler installation
--------------------------------------------------------------------------
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
--------------------------------------------------------------------------