Replace colorama dependency #958

Closed
opened 2024-12-24 03:37:12 +00:00 by witten · 2 comments
Owner

What I'd like to do and why

colorama, a borgmatic dependency for colorizing terminal output, is no longer super actively maintained and has potentially caused some borgmatic issues (#956, #827). So as part of this ticket, consider replacing it with another library to colorize borgmatic's terminal output. Potentially something actively maintained like Rich (which incidentally is an order of magnitude larger than colorama and includes a bunch of functionality we don't currently need).

Other notes / implementation ideas

No response

### What I'd like to do and why colorama, a borgmatic dependency for colorizing terminal output, is [no longer super actively maintained](https://github.com/tartley/colorama/issues/300) and has potentially caused some borgmatic issues (#956, #827). So as part of this ticket, consider replacing it with another library to colorize borgmatic's terminal output. Potentially something actively maintained like [Rich]([https://rich.readthedocs.io/en/latest/](https://github.com/Textualize/rich)) (which incidentally is an order of magnitude larger than colorama and includes a bunch of functionality we don't currently need). ### Other notes / implementation ideas _No response_
Author
Owner

Implemented in main by dropping colorama, replacing its color constants with our own and then adding a tiny bit of code to convert a color to the corresponding ANSI escape code. So to be clear, no other library was added in place of colorama; I was able to just remove it.

This will be part of the next release.

Implemented in main by dropping colorama, replacing its color constants with our own and then adding a tiny bit of code to convert a color to the corresponding ANSI escape code. So to be clear, no other library was added in place of colorama; I was able to just remove it. This will be part of the next release.
Author
Owner

Released in borgmatic 1.9.5!

Released in borgmatic 1.9.5!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#958
No description provided.