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

pacutils: Add PrettyProgressBar Config Option #297

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

1Naim
Copy link
Member

@1Naim 1Naim commented Aug 20, 2024

The pacman PKGBUILD we provide has a patch that enables a unicode progress bar when built with the
necessary options. When used with upstream pacutils however, pacconf will print out warnings about
"PrettyProgressBar" config option not existing because of the way it parses configs manually.

For QoL reasons, I have made a patch to add that config option to pacconf directly so it doesn't complain
when used with PrettyProgressBar'ed pacman

@1Naim 1Naim requested a review from vnepogodin August 20, 2024 11:11
The pacman PKGBUILD we provide has a patch that enables a unicode progress bar when built with the
necessary options. When used with upstream pacutils however, pacconf will print out warnings about
"PrettyProgressBar" config option not existing because of the way it parses configs manually.

For QoL reasons, I have made a patch to add that config option to pacconf directly so it doesn't complain
when used with PrettyProgressBar'ed pacman

Signed-off-by: Eric Naim <[email protected]>
@1Naim 1Naim force-pushed the feat/pacutils-prettyprogressbar branch from 513d48f to b7ac0aa Compare August 20, 2024 11:21
@ptr1337
Copy link
Member

ptr1337 commented Aug 22, 2024

Lets wait for pacman 7 to merge this.

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