Re: Test failures when Git is built with libpcre and grep is built without it

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 09, 2017 at 04:51:22AM -0600, A. Wilcox wrote:

> Interestingly enough, you seem to be right.  The failure is very
> bizarre and has nothing to do with system /bin/grep:
> 
> test_must_fail: command succeeded: git grep -G -F -P -E a\x{2b}b\x{2a}c ab
> not ok 142 - grep -G -F -P -E pattern
> #
> #               >empty &&
> #               test_must_fail git grep -G -F -P -E "a\x{2b}b\x{2a}c"
> ab >actual &&
> #               test_cmp empty actual
> #
> 
> However:
> 
> elaine trash directory.t7810-grep # git grep -G -F -P -E
> a\x{2b}b\x{2a}c ab >actual
> fatal: command line, 'ax{2b}bx{2a}c': Invalid contents of {}

Hrm, it looks like your shell eats the backslashes in a double-quoted
string.

What does:

  printf '%s\n' "a\x{2b}b\x{2a}c"

show?

I'm not sure if that's related or not. And as you said your shell is
bash, that seems weird. Lots of other people run bash, and we haven't
gotten a report (and your version isn't bleeding edge).

That said, the "invalid contents" error message is expected. What's
confusing is that the command returned success. The error comes from
compile_regexp(), which sees that regcomp() failed. It calls
compile_regexp_failed(), which calls die(), which should end in
exit(128). But the shell sees the exit code as 0. Weird.

Your example run after shows that the correct exit code comes out of
git. What does:

  ./t7810-grep.sh -x --verbose-only=142

say? That should enable "set -x" in the shell just for that test.

-Peff

PS In my repo, the test that is failing for you is actually 145. Are you
   trying with a version of git older than v2.7.x? If so, you might
   double-check that the problem persists on the latest "master". I
   don't know of any particular fixes, but it's worth a shot.

-Peff



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]