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.

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

ANSI escape sequence handling improvement #1824

Open
dra27 opened this issue Oct 1, 2017 · 0 comments
Open

ANSI escape sequence handling improvement #1824

dra27 opened this issue Oct 1, 2017 · 0 comments
Labels

Comments

@dra27
Copy link

dra27 commented Oct 1, 2017

Gcc's error messages use some ANSI escape sequences which confuse AppVeyor's colouring.

Broken example: https://ci.appveyor.com/project/dra27/ocaml/build/1.0.130#L4639
Work-around example: https://ci.appveyor.com/project/dra27/ocaml/build/1.0.129#L4639

I'm running the script with TERM=ansi and then post-processing the output with sed. It'd be lovely if you were able to address these three shortcomings of AppVeyor's ANSI processing:

  1. For ESC[K (and other unimplemented sequences) would it be possible for AppVeyor just to omit displaying the sequence?
  2. ESC[m should be treated as ESC[0m (ECMA-48 8.3.117)
  3. In the parameters for m, 01 should be treated as 1 - i.e. ESC[01m is a bold instruction and ESC[01;31m is a bold+red instruction - this gets mis-translated in the resulting HTML output (ECMA-48 5.4.2 f)
kennytm added a commit to kennytm/ripgrep that referenced this issue Jan 11, 2018
BurntSushi pushed a commit to BurntSushi/ripgrep that referenced this issue Jan 29, 2018
Write `Ansi::reset()` using `\x1b[0m` instead of `\x1b[m`.

This works around an AppVeyor bug: appveyor/ci#1824
dra27 added a commit to dra27/opam that referenced this issue Mar 17, 2018
dra27 added a commit to dra27/opam that referenced this issue Mar 17, 2018
dra27 added a commit to dra27/opam that referenced this issue Mar 17, 2018
dra27 added a commit to dra27/opam that referenced this issue Mar 17, 2018
dra27 added a commit to dra27/opam that referenced this issue Mar 17, 2018
dra27 added a commit to dra27/opam that referenced this issue Apr 4, 2018
dra27 added a commit to dra27/opam that referenced this issue Apr 6, 2018
dra27 added a commit to dra27/opam that referenced this issue Apr 6, 2018
dra27 added a commit to dra27/opam that referenced this issue May 3, 2018
dra27 added a commit to dra27/opam that referenced this issue May 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants