DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 7FA8D389683A
From: ASSI <Stromeko@nexgo.de>
To: cygwin@cygwin.com
Subject: Re: [Bug] tmux 3.2-0
References: <announce.CAALQ5rm2xisVPE8WqJOWVxs4o9Qr0Cpcnqx=VybTeyFD005kxA@mail.gmail.com>
 <87mtsx6six.fsf@Rainer.invalid> <87pmxe1rmk.fsf_-_@Rainer.invalid>
 <CAALQ5r=xF_y9cgMn2gRWR4tmvK4TQN-NHYSVx1CepW8NVp=yHg@mail.gmail.com>
 <871r9thlw0.fsf@Otto.invalid>
 <CAALQ5r=Zx3krUx_qw-c8-_4aEsEtS63kahCZo1GzfwXpDz2B0w@mail.gmail.com>
 <877djebwhi.fsf@Rainer.invalid>
 <CAALQ5r=kZ=bJV9S7fMzNH+SSQQXwKAfvxyd17gMY_TW3Jo3tAQ@mail.gmail.com>
 <CAALQ5r=3ih5kp_QQK8OPm-qE835yU88RJ2DsmTutembLUgTEZg@mail.gmail.com>
Date: Sat, 05 Jun 2021 07:33:27 +0200
In-Reply-To: <CAALQ5r=3ih5kp_QQK8OPm-qE835yU88RJ2DsmTutembLUgTEZg@mail.gmail.com>
 (Michael Wild via Cygwin's message of "Wed, 2 Jun 2021 19:38:48
 +0200")
Message-ID: <87lf7og9js.fsf@Otto.invalid>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
MIME-Version: 1.0
Precedence: list
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cygwin-bounces+archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie.com@cygwin.com>

Michael Wild via Cygwin writes:
> I just uploaded a patched test version 3.2-1 (yeah, I know, I'm off by
> one with the release number...). Can you give it a go and see whether
> the mosh test suite runs through with it?

No it doesn't and I don't even understand why anybody thinks that would
fix the issue.  The output is supposed to go to stdout (that's the sole
purpose of control mode) and it's the fd for the TTY that gets
duplicated there apparently (why that only exists for Cygwin if I'm
reading it correctly is another mystery).  So indeed the output now appears on the
terminal, but that's not where it should go when stdout gets redirected
elsewhere.

It _might_ be possible for this particular case that one could open
/proc/${clientpid}/fd/1 and have it do the right thing.  If so, then the
call just above that clobbers stdin with the ttyname would probably be
better off as /proc/${clientpid}/fd/0.

Otherwise with the code changes as implemented you'd need to wait for
the AF_UNIX branch by Ken Brown to finally land before tmux works as
intended again.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptation for Waldorf rackAttack V1.04R1:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple