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

chore(build): move to dnf5 instead of rpm-ostree #1953

Closed
wants to merge 1 commit into from

Conversation

tulilirockz
Copy link
Collaborator

@tulilirockz tulilirockz commented Nov 21, 2024

This PR still needs cliwrap to be disabled in main to actually function properly (e.g.: use dnf instead of dnf5, and a weird systemctl issue in the middle too).

I tested this out on a VM and here is the output from ujust device-info and rpm -qa:

ujust-device-info.txt (from the VM)
rpm-qa-vm.txt (from the VM)
rpm-qa-latest.txt (current rpm -qa)

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. dx Developer Experience Image specific labels Nov 21, 2024
@tulilirockz
Copy link
Collaborator Author

tulilirockz commented Nov 21, 2024

Funnily enough the builds failed because of the new kernel getting released

@tulilirockz tulilirockz mentioned this pull request Nov 21, 2024
5 tasks
@tulilirockz tulilirockz marked this pull request as draft November 21, 2024 05:47
@tulilirockz
Copy link
Collaborator Author

tulilirockz commented Nov 21, 2024

After diffing the rpm -qa files, the dnf5 build seems to install development dependencies for most of the packages? like, some rpm-build macros, kernel-devel (...). Super weird - Otherwise, it installed all of the same dependencies just fine.

KyleGospo added a commit to ublue-os/bazzite that referenced this pull request Nov 21, 2024
@tulilirockz tulilirockz marked this pull request as ready for review November 21, 2024 06:12
@tulilirockz tulilirockz marked this pull request as draft November 21, 2024 06:40
@tulilirockz
Copy link
Collaborator Author

Bsherman PR'd his progress! This one is obsolete now :p

#1954

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dx Developer Experience Image specific size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant