[AUCTeX-devel] Re: [AUCTeX-diffs] Changes to auctex/Makefile.in

[David Kastrup]

Ralf Angeli <address@hidden> writes:

> Index: auctex/Makefile.in
> diff -u auctex/Makefile.in:1.101 auctex/Makefile.in:1.102
> --- auctex/Makefile.in:1.101  Wed Jun  8 07:55:23 2005
> +++ auctex/Makefile.in        Thu Jun  9 08:42:48 2005
> @@ -36,7 +36,7 @@
>  AUTOLOAD=--eval '(let ((generated-autoload-file (expand-file-name "$@"))) \
>    (mapcar (function update-file-autoloads) command-line-args-left) \
>    (save-buffers-kill-emacs t))'
> -DESCEND=OLDPWD="`pwd`";for i in $(subdirs);do cd $$i;echo "Descending into 
> `pwd`";$(MAKE) 'DESTDIR=$(DESTDIR)' PACKAGE=$(PACKAGE) $@ || exit $$?;cd 
> $$OLDPWD;echo "Ascending into $$OLDPWD";done
> +DESCEND=if test -n "$(subdirs)"; then OLDPWD="`pwd`";for i in 
> ""$(subdirs);do cd $$i;echo "Descending into `pwd`";$(MAKE) 
> 'DESTDIR=$(DESTDIR)' PACKAGE=$(PACKAGE) $@ || exit $$?;cd $$OLDPWD;echo 
> "Ascending into $$OLDPWD";done;fi
>  

One note here: avoid "if" in Makefile rules, rather use the
short-circuit operators || and &&.  The reason is that not all Bourne
shells return "success" for "if false", causing "make" to bomb out.

The alternative is to put in

else :

at the end.  Yes, this is idiotic.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum