Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Force input stream ANSI emulation for ConEmu.#25578

Merged
cpuguy83 merged 1 commit into
moby:mastermoby/moby:masterfrom
rwilliams:conemuCopy head branch name to clipboard
Sep 8, 2016
Merged

Force input stream ANSI emulation for ConEmu.#25578
cpuguy83 merged 1 commit into
moby:mastermoby/moby:masterfrom
rwilliams:conemuCopy head branch name to clipboard

Conversation

@rwilliams

@rwilliams rwilliams commented Aug 10, 2016

Copy link
Copy Markdown
Contributor

Forces input stream ANSI emulation for ConEmu so arrow keys etc are read on docker run commands

Fixes #25475

![alt text](http://i.imgur.com/50QN6dO.jpg kitten)

Signed-off-by: Ron Williams ron.a.williams@gmail.com

Signed-off-by: Ron Williams <ron.a.williams@gmail.com>
@rwilliams rwilliams changed the title Force input emulation for ConEmu. Force input stream ANSI emulation for ConEmu. Aug 10, 2016
@thaJeztah

Copy link
Copy Markdown
Member

ping @jstarks @jhowardmsft

@thaJeztah

Copy link
Copy Markdown
Member

ping @jstarks @jhowardmsft PTAL!

@syepes

syepes commented Aug 22, 2016

Copy link
Copy Markdown

+1

2 similar comments
@viksabnis

Copy link
Copy Markdown

+1

@JohnLozano

Copy link
Copy Markdown

+1

@eug48

eug48 commented Aug 30, 2016

Copy link
Copy Markdown

Arrow keys are also not working in ConsoleZ. I'm using Docker for Windows 1.12.0-stable.

@lowenna

lowenna commented Sep 2, 2016

Copy link
Copy Markdown
Member

LGTM. Have not tested any conEmu functionality.

@cpuguy83

cpuguy83 commented Sep 8, 2016

Copy link
Copy Markdown
Member

LGTM

@cpuguy83
cpuguy83 merged commit 28d18b8 into moby:master Sep 8, 2016
@thaJeztah thaJeztah added this to the 1.12.2 milestone Sep 26, 2016
@thaJeztah

Copy link
Copy Markdown
Member

ping @vieux I added this one to the milestone for consideration. I discussed with @jstarks, and there should be no risk in this change, and it solves an issue that people run into when running interactive containers on a Windows client.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.