autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Autotest: point to log more prominently


From: Noah Misch
Subject: Re: Autotest: point to log more prominently
Date: Sun, 2 Apr 2006 15:34:06 -0700
User-agent: Mutt/1.5.6i

On Mon, Oct 10, 2005 at 06:35:57PM +0200, Ralf Wildenhues wrote:
>         * lib/autotest/general.m4 (AT_INIT): Add `at_testdir' to pointer
>         to log, point to testsuite output tree.

This is a valuable change.

> --- lib/autotest/general.m4   4 Oct 2005 09:14:12 -0000       1.193
> +++ lib/autotest/general.m4   10 Oct 2005 16:31:49 -0000
> @@ -947,7 +952,7 @@
>    AS_BOX([$as_me.log was created.])
>  
>    echo
> -  echo "Please send \`$as_me.log' and all information you think might help:"
> +  echo "Please send \`${at_testdir}/$as_me.log' and all information you 
> think might help:"

Some code in lib/autotest/general.m4 contemplates the possibility that
`atconfig' does not exist, a so-called standalone test suite.  In this case,
$at_testdir will be empty or unset, and the output will look funny.  You could
use $at_suite_log (`pwd`/$as_me.log), as an alternative.

We don't document this possibility; you might be safe ignoring it.

>    echo
>    echo "   To: <AT_PACKAGE_BUGREPORT>"
>    echo "   Subject: @<:@AT_PACKAGE_STRING@:>@ $as_me:dnl
> @@ -957,7 +962,8 @@
>    if test $at_debug_p = false; then
>      echo
>      echo 'You may investigate any problem if you feel able to do so, in 
> which'
> -    echo 'case the test suite provides a good starting point.'
> +    echo 'case the test suite provides a good starting point.  Its output 
> may'
> +    echo "be found below \`${at_testdir}/$as_me.dir'."

Likewise; you could use $at_suite_dir here, or just leave it as-is if the
non-atconfig case does not matter.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]