[PATCH] yet another makefile tweak

[Dean Gaudet <dg...@arctic.org>]

This is from Martin.  But Ken didn't like it because he wanted to see all
errors in one pass.  Ben liked it because the first error is frequently
the most common one we need when getting a bug report, and we don't want
all of them.   I like it because without it you can't hit ^C and get the
thing to reliably stop.  You have to hold down ^C ... which is lame.

Ken, you can get more errors by doing something like:

    make MAKE='make -k'

That will show you all errors in the first directory with errors.  Is that
good enough?

Dean

--- src/Makefile.tmpl.bak	Fri Sep 12 10:03:00 1997
+++ src/Makefile.tmpl	Fri Sep 12 14:07:01 1997
@@ -34,7 +34,7 @@
 
 subdirs:
 	for i in $(SUBDIRS); do \
-		( cd $$i; $(MAKE) CC='$(CC)' AUX_CFLAGS='$(CFLAGS)' RANLIB='$(RANLIB)') \
+		( cd $$i; $(MAKE) CC='$(CC)' AUX_CFLAGS='$(CFLAGS)' RANLIB='$(RANLIB)') || exit 1; \
 	done
 
 support: support-dir
@@ -45,7 +45,7 @@
 clean:
 	rm -f httpd *.o $(OBJS) 
 	for i in $(SUBDIRS) regex; do \
-		( cd $$i; $(MAKE) $@ ) \
+		( cd $$i; $(MAKE) $@ ) || exit 1; \
 	done
 
 dist.tar: 
--- src/modules/Makefile.bak	Fri Sep 12 11:58:53 1997
+++ src/modules/Makefile	Fri Sep 12 14:07:02 1997
@@ -14,6 +14,6 @@
 all clean :: 
 	for i in $(MODULES); do \
 	 (cd $$i; \
-	  $(MAKE) CC='$(CC)' AUX_CFLAGS='$(MOD_CFLAGS)' RANLIB='$(RANLIB)' $@);\
+	  $(MAKE) CC='$(CC)' AUX_CFLAGS='$(MOD_CFLAGS)' RANLIB='$(RANLIB)' $@) || exit 1;\
 	done
 

Re: [PATCH] yet another makefile tweak

[Paul Sutton <pc...@hyperreal.org>]

On Mon, 22 Sep 1997, Dean Gaudet wrote:
> This is from Martin.  But Ken didn't like it because he wanted to see all
> errors in one pass.  Ben liked it because the first error is frequently
> the most common one we need when getting a bug report, and we don't want
> all of them.   I like it because without it you can't hit ^C and get the

+1

//pcs