Add support for building UKIProfiles initrds with extra packages #4142
Conversation
bluca
force-pushed
the
uki_profile_initrd
branch
from
dec005f to
532c49a
Compare
If Packages= is specifed under a UKI profile, take the 'main' initrd and add the new packages on top of it, and use that as the profile's initrd. Profile's sections are complete replacements, they are not additive, so it needs to be the full thing.
bluca
force-pushed
the
uki_profile_initrd
branch
from
532c49a to
06f0097
Compare
| `.cmdline` section and the extra kernel command line arguments | ||
| specified with this setting. | ||
|
|
||
| `Packages=` |
There was a problem hiding this comment.
So while this does end in it's own structure in the config, I think calling this Packages= has the (only very slight!) potential for hard to debug errors, since during config parsing we do pass around dictionaries.
More importantly, though, we already have InitrdPackages= which would make this a bit confusing, since that's for the packages in the initrd.
How about ExtraInitrdPackages= to clearly communicate that this is different and that this is additive?
There was a problem hiding this comment.
I'd rather leave it as-is, given it's a different section and it has the same syntax as the other it seems clearer. More importantly OBS won't parse a new setting for dependency resolution, so that would be a massive pain to change
2 participants
If Packages= is specifed under a UKI profile, take the 'main' initrd and add the new packages on top of it, and use that as the profile's initrd. Profile's sections are complete replacements, they are not additive, so it needs to be the full thing.