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 flag to either consider or ignore EVR for extras #1676

Open
wants to merge 1 commit into
base: dnf-4-master
Choose a base branch
from

Conversation

PaulSD
Copy link

@PaulSD PaulSD commented Oct 24, 2024

The "extras" filter originally considered NEVRA, but was changed to ignore EVR and consider only NA: https://bugzilla.redhat.com/show_bug.cgi?id=1684517

We occasionally use third party repos that have RPMs with names that overlap with RPMs in standard repos, but with non-overlapping version numbers (eg. to get newer versions of software than is available in the standard repos.) If one of these third party repos is retired, we want to make sure that we remove any RPMs that came from it (either by downgrading to the standard RPMs or by switching to a replacement third party repo).

For many years, we used yum list extras or dnf list --extras to detect and handle orphaned RPMs like these. However, we recently encountered one of these repo retirements and noticed that dnf list --extras was no longer working as expected (it was no longer reporting any RPMs with names that overlap RPMs in other repos).

We would like to have some way to make dnf report these version-mismatch orphans. Since both the original and new behavior seem to have legitimate use cases, a flag to select the relevant behavior seems appropriate.

My intent (after this PR is merged) is to add another option to dnf (maybe dnf list --extra-vers?) to expose this flag to users.

The extras filter originally considered NEVRA, but was changed to ignore
EVR and consider only NA: https://bugzilla.redhat.com/show_bug.cgi?id=1684517

Since both the original and new behavior are required for different use
cases, add a flag to support either behavior.
@PaulSD
Copy link
Author

PaulSD commented Oct 24, 2024

(The test failures do not look related to any of my changes. Looks like some dependency issues?)

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.

1 participant