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

PPA install is broken #36

Closed
zoomosis opened this issue May 5, 2022 · 0 comments
Closed

PPA install is broken #36

zoomosis opened this issue May 5, 2022 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@zoomosis
Copy link

zoomosis commented May 5, 2022

Attempting to use "sudo deb-get install firefox-esr", the PPA's repo is added but the package is never installed.

I think the logic in install_ppa() is broken:

if ! dpkg -l "${APP}" >/dev/null 2>&1; then
     eula
     apt-get -q -y install "${APP}"

Here "dpkg -l" will return non-zero if a package is missing, but the if statement is checking for zero.

Even if this worked there's still the issue of "dpkg -l" returning zero for packages with only config-files installed. So if you had firefox-esr installed previously then uninstalled it with "apt remove", "deb-get install" won't reinstall it.

Rather than "dpkg -l" I suspect it would be more reliable to parse the output of "dpkg-query -s firefox-esr" and not rely on the return code at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants