unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master 70964b9: Mention `exec-path' in some process related doc strings
       [not found] ` <20200821141530.94B9D20A10@vcs0.savannah.gnu.org>
@ 2020-08-21 15:59   ` Michael Albinus
  2020-08-21 17:22     ` Gregory Heytings via Emacs development discussions.
  2020-08-21 19:04     ` Eli Zaretskii
  0 siblings, 2 replies; 5+ messages in thread
From: Michael Albinus @ 2020-08-21 15:59 UTC (permalink / raw)
  To: emacs-devel; +Cc: Gregory Heytings, Lars Ingebrigtsen

larsi@gnus.org (Lars Ingebrigtsen) writes:

Hi Lars,

[doc of make-process]

> --- a/src/process.c
> +++ b/src/process.c
>  :command COMMAND -- COMMAND is a list starting with the program file
> -name, followed by strings to give to the program as arguments.
> +name, followed by strings to give to the program as arguments.  If the
> +program file name is not an absolute file name, `make-process' will
> +look for the program file name in `exec-path' (which is a list of
> +directories).

This is not true in the remote case (default-directory is remote, and
:file-handler is non-nil). The program is not looked for; everything in
the :command list is used literally.

I don't know whether it is worth to explain this subtle difference,
though.

Best regards, Michael.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: master 70964b9: Mention `exec-path' in some process related doc strings
  2020-08-21 15:59   ` master 70964b9: Mention `exec-path' in some process related doc strings Michael Albinus
@ 2020-08-21 17:22     ` Gregory Heytings via Emacs development discussions.
  2020-08-21 17:35       ` Michael Albinus
  2020-08-21 19:04     ` Eli Zaretskii
  1 sibling, 1 reply; 5+ messages in thread
From: Gregory Heytings via Emacs development discussions. @ 2020-08-21 17:22 UTC (permalink / raw)
  To: emacs-devel; +Cc: Michael Albinus, Lars Ingebrigtsen


> larsi@gnus.org (Lars Ingebrigtsen) writes:
>
> Hi Lars,
>
> [doc of make-process]
>
>> --- a/src/process.c
>> +++ b/src/process.c
>>  :command COMMAND -- COMMAND is a list starting with the program file
>> -name, followed by strings to give to the program as arguments.
>> +name, followed by strings to give to the program as arguments.  If the
>> +program file name is not an absolute file name, `make-process' will
>> +look for the program file name in `exec-path' (which is a list of
>> +directories).
>
> This is not true in the remote case (default-directory is remote, and 
> :file-handler is non-nil). The program is not looked for; everything in 
> the :command list is used literally.
>

I'm not sure I understand what you wrote correctly, but if indeed "the 
program is not looked for", it seems to me that the program file name must 
be an absolute path, in which case the sentence added by Lars is in fact 
still true in that case.  Or is something else than PATH and exec-path 
used in that case (for example the shell's PATH environment variable on 
the remote host)?

Gregory



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: master 70964b9: Mention `exec-path' in some process related doc strings
  2020-08-21 17:22     ` Gregory Heytings via Emacs development discussions.
@ 2020-08-21 17:35       ` Michael Albinus
  0 siblings, 0 replies; 5+ messages in thread
From: Michael Albinus @ 2020-08-21 17:35 UTC (permalink / raw)
  To: Gregory Heytings; +Cc: Lars Ingebrigtsen, emacs-devel

Gregory Heytings <ghe@sdf.org> writes:

Hi Gregory,

>>> --- a/src/process.c
>>> +++ b/src/process.c
>>>  :command COMMAND -- COMMAND is a list starting with the program file
>>> -name, followed by strings to give to the program as arguments.
>>> +name, followed by strings to give to the program as arguments.  If the
>>> +program file name is not an absolute file name, `make-process' will
>>> +look for the program file name in `exec-path' (which is a list of
>>> +directories).
>>
>> This is not true in the remote case (default-directory is remote,
>> and :file-handler is non-nil). The program is not looked for;
>> everything in the :command list is used literally.
>>
>
> I'm not sure I understand what you wrote correctly, but if indeed "the
> program is not looked for", it seems to me that the program file name
> must be an absolute path, in which case the sentence added by Lars is
> in fact still true in that case.  Or is something else than PATH and
> exec-path used in that case (for example the shell's PATH environment
> variable on the remote host)?

The usual PATH settings of Tramp will be applied. That is, on the remote
host PATH is set to the value of tramp-remote-path, and a prgram (if not
given by an absolute path) must be found there.

Again, I don't believe this must be told in the docstring of
make-process. Tramp discusses it in its manual.

> Gregory

Best regards, Michael.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: master 70964b9: Mention `exec-path' in some process related doc strings
  2020-08-21 15:59   ` master 70964b9: Mention `exec-path' in some process related doc strings Michael Albinus
  2020-08-21 17:22     ` Gregory Heytings via Emacs development discussions.
@ 2020-08-21 19:04     ` Eli Zaretskii
  2020-08-22 16:18       ` Michael Albinus
  1 sibling, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2020-08-21 19:04 UTC (permalink / raw)
  To: Michael Albinus; +Cc: ghe, larsi, emacs-devel

> From: Michael Albinus <michael.albinus@gmx.de>
> Date: Fri, 21 Aug 2020 17:59:13 +0200
> Cc: Gregory Heytings <ghe@sdf.org>, Lars Ingebrigtsen <larsi@gnus.org>
> 
> > --- a/src/process.c
> > +++ b/src/process.c
> >  :command COMMAND -- COMMAND is a list starting with the program file
> > -name, followed by strings to give to the program as arguments.
> > +name, followed by strings to give to the program as arguments.  If the
> > +program file name is not an absolute file name, `make-process' will
> > +look for the program file name in `exec-path' (which is a list of
> > +directories).
> 
> This is not true in the remote case (default-directory is remote, and
> :file-handler is non-nil). The program is not looked for; everything in
> the :command list is used literally.
> 
> I don't know whether it is worth to explain this subtle difference,
> though.

Perhaps we could explain that in the ELisp manual.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: master 70964b9: Mention `exec-path' in some process related doc strings
  2020-08-21 19:04     ` Eli Zaretskii
@ 2020-08-22 16:18       ` Michael Albinus
  0 siblings, 0 replies; 5+ messages in thread
From: Michael Albinus @ 2020-08-22 16:18 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: ghe, larsi, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Michael Albinus <michael.albinus@gmx.de>
>> Date: Fri, 21 Aug 2020 17:59:13 +0200
>> Cc: Gregory Heytings <ghe@sdf.org>, Lars Ingebrigtsen <larsi@gnus.org>
>>
>> > --- a/src/process.c
>> > +++ b/src/process.c
>> >  :command COMMAND -- COMMAND is a list starting with the program file
>> > -name, followed by strings to give to the program as arguments.
>> > +name, followed by strings to give to the program as arguments.  If the
>> > +program file name is not an absolute file name, `make-process' will
>> > +look for the program file name in `exec-path' (which is a list of
>> > +directories).
>>
>> This is not true in the remote case (default-directory is remote, and
>> :file-handler is non-nil). The program is not looked for; everything in
>> the :command list is used literally.
>>
>> I don't know whether it is worth to explain this subtle difference,
>> though.
>
> Perhaps we could explain that in the ELisp manual.

`exec-path' is discussed in the Elisp manual, node "(elisp) Subprocess Creation".
It seems to be sufficient.

Best regards, Michael.



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-08-22 16:18 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20200821141528.30109.99327@vcs0.savannah.gnu.org>
     [not found] ` <20200821141530.94B9D20A10@vcs0.savannah.gnu.org>
2020-08-21 15:59   ` master 70964b9: Mention `exec-path' in some process related doc strings Michael Albinus
2020-08-21 17:22     ` Gregory Heytings via Emacs development discussions.
2020-08-21 17:35       ` Michael Albinus
2020-08-21 19:04     ` Eli Zaretskii
2020-08-22 16:18       ` Michael Albinus

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).