-
Notifications
You must be signed in to change notification settings - Fork 50
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
manifest: extract tomlPkgsFor() helper and fix ordering #862
Conversation
Let's bump the osbuild commit ID to |
071d77c
to
1747f6d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
15baaa3
to
59b2226
Compare
59b2226
to
23bba03
Compare
The code is a bit too terse for my liking right now and does just have an empty `case DISTO_EL10` - so this commit adds a comment.
The current code will install python3-toml on fedora and also on rhel11 (once it comes out) because that is the default for unhandled distros. This commit makes the various toml libs for each distro more explicit, see also osbuild/osbuild#1851
23bba03
to
039febe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff. Thank you!
For reference, I had a look at the package selection diff this PR introduces and the only change is that Fedora iot commits/containers that embed containers now use the python3-tomli-w
library instead of python3-toml
.
Some job on the merge queue got stuck, didn't even start, just appears in a paused/waiting state. Weird! |
The current code will install python3-toml on fedora and also
on rhel11 (once it comes out) because that is the default for
unhandled distros. With
tomlib
in the stdlib this is nolonger needed.
[draft as it has no test(s)]