Kyle Evans <kev...@freebsd.org> wrote:

> > Which would have revealed:
> > 
> > Standard output:
> > Executing command [ zgrep -e  test ]
> > 
> > Standard error:
> > Fail: incorrect exit status: 134, expected: 0
> > stdout:
> > 
> > stderr:
> > Assertion failed: (pc->matchidx > 0), function procmatch_match, file / 
> > usr/src/usr.bin/grep/util.c, line 223.
> > Abort trap (core dumped)
> > -- 
> > 
> > I'll take a little bit to understand the patch I wrote back then, add an 
> > extra test to cover the originally-reported bug, fix the patch, then get 
> > it into Phabricator ASAP.  Sorry for dropping this-
> > 
> > Thanks,
> > 
> > Kyle Evans
> > 
>
> Here we go: https://reviews.freebsd.org/D49324

Wow, that was quick! Cheers!

Yes, despite my best efforts :-), it works perfectly for me (14-stable)

Anyway, not only should I have run the tests, I totally forgot about the
original issue too - your patch was in my local patch tree, which is
distributed to all my hosts, so I forgot the patch hadn't been commited -
so much so, when I stumbled upon the current assert issue, I was initially
puzzled where the difference between my copy of util.c and the one in
current came from!

Thanks for your help!

Jamie

Reply via email to