Visitar URL original
Second attempt at fixing msys terminal width. by ehuss · Pull Request #6122 · rust-lang/cargo · GitHub
Skip to content

Second attempt at fixing msys terminal width. - #6122

Merged
bors merged 1 commit into
rust-lang:masterfrom
ehuss:msys-width
Oct 3, 2018
Merged

bors merged 1 commit into
rust-lang:masterfrom
ehuss:msys-width

Conversation

@ehuss

@ehuss ehuss commented Oct 3, 2018

Copy link
Copy Markdown
Contributor

Lock the max width on msys-based terminals to 60. I tried a lot of different
things, but I was unable to find a way to detect the correct width in mintty.
Unfortunately this means that terminals that work correctly like ConEmu will
also be capped at 60. C'est la vie.

Of course this does not affect cmd, powershell, etc.

Fixes #6108

@rust-highfive

Copy link
Copy Markdown

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

Lock the max width on msys-based terminals to 60. I tried a lot of different
things, but I was unable to find a way to detect the correct width in mintty.
Unfortunately this means that terminals that work correctly like ConEmu will
also be capped at 60. C'est la vie.

Of course this does not affect cmd, powershell, etc.
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Oct 3, 2018

Copy link
Copy Markdown
Contributor

📌 Commit 0503e34 has been approved by alexcrichton

@bors

bors commented Oct 3, 2018

Copy link
Copy Markdown
Contributor

⌛ Testing commit 0503e34 with merge cca192a...

bors added a commit that referenced this pull request Oct 3, 2018
Second attempt at fixing msys terminal width.

Lock the max width on msys-based terminals to 60. I tried a lot of different
things, but I was unable to find a way to detect the correct width in mintty.
Unfortunately this means that terminals that work correctly like ConEmu will
also be capped at 60. C'est la vie.

Of course this does not affect cmd, powershell, etc.

Fixes #6108
@bors

bors commented Oct 3, 2018

Copy link
Copy Markdown
Contributor

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing cca192a to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New cargo output is broken in MSYS2

4 participants