bug#15441: bug#15414: 24.3; piping problems in eshell

2013-09-23 Thread Joseph Mingrone
Hello; Aidan Gauland writes: > I can not reproduce this on my system (Debian 7.1 x86_64). Which OS are > you running? I originally stumbled upon the problem running 24.3 on FreeBSD 9. I also tested on a couple of GNU/Linux systems and things worked as expected, but the problem exists on OS X.

bug#15440: [PATCH] dfa: fix \s and \S to work for multibyte

2013-09-23 Thread Jim Meyering
[using the right bug address, this time] On Mon, Sep 23, 2013 at 11:26 AM, Aharon Robbins wrote: > Hi. > >> $ printf '\x82\n' > in; ./grep -q '\S' in && echo match >> match >> >> Now, require a back-reference (forcing switch from grep's DFA matcher >> to use of the regex functions), and y

bug#15444: One character can be lost if colors are enabled

2013-09-23 Thread Jim Meyering
On Mon, Sep 23, 2013 at 6:34 AM, Jaroslav Skarvada wrote: > printf > '1234x\n' > | grep 1234 --color=always Thank you for the report. I confirm that setting GREP_COLORS=ne is a work-around. Does that have unwel

bug#15441: [PATCH] tests: ensure neither \s nor \S matches an invalid multibyte character

2013-09-23 Thread Jim Meyering
On Mon, Sep 23, 2013 at 6:26 AM, Stefano Lattarini wrote: > Hi Jim. > > On 09/23/2013 06:18 AM, Jim Meyering wrote: >> >> Here's a test for the original problem, which is also fixed by the >> preceding patch: >> >> ... > >> >> >> +grep '^\S$' in > out-S && fail=1 > >> +compare /dev/null out-S || e

bug#15444: One character can be lost if colors are enabled

2013-09-23 Thread Jaroslav Skarvada
Reproducer: $ printf '1234x\n' | grep 1234 --color=always 123x This can be reproduced at least on xterm and linux console, but i

bug#15441: [PATCH] tests: ensure neither \s nor \S matches an invalid multibyte character

2013-09-23 Thread Stefano Lattarini
Hi Jim. On 09/23/2013 06:18 AM, Jim Meyering wrote: Here's a test for the original problem, which is also fixed by the preceding patch: ... > +grep '^\S$' in > out-S && fail=1 > +compare /dev/null out-S || exit=1 > Shouldn't this be "|| fail=1"? > +grep '^\s$' in > out-s && fail=1 > +compar