[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master 2c79a8f 2/2: Use posix_spawn if possible.
From: |
Eli Zaretskii |
Subject: |
Re: master 2c79a8f 2/2: Use posix_spawn if possible. |
Date: |
Fri, 25 Dec 2020 15:16:54 +0200 |
I'm sorry, but I reverted this commit and the previous one. They
broke the MinGW build in fundamental ways, and I cannot afford having
the master branch broken so frequently, definitely not on weekends,
which are the only time I can work seriously on Emacs.
If you cannot make sure Emacs still builds on all supported platforms,
please either post your patches first, or push them to a scratch
branch, and ask people who have access to those platforms to test it
before the changes land on master.
Btw, regarding use of posix_spawn, I'd expect a discussion before we
make such a change. AFAIU it is not a trivial decision, as
posix_spawn has its down sides, and therefore is not necessarily the
best API for running sub-processes on every supported platform, even
if you consider only the Posix ones. We should consider the
advantages and disadvantages before we make the decision.
Thanks.
- Re: master 2c79a8f 2/2: Use posix_spawn if possible.,
Eli Zaretskii <=
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2020/12/26
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2020/12/26
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2020/12/26
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2020/12/29
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2020/12/31
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2020/12/31
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2020/12/31
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2020/12/31
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Philipp Stephani, 2020/12/31
- Re: master 2c79a8f 2/2: Use posix_spawn if possible., Eli Zaretskii, 2020/12/31