autoconf-patches
[Top][All Lists]
Advanced

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

Re: [PATCH] tests: skip broken automake wrapper on MirBSD


From: Stefano Lattarini
Subject: Re: [PATCH] tests: skip broken automake wrapper on MirBSD
Date: Mon, 13 Sep 2010 18:41:55 +0200
User-agent: KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; )

On Monday 13 September 2010, Eric Blake wrote:
> On 09/13/2010 10:20 AM, Stefano Lattarini wrote:
> > Hi Eric.  Just a little nitpicking...
> > 
> > On Monday 13 September 2010, Eric Blake wrote:
> >> diff --git a/tests/tools.at b/tests/tools.at
> >> index 167d68a..e4e119b 100644
> >> --- a/tests/tools.at
> >> +++ b/tests/tools.at
> >> @@ -1164,8 +1164,9 @@ AT_SETUP([autom4te preselections])
> >> 
> >>   # If this test should run on FAT file systems and older w32,
> >>   # then setting $sleep correctly needs to be revisited.
> >> 
> >> -# We use aclocal and automake.
> >> -AT_CHECK([automake --version || exit 77], [], [ignore],
> >> [ignore]) +# We use aclocal and automake.  Skip broken automake
> >> wrappers. +AT_CHECK([automake --version || exit 77], [],
> >> [stdout], [ignore]) +AT_CHECK([[grep '[1-9]\.[0-9][0-9]*'
> >> stdout || exit 77]], [],
> > 
> > Wouldn't "grep '[1-9]\.[0-9]'" be enough here?  Ditto for the
> > other occurrences in the patch.
> 
> Hmm, yeah.  The current version is harmless, but a patch to use the
> shorter regex won't hurt, either.  Do you want to write it?
Sure, but not right away (if that's ok with you).

In thruth, I though you hadn't pushed yet when I wrote my "review",
otherwise I wouldn't have bothered you with such a minor nit (as you 
noticed, your version is perfectly correct).

Regards,
    Stefano



reply via email to

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