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

Add colour to checkupdates output #80

Closed
wants to merge 1 commit into from
Closed

Conversation

wmemcpy
Copy link
Contributor

@wmemcpy wmemcpy commented Dec 31, 2023

Improved 'checkupdates' output. I don't know if it's normal that there's no colour difference between the two versions of the packages, but I found it sad when compared with the AUR manager, which has colours everywhere .

Example:
20231231-160527Z

@Antiz96
Copy link
Owner

Antiz96 commented Dec 31, 2023

Hello,

This is actually an upstream checkupdates bug.
Indeed, the output is supposed to be colored by default but it does not work for some reason.

The issue has been raised on upstream side here.
I'll see if I can find a fix to propose on pacman-contrib side.

Thanks a lot for your effort, but I'd prefer that to be properly fixed on upstream side rather than implement a workaround on Arch Update side.

@Antiz96 Antiz96 closed this Dec 31, 2023
@wmemcpy
Copy link
Contributor Author

wmemcpy commented Dec 31, 2023

No problem i was expecting such a response and that's quite normal. Thank you for your message ! :D

@Antiz96
Copy link
Owner

Antiz96 commented Dec 31, 2023

No problem! ;)

Actually, I created an issue (linked to a milestone for v1.10.0) to include a lightly colorized output for Arch Update and I mentioned that it would be great that the upstream checkupdates bug could be fixed before the release to get a consistent colorized output accros the whole script 😉

@Antiz96
Copy link
Owner

Antiz96 commented Jan 5, 2024

No problem i was expecting such a response and that's quite normal. Thank you for your message ! :D

I found the checkupdates upstream bug that prevents colors from being shown, I proposed a fix:
https://gitlab.archlinux.org/pacman/pacman-contrib/-/merge_requests/42

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 this pull request may close these issues.

2 participants