Wilfred / ag.el

An Emacs frontend to The Silver Searcher
http://agel.readthedocs.org/en/latest/
525 stars 61 forks source link

Use variable `insert-directory-program` instead of hard coding `ls` #133

Closed arthurl closed 7 years ago

arthurl commented 7 years ago

Not an emacs expert, but would it be better to use the variable insert-directory-program defined in files.el (part of emacs itself)?

My use case for this is that I customise insert-directory-program to something other than ls, namely gls (i.e. GNU ls).

Thanks!

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.07%) to 19.118% when pulling 2e18cd2ea858b44d31efbe213606d0446254169b on arthurl:master into b390f212947546498a098f2ceb16033709ff32fd on Wilfred:master.

Wilfred commented 7 years ago

Seems reasonable :)

I've written a fair bit of elisp but never seen insert-directory-program before! I agree it makes sense here.