Thread (21 messages) 21 messages, 4 authors, 2020-09-10

Re: [PATCH 1/4] Use $(SHELL_PATH) instead of sh in Makefile.

From: Eric Sunshine <hidden>
Date: 2020-08-06 04:10:24

On Wed, Aug 5, 2020 at 10:14 PM brian m. carlson
[off-list ref] wrote:
On 2020-08-06 at 01:05:01, lufia via GitGitGadget wrote:
quoted
In the not POSIX environment, like Plan 9, sh might not be looked up
in the directories named by the $PATH.
I think Git's editor handling assumes that sh is somewhere in the PATH,
so it might be fine for us to just ask the user to adjust PATH
appropriately before running make.  I don't have a strong preference; if
this works on a standard Unix machine, which it looks like it should
(although I haven't tested), I'm fine with it.
This does, however, have a bit of a chicken-and-egg feel to it. The
results of the "uname_FOO" assignments in config.mak.uname are
consulted later in the file to _assign_ a value to SHELL_PATH on a
number of platforms. So, making the "uname_FOO" assignments themselves
depend upon SHELL_PATH is rather circular and confusing.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help