Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Disable terminal colorization if TERM=dumb is set #320

Merged
merged 2 commits into from
May 9, 2022

Conversation

Urhengulas
Copy link
Member

This PR fixes the problem @fhars described in #318:

The current version of the colored crate ignores the capabilities of the terminal (see colored-rs/colored#108), which makes the outupt of probe-run unparseable in e.g. emacs compilation mode.

It also supersedes #318 as a fix to the problem. The reason for not going with #318 is that it adds the terminfo crate as a dependency which seems unmaintained and with many outdated dependencies.

@jonas-schievink
Copy link
Contributor

bors r+

@bors
Copy link
Contributor

bors bot commented May 9, 2022

Build succeeded:

@bors bors bot merged commit 4c71afb into main May 9, 2022
@bors bors bot deleted the disable-term-colorization branch May 9, 2022 13:27
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants