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

build-driver: ship ipxe + memtest86+; grml-live: fix NO_ADDONS autodetection #222

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

mika
Copy link
Member

@mika mika commented Dec 18, 2024

On arm64 we ended up with the following addon behavior:

  [*] Installing boot addons.
  [x] Missing addon file: "ipxe.lkrn"
  [x] Missing addon file: "ipxe.efi"
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

And on amd64 we ended up with the following addon behavior:

  [*] Installing boot addons.
  [x] Missing addon file: "ipxe.lkrn"
  [x] Missing addon file: "ipxe.efi"
  [x] Missing addon file: "memtest86+x64.efi"
  [x] Missing addon file: "memtest86+x64.bin"
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

Now by not using the NO_ADDONS autodetection and shipping ipxe on amd64 + arm64 and memtest86+ on amd64 only (it's not available for arm64), we get the expected behavior, as in:

On arm64:

  [*] Installing boot addons.
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

On amd64:

  [*] Installing boot addons.
  [x] Missing addon file: bsd4grml

Fixes commit a2a16ec Related to #161 and #128

…tection

On arm64 we ended up with the following addon behavior:

  [*] Installing boot addons.
  [x] Missing addon file: "ipxe.lkrn"
  [x] Missing addon file: "ipxe.efi"
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

And on amd64 we  ended up with the following addon behavior:

  [*] Installing boot addons.
  [x] Missing addon file: "ipxe.lkrn"
  [x] Missing addon file: "ipxe.efi"
  [x] Missing addon file: "memtest86+x64.efi"
  [x] Missing addon file: "memtest86+x64.bin"
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

Now by not using the NO_ADDONS autodetection and shipping ipxe on amd64
+ arm64 and memtest86+ on amd64 only (it's not available for arm64), we
get the expected behavior, as in:

On arm64:

  [*] Installing boot addons.
  [x] Missing addon file: "memtest86+.bin"
  [x] Missing addon file: bsd4grml

On amd64:

  [*] Installing boot addons.
  [x] Missing addon file: bsd4grml

Fixes commit a2a16ec
Related to #161
and #128
@mika mika requested review from zeha and jkirk December 18, 2024 21:19
@mika mika merged commit 4d9184a into master Dec 18, 2024
24 checks passed
@mika mika deleted the mika/addons branch December 18, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants