Page MenuHomeFreeBSD

net/ipxe: build undionly and efi images

Authored by on Apr 25 2018, 12:40 PM.



This patch adds default-on options to:

  • build a PXE-bootable image containing only the UNDI driver (undionly.kpxe)
  • build EFI images for i386 and x86_64
Test Plan
  • built using poudriere testport
  • checked EFI images using bhyve(8)

Diff Detail

rP FreeBSD ports repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

tobik requested changes to this revision.Apr 25 2018, 1:08 PM
tobik added inline comments.
79 ↗(On Diff #41845)

Options description should form their own block, so please move EFI_DESC to the options description block i.e. before ISO_DESC

The other EFI_* options helper need to be moved before the ISO_* options helpers. Keep them sorted.

86 ↗(On Diff #41845)

Please remove UNDIONLY . I see no reason why this has to be an option. We can just always build it.

The new images also need to be mentioned in the pkg-message.

102 ↗(On Diff #41845)

This needs to be after pre-build-ISO-on

104 ↗(On Diff #41845)


107 ↗(On Diff #41845)

... and this after do-install

109 ↗(On Diff #41845)

cd ${WRKSRC} is pointless here. Make it part of the INSTALL_DATA command

This revision now requires changes to proceed.Apr 25 2018, 1:08 PM
104 ↗(On Diff #41845)

bin-x86_64-efi/ipxe.efi fails to build on i386, so I think we need to either not build the EFI images on i386 via OPTIONS_EXCLUDE_i386=EFI or restrict ourselves to only build bin-i386-efi/ipxe.efi on i386

libgcc/__divdi3.c:1:0: sorry, unimplemented: 64-bit mode not compiled in
libgcc/icc.c:1:0: sorry, unimplemented: 64-bit mode not compiled in
 /* marked 6 inline comments as done.

Adressed some comments.

Unfortunately I'm not completely sure how to conditionally include the EFI image filenames in when the option is set.

86 ↗(On Diff #41845)

I'm not sure how to best do this conditionally.

104 ↗(On Diff #41845)

I just took this from the default do-build target in Changed it now though.

tobik added inline comments.
86 ↗(On Diff #41845)

Always having them in the pkg-message is fine with me. No point in making this more complicated.

This revision is now accepted and ready to land.Apr 25 2018, 3:12 PM
This revision was automatically updated to reflect the committed changes.