-
Notifications
You must be signed in to change notification settings - Fork 312
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
Add support for building UKI addons #3074
Conversation
760bfd1
to
b7796d9
Compare
b7796d9
to
deb492e
Compare
862e239
to
105d81c
Compare
105d81c
to
def34ea
Compare
def34ea
to
807f429
Compare
@@ -4479,6 +4489,7 @@ def summary(config: Config) -> str: | |||
Shim Bootloader: {config.shim_bootloader} | |||
Unified Kernel Images: {config.unified_kernel_images} | |||
Unified Kernel Image Format: {config.unified_kernel_image_format} | |||
Unified Kernel Image Addons: {line_join_list(config.pe_addons)} |
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.
A bit too late at this, but I think this should mention "PE" somehow, too, e.g. "Unified Kernel Image PE Addons", since the option is named --pe-addon
, that is what I would grep for in the output of mkosi summary
.
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.
Woops, forgot to rename that. Probably should only be "PE Addons" since that is what they are referred to in the spec.
Not something I personally need anymore (so feel free to close), but I already implemented it and thought this might be useful to some.
This basically just reuses
ukify
s ability to take config files in which you can configure whatever (e.g.Cmdline=
,DeviceTree=
, ...) and passes the output target and secure boot parameters when needed. This should allow additions toukify
without needing to update the code here.