On Tuesday 08 November 2011 17:31:45, Richard Henderson wrote:
> On 11/08/2011 09:26 AM, Pedro Alves wrote:
> > On Tuesday 08 November 2011 16:33:52, Richard Henderson wrote:
> >> toplevel/
> >> * configure.ac: Adjust srcdir for running libitm/configure.tgt.
> >
> >> +# Disable libitm on unsupported hosted systems.
> >> +if test x$enable_libitm = x; then
> >> + AC_MSG_CHECKING([for libitm support])
> >> + if (srcdir=${srcdir}/libitm; \
> >> + . ${srcdir}/configure.tgt; \
> >
> > This should check that libitm exists before sourcing the script,
> > as otherwise it will error out on src tree
> > checkouts (binutils, gdb, ...), given that the libitm directory
> > doesn't exist there.
>
> Should I bother with an explicit test, or just use "set -e"?
Whatever works is fine with me. :-)
--
Pedro Alves
