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

feat(ci): Replace rpm-ostree with dnf5 during build #1905

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

KyleGospo
Copy link
Member

No description provided.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 21, 2024
@KyleGospo KyleGospo changed the title Replace rpm-ostree with dnf5 during build feat(ci): Replace rpm-ostree with dnf5 during build Nov 21, 2024
@KyleGospo KyleGospo marked this pull request as draft November 21, 2024 07:42
@KyleGospo KyleGospo marked this pull request as ready for review November 21, 2024 07:42
@dosubot dosubot bot added the enhancement New feature or request label Nov 21, 2024
Copy link
Member

@HikariKnight HikariKnight left a comment

Choose a reason for hiding this comment

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

so much cleaner!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 22, 2024
Copy link

@LukeShortCloud LukeShortCloud left a comment

Choose a reason for hiding this comment

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

I haven't personally encountered any of these problems in PlaytronOS, but Bluefin has run into some odd issues with DNF 5 over in their (new) related PR: ublue-os/bluefin#1954

@@ -37,125 +37,79 @@ COPY system_files/desktop/shared system_files/desktop/${BASE_IMAGE_NAME} /

# Update packages that commonly cause build issues
RUN --mount=type=cache,dst=/var/cache/rpm-ostree \

Choose a reason for hiding this comment

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

We've been using DNF 5 in PlaytronOS for a few months now and, yes, it's amazing!

One of the great things is that it takes care of dependencies. It's okay if a package is slightly out-of-date, DNF will automatically deal with that by updating the package(s) as necessary. rpm-ostree had a hard time because it didn't know how to update base packages. You should be able to remove this entire "Update packages that commonly cause build issues" block.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants