Re: [PATCH v2 0/4] a couple of read_key_without_echo() fixes
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Mar 24, 2022, 13:29 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2203241427550.388@tvgsbejvaqbjf.bet>
- In-Reply-To
- <xmqqy211t512.fsf@gitster.g>
Hi,
On Wed, 23 Mar 2022, Junio C Hamano wrote:
Show 18 quoted lines
> Carlo Arenas <carenas@gmail.com> writes: > > > On Thu, Feb 24, 2022 at 6:30 AM Phillip Wood <phillip.wood123@gmail.com> wrote: > >> That sounds good. I've got a couple more patches based on top of these > >> which hopefully fix the remaining problems (notably the macos poll() > >> bug). I'll polish and post them next week. Once those are in I hope > >> we'll be able to enable the builtin "add -p" by default. > > > > As this topic just hit master noticed (I apologize for not doing it > > sooner) the macOS problem (tested in 10.15.7) was gone (suspect fixed > > with 1/4) and therefore enabling the builtin by default as proposed > > originally by dscho could proceed without the additional series. > > As long as it is a belated success report, nobody would mind. A > belated failure report would be a cause of sorrow and grief, but > even then, it is better to have it late than never. > > Thanks for testing.
FWIW I tried to reproduce the reported issues, but since I do not have any hardware running macOS anymore, I only had the avenue to debug via `tmate` on one of GitHub Actions' macOS agents, but the problem did not reproduce for me via SSH.
I am grateful for the great work put in by Carlo and Phillip to fix these issues.
Thanks, Dscho