-
Notifications
You must be signed in to change notification settings - Fork 40
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
Make default color mapping support OSX Clang #23
Comments
This file is not perfect though :) I think it has problems with bold/unbold. I can upload a sample clang output if that is useful. |
If someone is interested, I made a fork without regular expressions but with a really simple ANSI parser. It solves problem with clang output. You can find it here https://github.com/PurplePowder/SublimeANSI |
It works pretty well out of the box. This also looks like way less of a headache to understand and configure. I had user settings already, and it puked on them, even if I emptied the file and saved it. Is there something you can do to make error recovery on a bad user (not default) settings file slightly better? It seems to default to an "all grey" theme when it dies on user configuration, instead of falling back. Of course, once I copied the default settings to the user settings path it worked :) I have about 3 feature requests now, but I think it makes more sense to focus on verifying if your branch is stable before worrying about that. |
Fixed. Now when plugin founds out that some required color isn't described in settings, it simply doesn't colorize it, so default color is used |
This is still an issue. |
You can file a PR or use https://packagecontrol.io/packages/Terminus instead nowadays. |
Clang on OSX outputs escape sequences that look like this:
I don't know escape sequences well enough to know if clang is just flat out wrong, or if it is expecting more complete ANSI support. Either way, the default settings only match color escape sequences with two values.
Here's a mapping that seemed to work for me:
The text was updated successfully, but these errors were encountered: