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
fix pacman package state detection #51311
Conversation
This is a fix to correctly detect a package's installation status as being absent when it is not installed but another package provides it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code LGTM, but I am not a arch user, giving it a 👍 and waiting for another reviewer
shipit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you feel about those suggestions ?
(feel free to apply them to get_version too)
Co-Authored-By: greyltc <grey@christoforo.net>
Co-Authored-By: greyltc <grey@christoforo.net>
The test
|
Co-Authored-By: greyltc <grey@christoforo.net>
Co-Authored-By: greyltc <grey@christoforo.net>
Is there anything left that needs to be done here? |
ready_for_review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
shipit |
SUMMARY
Fixes #51265
ISSUE TYPE
COMPONENT NAME
pacman
ADDITIONAL INFORMATION
This is a fix to correctly detect a package's installation status as being absent when it is not installed but another package provides it.