Migrate from the silver searcher to ripgrep

[stevenbarragan]

ripgrep is faster than {grep, ag, git grep, ucg, pt, sift}.

check this blog post.

[lfilho]

@skwp this seems super nice. Any thoughts?

@stevenbarragan the branch has conflicts now :/

[stevenbarragan]

@lfilho merge conflicts fixed.

[stevenbarragan]

@skwp any thoughts about this PR? or maybe you need some help maintaining the repo? I've seen some PR's stay open for ever

[skwp]

Sorry guys, I've really been unable to do a good job maintaining. I'm revisiting the PRs

[skwp]

My main issue with this it will break a lot of people who don't have it brew installed and just pull on the repo. Why not provide this as a new mapping ,rg?

[skwp]

i would do a fallback to ag here

[stevenbarragan]

@skwp I added the fall back to ag, and I added the mapping ,rg in here.

[templeman]

Also worth noting that ag.vim has been deprecated: rking/ag.vim#124 (comment).

[lfilho]

Speaking of backwards compatibilities, should we try to avoid key mappings too specific for plugins? For example, instead of ,ag and ,rg, why not have only something like ,fa (find in all) or ,fe (find everywhere) or whatever?

This way we would be less dependent and coupled with which plugins provide certain features...

[skwp]

We do have a generic mapping of ,gg also (go grep, git grep, or whatever mnemonic you like there)

[skwp]

Sorry guys I had to revert, it turns out ripgrep doesn't install cleanly on my machine. It tried to compile but failed. Until I understand that I dont' want to break other people

[skwp]

configure: error: in `/private/tmp/rust-20170530-4679-511zir/rustc-1.17.0-src/build/x86_64-apple-darwin/native/jemalloc':
configure: error: C compiler cannot create executables
See `config.log' for more details

Build failed, waiting for other jobs to finish...
error: build failed


command did not execute successfully: "/private/tmp/rust-20170530-4679-511zir/rustc-1.17.0-src/build/x86_64-apple-darwin/stage0/bin/cargo" "build" "-j" "8" "--target" "x86_64-apple-darwin" "--release" "--features" "panic-unwind jemalloc backtrace" "--manifest-path" "/private/tmp/rust-20170530-4679-511zir/rustc-1.17.0-src/src/libstd/Cargo.toml"
expected success, got: exit code: 101

[lfilho]

That's weird. I installed via brew and it works fine.

[stevenbarragan]

I'm sorry about that, I didn't have any issue either, I'll take a look.

[lfilho]

@skwp sanity check: have you update your brew recently?

I'm running brew 1.2.1, OSX El Capitan and zsh 5.3.1

[lfilho]

@stevenbarragan and @skwp did you have a chance to have a look? I think this PR will greatly speed things for everyone in #661