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

Fixes for RHEL-15902 #2085

Merged

Conversation

ppisar
Copy link
Contributor

@ppisar ppisar commented Apr 23, 2024

Upstream commit: a4d815e
Upstream commit: 824a95e
Upstream commit: a6d4cd7
Upstream commit: 929d913
Tests: rpm-software-management/ci-dnf-stack#1496
Resolves: https://issues.redhat.com/browse/RHEL-15902

@j-mracek, please review this rhel-9.5 backport of your commits resolving RHEL-15902.

Upstream commit: a4d815e

The original code took the first item from all remaining packages.
It means a random reason and use it to keep installonly package reason.

Related: https://issues.redhat.com/browse/RHEL-15902
Closes: rpm-software-management#2061
Upstream commit: 824a95e

Installonly packages are handled by autoremove command in a similar
way like other package.

Resolves: https://issues.redhat.com/browse/RHEL-15902
Upstream commit: a6d4cd7

Leaf packages term is not define in documentation and it even
represent different set of packages that are showed by leaves command.

Related: https://issues.redhat.com/browse/RHELDOCS-17711
Resolves: https://issues.redhat.com/browse/RHEL-15902
Upstream commit: 929d913

The described behavior differs from current behavior therefore
it might create a confusion.

Resolves: https://issues.redhat.com/browse/RHEL-15902
@ppisar
Copy link
Contributor Author

ppisar commented May 2, 2024

Any progress?

@j-mracek
Copy link
Contributor

j-mracek commented May 3, 2024

My apology, I will make a review on Monday.

@j-mracek
Copy link
Contributor

j-mracek commented May 6, 2024

LGTM

@j-mracek j-mracek merged commit c09865a into rpm-software-management:rhel-9.5 May 6, 2024
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants