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