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

system-upgrade: backwards-compat offline-{upgrade,distrosync} #1344

Conversation

evan-goode
Copy link
Member

Make dnf5 offline-distrosync and dnf5 offline-upgrade backwards-compatible with DNF 4. There didn't seem to be a way to do this with aliases alone since we need dnf5 offline-distrosync and dnf5 offline-distrosync download to be aliased to different commands (dnf5 offline and dnf5 distro-sync, respectively). We also want dnf5 offline-distrosync --help to show the help for dnf5 offline.

Resolves #1343.

Marked draft until I have a ci-dnf-stack PR ready.

@evan-goode
Copy link
Member Author

I added tests for these commands to this existing ci-dnf-stack PR: rpm-software-management/ci-dnf-stack#1475. Marking as RFR.

@m-blaha m-blaha self-assigned this Mar 27, 2024
@m-blaha
Copy link
Member

m-blaha commented Mar 27, 2024

@evan-goode rebase the PR please

Copy link
Member

@m-blaha m-blaha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other then the missing aliases descriptions it looks good.

@evan-goode evan-goode requested a review from m-blaha March 27, 2024 16:51
Make `dnf5 offline-distrosync` and `dnf5 offline-upgrade` backwards-compatible with
DNF 4. There didn't seem to be a way to do this with aliases alone since
we need `dnf5 offline-distrosync` and `dnf5 offline-distrosync download`
to be aliased to different commands (`dnf5 offline` and `dnf5
distro-sync`, respectively). We also want `dnf5 offline-distrosync --help`
to show the help for `dnf5 offline`.
@evan-goode evan-goode force-pushed the evan-goode/offline-transactions branch from be156f9 to b07f425 Compare March 27, 2024 16:58
Copy link
Member

@m-blaha m-blaha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@m-blaha m-blaha added this pull request to the merge queue Mar 28, 2024
Merged via the queue into rpm-software-management:main with commit c1565a0 Mar 28, 2024
15 of 18 checks passed
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.

system-upgrade: Add offline-distrosync, offline-upgrade subcommand aliases
2 participants