New formula: git-diff-grep

[bilke]

This tool will run grep in the last n commit diffs that you specify.

[mistydemeo]

The => syntax is only necessary when you're renaming the file. If the name is staying the same, you can write it shorter as bin.install "git-diff-grep"

[jacknagel]

Perhaps other maintainers disagree, but I am generally -1 on "git wrapper script foo" as candidates for core, unless there is user demand.

Obviously I haven't used this, but how does it differ from things like git log with -S<string> or -G<regex>?

[jacknagel]

This is wrong; brew test invokes this test method.

[mistydemeo]

While I can see the convenience in tracking version information, this kind of simple script doesn't have any complex installation and could be curled into /usr/local by the user about as easily as it could be brew installed. User demand sounds like a good metric to me.

In this case there's no stable version and HEAD is changing rapidly so it's not the best formula candidate in any case.

[adamv]

-1 on this as it stands.

There are a couple options though.

• The project could include its own install formula for use with brew install <url>
• The project could include its own diy installer script, like: https://github.com/adamv/toggle-extra/blob/master/installer/diy

[Sharpie]

We do have some script wrappers and it is nice to easily manage installation/uninstallation.

However, -1 on another HEAD-only wrapper.

I'll be downvoting these in general once multi-repo support is finalized.

[jacknagel]

Closing; head-only wrappers are usually a non-starter these days.