Re: [PATCH] Use character class for sed expression instead of \s

2012-10-26 Thread Jeff King
On Thu, Oct 25, 2012 at 04:58:19PM +0100, Ben Walton wrote: > Sed on Mac OS X doesn't handle \s in a sed expressions so use a more > portable character set expression instead. > > Signed-off-by: Ben Walton Thanks, I think this simple solution is the best. -Peff -- To unsubscribe from this list

Re: [PATCH] Use character class for sed expression instead of \s

2012-10-25 Thread Torsten Bögershausen
On 10/25/2012 08:08 PM, Ben Walton wrote: Hi Torsten, On Thu, Oct 25, 2012 at 5:28 PM, Torsten Bögershausen wrote: BTW: While we are talking CVS: (I installed a fresh version) cvs --version Concurrent Versions System (CVS) 1.11.23 (client/server) I have 1.12.13-MirDebian-8 here. And t9200

Re: [PATCH] Use character class for sed expression instead of \s

2012-10-25 Thread Ben Walton
Hi Torsten, On Thu, Oct 25, 2012 at 5:28 PM, Torsten Bögershausen wrote: > BTW: While we are talking CVS: (I installed a fresh version) > cvs --version > Concurrent Versions System (CVS) 1.11.23 (client/server) I have 1.12.13-MirDebian-8 here. > And t9200 fails: > git checkout t9200-git-cvsexp

Re: [PATCH] Use character class for sed expression instead of \s

2012-10-25 Thread Torsten Bögershausen
On 25.10.12 18:00, Brian Gernhardt wrote: > > On Oct 25, 2012, at 11:58 AM, Ben Walton wrote: > >> Sed on Mac OS X doesn't handle \s in a sed expressions so use a more >> portable character set expression instead. >> >> Signed-off-by: Ben Walton > > Acked-by: Brian Gernhardt > > I have an id

Re: [PATCH] Use character class for sed expression instead of \s

2012-10-25 Thread Brian Gernhardt
On Oct 25, 2012, at 11:58 AM, Ben Walton wrote: > Sed on Mac OS X doesn't handle \s in a sed expressions so use a more > portable character set expression instead. > > Signed-off-by: Ben Walton Acked-by: Brian Gernhardt I have an identical change sitting in my git.git, I've just been too di