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

2017-03-20 Thread Michel Dänzer
On 20/03/17 12:19 AM, Eric Engestrom wrote: > On Sunday, 2017-03-19 13:44:30 +, Jan Beich wrote: >> Vinson Lee writes: >> >>> --e 's/[[[:space:]]]+-DNDEBUG\>//g' \ >>> --e 's/[[[:space:]]]+-D_GNU_SOURCE\>//g' \ >>> --e 's/[[[:space:]]]+-pedantic\>//g' \ >>> +-e

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

2017-03-20 Thread Michel Dänzer
In addition to other comments, the Git shortlog no longer matches the patch itself. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer ___ mesa-dev maili

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

2017-03-19 Thread Eric Engestrom
On Sunday, 2017-03-19 13:44:30 +, Jan Beich wrote: > Vinson Lee writes: > > > --e 's/[[[:space:]]]+-DNDEBUG\>//g' \ > > --e 's/[[[:space:]]]+-D_GNU_SOURCE\>//g' \ > > --e 's/[[[:space:]]]+-pedantic\>//g' \ > > +-e 's/[[[:space:]]]+-DNDEBUG[[[:space:]]]//g' \ >

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

2017-03-19 Thread Emil Velikov
On 19 March 2017 at 11:57, Eric Engestrom wrote: > On Saturday, 2017-03-18 05:18:17 +, Vinson Lee wrote: >> Fixes: fe56c745b8cb ("Convert sed(1) syntax to be compatible with FreeBSD >> and OpenBSD") > > I'm not sure about that Fixes: tag… you may have seen this issue after > this commit, but

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

2017-03-19 Thread Jan Beich
Vinson Lee writes: > --e 's/[[[:space:]]]+-DNDEBUG\>//g' \ > --e 's/[[[:space:]]]+-D_GNU_SOURCE\>//g' \ > --e 's/[[[:space:]]]+-pedantic\>//g' \ > +-e 's/[[[:space:]]]+-DNDEBUG[[[:space:]]]//g' \ > +-e 's/[[[:space:]]]+-D_GNU_SOURCE[[[:space:]]]//g' \ > +

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

2017-03-19 Thread Eric Engestrom
On Saturday, 2017-03-18 05:18:17 +, Vinson Lee wrote: > Fixes: fe56c745b8cb ("Convert sed(1) syntax to be compatible with FreeBSD and > OpenBSD") I'm not sure about that Fixes: tag… you may have seen this issue after this commit, but it's unrelated, code-wise. I don't really care though, and

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

2017-03-17 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 Suggested-by: Jan Beich Suggested-by: Michel Dänzer Signed-off-by: Vinson Lee --- configure.ac | 16 1 file changed, 8 insertio