Thread (1 message) 1 message, 1 author, 2016-06-15

Re: feature request: excluding files/paths from "git grep"

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:03:56

Michael J Gruber [off-list ref] writes:
Junio C Hamano venit, vidit, dixit 26.02.2015 21:59:
quoted
So that does not sound to me a summary of the discussion at all.
Well, your conditional
quoted
I do not recall its conclusion, but it it were "Yes, that is what it
means", then it might be reasonable to:

 - have "git grep" ignore paths marked with -diff by default
   (perhaps "-a" option to disable, just like GNU)

 - have "git grep" pay attention to diff.textconv and search in the
   result of textconv filter.
and Jeff's "Yes" on that condition certainly read like that to me: Make
"git grep" react to "diff", "-diff" attributes in the same way as "git
diff".
Ah, OK, I missed that flow of thought.

I read the conclusion as "_if_ using "diff" attributes is enough for
this purpose, then there is no code to be written ... but 'grep' and
'diff' may want to be different."

Once we know if they do *not* want to be different, I agree that it
may make things more consistent to turn --textconv on for binary
files when running grep.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help