Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

t7815-grep-binary.sh subtest 12 inexplicably passes #8

Open
me-and opened this issue Aug 6, 2014 · 8 comments
Open

t7815-grep-binary.sh subtest 12 inexplicably passes #8

me-and opened this issue Aug 6, 2014 · 8 comments

Comments

@me-and
Copy link
Member

me-and commented Aug 6, 2014

The t7008-grep-binary.sh test "git grep .fi a" passes. It's marked as an expected failure, and it does fail on my v2.0.4 Git build on CentOS.

@me-and me-and added the bug label Aug 6, 2014
@me-and
Copy link
Member Author

me-and commented Jun 18, 2015

Huh, that's interesting. If you skip the make configure && ./configure stages of the build process and just run make, the test continues to fail.

@me-and
Copy link
Member Author

me-and commented Jun 18, 2015

I thought it might be the --with-libpcre argument to ./configure which was causing the difference, but running ./configure without it produces the same result...

@me-and
Copy link
Member Author

me-and commented Jul 2, 2015

Just checked for comparison on my CentOS VM: whether or not you include the configure step, that test always fails.

Next step is probably to report this upstream.

@me-and
Copy link
Member Author

me-and commented Apr 18, 2016

Now this is odd: building using Cygport has the test unexpectedly passing, but building directly from a Git source tree including the Cygwin patches has the test failing!

@me-and
Copy link
Member Author

me-and commented Apr 18, 2016

Ah, here we go: the Cygport compilation runs make configure and ./configure --htmldir=<dir> --mandir=<dir> --with-libpcre before make. Presumably at least part of that is what's making the difference here.

@me-and
Copy link
Member Author

me-and commented Apr 18, 2016

… which is something I'd already discovered in my notes above. OH WELL.

@me-and
Copy link
Member Author

me-and commented Apr 18, 2016

@me-and me-and added the blocked label Apr 18, 2016
@me-and me-and removed the blocked label Aug 25, 2017
@me-and
Copy link
Member Author

me-and commented Apr 30, 2020

Test is now t7815, still passing at 811da4d (v2.26.1) when marked as an expected failure.

@me-and me-and changed the title t7008-grep-binary.sh subtest 12 inexplicably passes t7815-grep-binary.sh subtest 12 inexplicably passes Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant