PCRE passes even if ggrep is not installed on OS X
See original GitHub issueI was scratching my head because some hooks were passing even though they should clearly have failed.
When running run --all-files
with the --verbose
flag, I noticed why this was happening:
[do_not_commit] Block if "DO NOT COMMIT" is found..................................................................Passed
hookid: do_not_commit
Executable `ggrep` not found
It seems like the correct behaviour would be to fail and warn user to install ggrep
?
For the record, I installed ggrep
as follow: brew install homebrew/dupes/grep
, and the hooks now fail as expected.
Issue Analytics
- State:
- Created 7 years ago
- Comments:7 (4 by maintainers)
Top Results From Across the Web
macos - grep - Stack Overflow
--with-default-names is removed from brew. I had to brew install grep to get ggrep and then do as @rymo says and do alias...
Read more >How to install and use GNU Grep in macOS - Ask Different
The answer from nohillside needs updating as follows: If grep was already installed by brew, remove grep first.
Read more >PCRE(3) Library Functions Manual
In the 16-bit library, strings are passed as vectors of unsigned 16-bit ... This implies UTF support, even if you have not explicitly...
Read more >Perl Regex Removed From Grep in Mountain Lion
As far as I know, all GNU software went to GPLv3. Literally all of it. Even things with no substantial updates since 2007....
Read more >Regular expressions in grep ( regex ) with examples - nixCraft
Regular Expressions in grep - Learn how to use regex in grep using egrep command to search for text/words in Linux, macOS or...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
And I’m wrong again. Installing ggrep now worked. Sorry about this. I need a nap.
😄 cheers!