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

updating lscolors #118

Closed
plugwash opened this issue Jun 24, 2022 · 1 comment · Fixed by #119
Closed

updating lscolors #118

plugwash opened this issue Jun 24, 2022 · 1 comment · Fixed by #119

Comments

@plugwash
Copy link

I just fixed/updated the rust-tre package in Debian Unstable. Since in Debian we try to avoid multiple versions of the same crate this involved patching the lscolors dependency to version 0.10.

After patching the dependency the build failed due to new "bright" colors being added to lscolors::style::Color (aka lscolors::Color), these do not appear to have any direct equivilents in termcolor::Color.

lscolors offers a converter to convert it's values to ansi-term colors, in that converter the new colors are converted to numbered "ansi 256" colors, I did the same for the converter in tre-command.

A patch can be found at https://salsa.debian.org/rust-team/debcargo-conf/-/blob/master/src/tre-command/debian/patches/lscolors-0.10.diff

dduan added a commit that referenced this issue Jun 24, 2022
@dduan
Copy link
Owner

dduan commented Jun 24, 2022

Thank you so much for helping with the packaging and coming up with the patch! I'm incorporating the patch in #119.

dduan added a commit that referenced this issue Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants