-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fixes for UKI without sd-boot, UKI with legacy installkernel, and rEFInd #51
Conversation
721246a
to
34cfd7f
Compare
0be7bb9
to
9f32a14
Compare
On systems without systemd-boot we might still install and boot UKIs, with e.g. refind, or directly from firmware. On such systems the 'loader' directory (bls type 1) won't be present. Instead we only have the ESP/EFI/Linux directory (bls type 2), which we still want to clean-up. Signed-off-by: Andrew Ammerlaan <[email protected]>
a93febd
to
ee5f25b
Compare
0b41b5f
to
85edf4c
Compare
@mgorny This is now ready I think. I added the fixes for rEFInd icon files here as well to avoid rebasing later. |
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.
Also add copyright lines for yourself.
36d8754
to
e946e6f
Compare
f208bb1
to
b1db927
Compare
b1db927
to
6197857
Compare
See-also: projg2/eclean-kernel#51 Signed-off-by: Andrew Ammerlaan <[email protected]>
dc5f2bc
to
1a1a4f4
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.
Also feel free to add a commit fixing quotes in surrounding code.
1a1a4f4
to
b90dd04
Compare
df128f5
to
2090440
Compare
07d64fc
to
8d15663
Compare
Closes: projg2#49 Signed-off-by: Andrew Ammerlaan <[email protected]>
Signed-off-by: Andrew Ammerlaan <[email protected]>
Signed-off-by: Andrew Ammerlaan <[email protected]>
Signed-off-by: Andrew Ammerlaan <[email protected]>
this works around a bug in tox.ini where it does not take into account the dependencies of the package Signed-off-by: Andrew Ammerlaan <[email protected]>
8d15663
to
a754019
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.
LGTM.
ESP/EFI/Linux
but noESP/loader
directory. Extend ESP finding logic to also take into account this case.