Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Route "Device Status Report" responses correctly #72

Merged
merged 2 commits into from
Dec 10, 2021
Merged

Route "Device Status Report" responses correctly #72

merged 2 commits into from
Dec 10, 2021

Commits on Dec 10, 2021

  1. fixup! Rename dll from cygwin to msys 

    These two instances of pty pipes have been forgotten somewhere along the
lines.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
    @dscho
    dscho committed Dec 10, 2021
    Configuration menu
    Copy the full SHA
    c81336c View commit details
    Browse the repository at this point in the history

  2. Cygwin: pty: fix bug in reduced input transfer 

    When launching an MSYS2 process via a non-MSYS2 process, the response
for, say, a CSI [ 6n (Device Status Report) sent from that MSYS2 process
was sometimes not sent (or sent only partially) to the process but to
the terminal instead.

This regression was introduced in f206417 (Cygwin: pty: Reduce
unecessary input transfer., 2021-02-11).

While it is not yet completely clear why, this patch fixes this. The
explanation is left for the commit message that Takashi will craft for
the Cygwin project, but we will take this patch early into the MSYS2/Git
for Windows projects.

This addresses git-for-windows/git#3579
    @tyan0 @dscho
    tyan0 authored and dscho committed Dec 10, 2021
    Configuration menu
    Copy the full SHA
    8c3e53c View commit details
    Browse the repository at this point in the history