[m-rev.] for review: configure.in improvements

Simon Taylor stayl at cs.mu.OZ.AU
Fri Nov 8 02:25:14 AEDT 2002


On 07-Nov-2002, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> On 06-Nov-2002, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> > 
> > configure.in:
> ...
> > +case $remake_dependencies in true)
> > +	mmake depend ;;
> > +esac
> 
> If `mmake depend' fails, then configure should return a non-zero exit
> status.
> 
> Also, it might be nice to output a message or two explaining what was
> happening here.

diff -u configure.in configure.in
--- configure.in
+++ configure.in
@@ -3193,6 +3193,13 @@
 )
 
 case $remake_dependencies in true)
-	mmake depend ;;
+	MERCURY_MSG("regenerating dependencies to enable GCC backend")
+	MERCURY_MSG("and/or disable Aditi backend.")
+
+	mmake depend >&AC_FD_CC 2>&1
+	if test $? != 0; then
+		exit $?
+	fi
+	;;
 esac
 #-----------------------------------------------------------------------------#
--------------------------------------------------------------------------
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