From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org, Johannes Sixt <j6t@kdbg.org>,
Jeff Hostetler <jeffhost@microsoft.com>
Subject: Re: [PATCH] mingw: make stderr unbuffered again
Date: Tue, 14 Feb 2017 15:47:05 +0100 (CET) [thread overview]
Message-ID: <alpine.DEB.2.20.1702141545380.3496@virtualbox> (raw)
In-Reply-To: <xmqqlgt9btrv.fsf@gitster.mtv.corp.google.com>
Hi Junio,
On Mon, 13 Feb 2017, Junio C Hamano wrote:
> Johannes Schindelin <johannes.schindelin@gmx.de> writes:
>
> > When removing the hack for isatty(), we actually removed more than just
> > an isatty() hack: we removed the hack where internal data structures of
> > the MSVC runtime are modified in order to redirect stdout/stderr.
> >
> > Instead of using that hack (that does not work with newer versions of
> > the runtime, anyway), we replaced it by reopening the respective file
> > descriptors.
> >
> > What we forgot was to mark stderr as unbuffered again.
> >
> > Reported by Hannes Sixt. Fixed with Jeff Hostetler's assistance.
> >
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> > Published-As: https://github.com/dscho/git/releases/tag/mingw-unbuffered-stderr-v1
> > Fetch-It-Via: git fetch https://github.com/dscho/git mingw-unbuffered-stderr-v1
>
> OK. Should this go directly to 'master', as the isatty thing is
> already in?
From my point of view, it is not crucial. The next Git for Windows version
will have it, of course, and Hannes is always running with his set of
patches, he can easily cherry-pick this one, too.
Ciao,
Johannes
P.S.: Could you please cut the remainder of the mail that you are not
responding to? Thanks.
next prev parent reply other threads:[~2017-02-14 14:47 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-13 22:34 [PATCH] mingw: make stderr unbuffered again Johannes Schindelin
2017-02-13 22:39 ` Junio C Hamano
2017-02-14 14:47 ` Johannes Schindelin [this message]
2017-02-14 18:45 ` Johannes Sixt
2017-02-14 18:58 ` Junio C Hamano
2017-02-15 12:32 ` Johannes Schindelin
2017-02-15 20:45 ` Johannes Sixt
2017-02-16 17:10 ` Johannes Schindelin
2017-02-16 17:55 ` Johannes Sixt
2017-02-16 18:01 ` Junio C Hamano
2017-02-14 18:48 ` Junio C Hamano
2017-02-15 12:48 ` Johannes Schindelin
2017-02-15 22:22 ` Junio C Hamano
2017-02-15 23:34 ` Junio C Hamano
2017-02-17 16:00 ` Johannes Schindelin
2017-02-17 23:49 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.20.1702141545380.3496@virtualbox \
--to=johannes.schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=j6t@kdbg.org \
--cc=jeffhost@microsoft.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox