Re: master 2c79a8f 2/2: Use posix_spawn if possible.

[Robert Pluim]

>>>>> On Tue, 01 Feb 2022 21:52:18 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Cc: Eli Zaretskii <eliz@gnu.org>,  Matt Armstrong <matt@rfc20.org>,
    >> p.stephani2@gmail.com,  alan@idiocy.org,  mituharu@math.s.chiba-u.ac.jp,
    >> emacs-devel@gnu.org
    >> Date: Tue, 01 Feb 2022 20:23:23 +0100
    >> 
    >> >>>>> On Tue, 1 Feb 2022 20:30:18 +0200, Saulius Menkevicius 
<sauliusmenkevicius@fastmail.com> said:
    >> 
    Saulius> Thanks Robert!
    Saulius> I applied your patch and that has actually has fixed the issue. So 
I
    Saulius> suggest this patch be applied on master.
    >> 
    >> I can do that, if Eli/Lars agree.

    Eli> Are there any downsides to the patch?  If not, I see no reasons not to
    Eli> install that.

Not as far as I can tell, but I shall test on a few more platforms
before pushing.

    >> What should we do for emacs-28?

    Eli> Nothing, because on emacs-28 we use posix_spawn only on macOS.

Right, Iʼd missed that, master it is then. We could put it in 28.2, I
guess, but I hadn't noticed any issues with subprocesses on emacs-28
on macOS.

Robert
--