[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: rev 1973 - trunk trunk/subversion/libsvn_subr

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-05-17 13:46:05 CEST

brane@xbc.nu writes:

> > Modified: trunk/configure.in
> > ==============================================================================
> > --- trunk/configure.in (original)
> > +++ trunk/configure.in Thu May 16 23:52:34 2002
> > @@ -262,6 +262,19 @@
> > AC_DEFINE_UNQUOTED(SVN_CLIENT_DIFF3, "$SVN_CLIENT_DIFF3",
> > [Define to be the full path to your GNU diff3 program])
> >
> > +dnl Determine whether diff3 supports the --diff-program arg.
> > +AC_MSG_CHECKING([whether diff3 supports --diff-program arg])
> > +touch A
> > +$SVN_CLIENT_DIFF3 --diff-program=$SVN_CLIENT_DIFF A A A > /dev/null
> > 2>&1
> > +if test "$?" = "0"; then
> > + AC_MSG_RESULT([yes])
> > + AC_DEFINE_UNQUOTED(SVN_DIFF3_HAS_DIFF_PROGRAM_ARG, 1,
> > + [Defined if diff3 supports the --diff-program argument])
> > +else
> > + AC_MSG_RESULT([no])
> > +fi
> > +rm -f A
>
> "A" is not a very good choice for a temporary file name (my comments on
> IRC notwithstanding). I'd either use a file that we know exists (for instance,
> $abs_srcdir/README) or somehing more obscure, perhaps including the PID.
> You could even use the name Autoconf generates for test programs.

Or just use the current directory: 'diff . .'

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 17 15:00:37 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.