[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master 2c79a8f 2/2: Use posix_spawn if possible.
From: |
Philipp |
Subject: |
Re: master 2c79a8f 2/2: Use posix_spawn if possible. |
Date: |
Thu, 11 Nov 2021 18:52:22 +0100 |
> Am 10.11.2021 um 15:10 schrieb Eli Zaretskii <eliz@gnu.org>:
>
>> From: Philipp Stephani <p.stephani2@gmail.com>
>> Date: Wed, 10 Nov 2021 13:42:07 +0100
>> Cc: Alan Third <alan@idiocy.org>, YAMAMOTO Mitsuharu
>> <mituharu@math.s.chiba-u.ac.jp>,
>> Emacs developers <emacs-devel@gnu.org>
>>
>>>> I am concerned that there could be hidden issues with the branch since
>>>> it doesn't seem to have had much testing, so it's probably best going
>>>> into master, but I expect we'll see an increasing number of complaints
>>>> about performance as more people switch to macOS 12. Perhaps we should
>>>> keep it in mind as a possible update for 28.2, whenever that may be.
>>>
>>> It is fine with me to merge this to master, as a macOS-only feature,
>>> unless Philipp has reasons not to do that yet. We can reconsider
>>> backporting to emacs-28 at a later date, if and when we get enough
>>> complaints.
>>
>> Is there a specific reason why we'd want to restrict this to macOS on
>> master? If the concern is lack of test coverage, then using the branch
>> on as many OSes as possible would help address that.
>
> Since you didn't ask to merge it when I was asked about this, I
> presumed that the branch was not yet ready for prime time, from your
> POV. On macOS this seems to be urgent due to incompatible changes by
> Apple, but other platforms don't have much to gain, which is why I
> answered as I did.
I think it's ready for prime time.
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Alan Third, 2021/11/02
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2021/11/02
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Alan Third, 2021/11/02
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2021/11/02
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2021/11/10
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2021/11/10
- Re: master 2c79a8f 2/2: Use posix_spawn if possible.,
Philipp <=
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2021/11/11
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp, 2021/11/11