Wilfred / ag.el

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

Documentation bug #172

Open ndokos opened 3 years ago

ndokos commented 3 years ago

Thanks for reporting a bug/feature request! Make sure you include:

On my machine, the output of ag --version is: nothing - I do not have ag installed.

My Emacs version is: 28.0.50

I'm using: Fedora 33.

The Configuration doc contains a function to set the Emacs exec-path from the shell PATH. That function tries to invoke the shell using

(shell-command-to-string "$SHELL --login -i -c 'echo $PATH'")

The -i is wrong: it causes the plumbing of the output to fail with

"bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shel
... path follows ...

confusing the subsequent parsing and screwing up both the env PATH and the exec-path. Without the -i, everything works as it should.