SF patch #683906, add $DESTDIR to install target, by Ademar de Souza Reis

Jr, after much checking and rechecking (and some massaging) by me. Checking
in now before I fall asleep and forget what all this was for.

This patch adds the ability to 'make DESTDIR=/some/dir/tree install' that
doesn't influence the paths stored in e.g. Defaults.py at all, which is of
good use for various package managers. It is not all that package managers
must do, however! Running make install with DESTDIR set means bin/update is
never run, and mm_cfg.py is always written; package managers should make
sure the appropriate post-installation is done, and that mm_cfg.py is
treated as a config file.

This patch inadvertently fixes some bogus whitespace: 8-spaces where
surrounding code used tabs. The difference was harmless because the 8-spaces
were used inside shell-continued-oneliners, but it is confusing and could
lead to future harm. I'm too tired to make those two or three changes in a
separate checkin, sorry.

This patch also assumes the various packages that are installed using
distutils do not record (or rather, use) their installation paths anywhere,
but this seems to hold true at least for the moment.

Also, I've done so many slow cvs diff's, I'm wondering when we'll switch to
Subversion. Unfortunately, I've also done so many 'cvs diff -c > file; patch
-p0 -R < file's to switch back and forth between patches and change sets,
I'm wondering when we'll switch to Aegis as well. :-P

1 files changed, 5 insertions, 4 deletions

diff --git a/src/Makefile.in b/src/Makefile.in
index 05e144f27..29918015a 100644
--- a/src/Makefile.in
+++ b/src/Makefile.in
@@ -25,6 +25,7 @@ exec_prefix=	@exec_prefix@
 VPATH=		@srcdir@
 srcdir= 	@srcdir@
 bindir= 	@bindir@
+DESTDIR=
 
 CC=		@CC@
 CHMOD=  	@CHMOD@
@@ -107,20 +108,20 @@ vsnprintf.o: $(srcdir)/vsnprintf.c Makefile
 install: all
 	for f in $(CGI_PROGS); \
 	do \
-	    exe=$(CGIDIR)/$$f$(CGIEXT); \
+	    exe=$(DESTDIR)$(CGIDIR)/$$f$(CGIEXT); \
 	    $(INSTALL_PROGRAM) $$f $$exe; \
 	    $(DIRSETGID) $$exe; \
 	done
 	for f in $(MAIL_PROGS); \
 	do \
-	    $(INSTALL_PROGRAM) $$f $(MAILDIR); \
-	    $(DIRSETGID) $(MAILDIR)/$$f; \
+	    $(INSTALL_PROGRAM) $$f $(DESTDIR)$(MAILDIR); \
+	    $(DIRSETGID) $(DESTDIR)$(MAILDIR)/$$f; \
 	done
 
 finish:
 	-for f in $(SUID_CGI_PROGS); \
 	do \
-	    exe=$(CGIDIR)/$$f$(CGIEXT); \
+	    exe=$(DESTDIR)$(CGIDIR)/$$f$(CGIEXT); \
 	    chmod u+s $$exe; \
 	    chown $(MAILMAN_USER) $$exe; \
 	done