search

dajva / rg.el

Emacs search tool based on ripgrep
https://rgel.readthedocs.io
GNU General Public License v3.0
484 stars 41 forks source link

use (executable-find "rg") instead of assuming "rg" is in path #31

Closed nicolamontecchio closed 6 years ago

nicolamontecchio commented 6 years ago

in some setups rg isn't found because it's not in the environment's $PATH

this checks in emacs' exec-path

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.05%) to 83.106% when pulling 81e6fd9193ef61bbda7b27f6c624af12cb9e0def on nicolamontecchio:master into 26413536ee71ac6ee07f193b94e60e544d800655 on dajva:master.

dajva commented 6 years ago

Thanks, there is another usage of this in rg-list-builtin-type-aliases that would need the same fix. Do you mind fixing that as well while you're at it?

nicolamontecchio commented 6 years ago

wow that was fast :) let me take a look ...

nicolamontecchio commented 6 years ago

i fixed rg-list-builtin-type-aliases and squashed the fix into the same commit

dajva commented 6 years ago

Thanks