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 code characters should be avoided on Windows #2634

Closed
HoloRin opened this issue Jan 21, 2020 · 4 comments
Closed

ANSI escape code characters should be avoided on Windows #2634

HoloRin opened this issue Jan 21, 2020 · 4 comments
Assignees
Milestone

Comments

@HoloRin
Copy link
Contributor

HoloRin commented Jan 21, 2020

For example, Help: gets printed as �[1mHelp�[0m:.

Screen Shot 2020-01-21 at 10 01 21 AM

This is because we are attempting to print "Help:" in bold via ANSI escape codes, which don't appear to be supported on Windows (at least for common configurations).

Apparently the necessary APIs to properly detect support for these codes isn't provided by Erlang, so my suggestion would be that we fall back to unformatted output when running on Windows.

@michaelklishin michaelklishin changed the title Extra characters in rabbitmqctl.bat output makes things harder to read ANSI escape code characters should be avoided on Windows Apr 20, 2020
@dcorbacho dcorbacho transferred this issue from rabbitmq/rabbitmq-cli Nov 17, 2020
@boldMahoney
Copy link

Hi
Is there any news about subject?

@lhoguin
Copy link
Contributor

lhoguin commented Jun 25, 2021

It is not a priority right now as we are fairly busy with more impactful changes. A PR would be most welcome though.

@boldMahoney
Copy link

boldMahoney commented Jun 25, 2021

Ok, thanks for info.

@michaelklishin
Copy link
Member

Making all functions in this module to return values as is on Windows should be all there is to do.

@lukebakken lukebakken self-assigned this Jan 10, 2024
@michaelklishin michaelklishin added this to the 3.12.13 milestone Jan 10, 2024
michaelklishin added a commit that referenced this issue Jan 10, 2024
#2634 - if ASCII disabled, remove escape codes
michaelklishin added a commit that referenced this issue Jan 10, 2024
#2634 - if ASCII disabled, remove escape codes (backport #10303)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants