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

[Aaron Jensen]

On Fri, Nov 12, 2021 at 5:05 PM Alan Third <alan@idiocy.org> wrote:
>
> On Fri, Nov 12, 2021 at 08:42:57AM -0500, Aaron Jensen wrote:
> > On Fri, Nov 12, 2021 at 6:48 AM Philipp <p.stephani2@gmail.com> wrote:
> >
> > > For now I did the second part.  There's a small slowdown on GNU/Linux 
> > > (because posix_spawn performs a few additional safety checks), and a 
> > > large speedup on macOS, see the attached plot
> >
> > Great, thanks. I've added the patch to the homebrew installed
> > emacs-plus for version 28, which is one of the more popular ways of
> > installing Emacs on macOS. I believe a future version of emacs-mac
> > will have the patch applied too, so there's not necessarily a rush to
> > patch 28 and we will be getting some testing.
>
> I think emacsformacosx.com builds dwarf pretty much any other macOS
> builds in terms of user-base, and IIRC they're totally vanilla, so we
> still might want to install this on the release branch only for macOS.

Ah, I didn't think about that one, good point. Then yeah, that seems
like a good idea to install to emacs-28. If that's done, please let me
know so I can remove the patch from emacs-plus.

Thanks,

Aaron