[Github-comments] [geany/geany] Make the shell used by Geany to run various commands configurable (#2363)

elextr notifications at xxxxx
Tue Nov 5 23:14:13 UTC 2019


In relation to [kugel comment](https://github.com/geany/geany/issues/2344#issuecomment-550059777)

> We use /bin/sh as a helper to spawn processes, not as a tool to read further configuration.

But whichever shell is used to spawn processes has to have PATH set up correctly so it can find the command to run.  

If Geany is started from a terminal then that will have happened when the shell in the terminal read its startup files and Geany will inherit that and the spawn shell will inherit it, all good.  

But if Geany is started from a window menu it won't have inherited any PATH, so the spawn shell won't be able to find the command to run (and this has been the subject of various issues) so the spawn shell will have to set PATH by reading its configuration files rather than inheriting it.

On mac now zsh is the standard user shell, the user will set their PATH variable in zsh startup files, not in any of the `/bin/sh` startup files, so Geany spawning commands won't have the correct path if it uses `/bin/sh` or the user has to set up two sets of startup files.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2363#issuecomment-550066319
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20191105/3960429b/attachment-0001.html>


More information about the Github-comments mailing list