Re: [Mesa-dev] [PATCH] configure.ac: Use POSIX word boundary regex.

2017-03-17 Thread Jonathan Gray
On Fri, Mar 17, 2017 at 03:45:08PM +0900, Michel D??nzer wrote: > On 17/03/17 07:42 AM, Vinson Lee wrote: > > Fixes: fe56c745b8cb ("Convert sed(1) syntax to be compatible with FreeBSD > > and OpenBSD") > > Not sure how, since \> was already used (way) before that. Indeed FreeBSD and OpenBSD libc

Re: [Mesa-dev] [PATCH] configure.ac: Use POSIX word boundary regex.

2017-03-16 Thread Michel Dänzer
On 17/03/17 09:16 AM, Jan Beich wrote: > Vinson Lee writes: > >> echo " `$1`" | sed -E \ >> -e 's/[[[:space:]]]+-m[[^[:space:]]]*//g' \ >> --e 's/[[[:space:]]]+-DNDEBUG\>//g' \ >> +-e 's/[[[:space:]]]+-DNDEBUG[[[:>:]]]//g' \ > > Try matching some whitespace after th

Re: [Mesa-dev] [PATCH] configure.ac: Use POSIX word boundary regex.

2017-03-16 Thread Michel Dänzer
On 17/03/17 07:42 AM, Vinson Lee wrote: > Fixes: fe56c745b8cb ("Convert sed(1) syntax to be compatible with FreeBSD and > OpenBSD") Not sure how, since \> was already used (way) before that. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast

Re: [Mesa-dev] [PATCH] configure.ac: Use POSIX word boundary regex.

2017-03-16 Thread Jan Beich
Vinson Lee writes: > -# Use \> (marks the end of the word) > +# Use [[:>:]] (marks the end of the word) [[:>:]] is "an extension, compatible with but not specified by POSIX 1003.2". GNU libc doesn't support it. $ echo 'foot foo bar' | sed -E 's/foo[[:>:]]//g' sed: -e expression #1, char

[Mesa-dev] [PATCH] configure.ac: Use POSIX word boundary regex.

2017-03-16 Thread Vinson Lee
Fixes: fe56c745b8cb ("Convert sed(1) syntax to be compatible with FreeBSD and OpenBSD") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100236 Signed-off-by: Vinson Lee --- configure.ac | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure