Re: [PATCH] remote-helpers/test-bzr.sh: do not use grep \s

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Apr 7, 2013, 09:02 UTC
Message-ID: <CAMP44s06=KCDfQuK1t2ZURoOzmF69pDXF0G6eji6EsiMAoBLbQ@mail.gmail.com>
In-Reply-To: <201304071048.31790.tboegi@web.de>

On Sun, Apr 7, 2013 at 3:48 AM, Torsten Bögershausen <tboegi@web.de> wrote:

Show 7 quoted lines

> Using grep "devel\s\+3:" to find at least one whitspace
> is not portable on all grep versions:
> Not all grep versions understand "\s" as a "whitespace".
>
> Use a literal TAB followed by SPACE like this [  ] instead.
> The + as a qualifier for "one or more" is not a basic regular expression:
> use egrep instead of grep.

If "\t" is not possible, I would rather use [:space:].

Cheers.

-- 
Felipe Contreras