automake-patches
[Top][All Lists]
Advanced

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

Re: testsuite issues on MinGW/MSYS


From: Stefano Lattarini
Subject: Re: testsuite issues on MinGW/MSYS
Date: Sat, 15 Jan 2011 12:59:55 +0100
User-agent: KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; )

On Saturday 15 January 2011, Ralf Wildenhues wrote:
> Like this?
> 
> Thanks,
> Ralf
> 
> 2011-01-15  Ralf Wildenhues  <address@hidden>
>           Stefano Lattarini  <address@hidden>
> 
>       tests: explain MSYS setup failure issue, improve test.
>       * tests/defs: Add comment and failure message, improve fail
>       logic.
> 
> diff --git a/tests/defs b/tests/defs
> index 0f84d0a..b357df6 100644
> --- a/tests/defs
> +++ b/tests/defs
> @@ -31,8 +31,10 @@ test -f ./defs-static || {
>  . ./defs-static || exit 99
>  
>  # The name of the current test (without the `.test' suffix.)
> -me=`echo "$argv0" | sed -e 's,.*[\\/],,;s/\.test$//'`
> -test -n "$me" || exit 99
> +# Guard against failure to spawn sed (seen on MSYS), or empty $argv0.
> +me=`echo "$argv0" | sed -e 's,.*[\\/],,;s/\.test$//'` \
> +  && test -n "$me" \
> +  || { echo "$argv0: failed to define \$me" >&2; exit 99; }
>  
>  
Yes (good comment BTW, terse but clear).

Thanks,
  Stefano



reply via email to

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