Page MenuHomeFreeBSD

sysutils/edk2: Add qemu_x64 and qemu_i386 as new FLAVORs of edk2
ClosedPublic

Authored by bcran on Jun 5 2022, 8:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 9:39 PM
Unknown Object (File)
Sun, May 5, 9:39 PM
Unknown Object (File)
Sun, May 5, 9:39 PM
Unknown Object (File)
Sun, May 5, 9:39 PM
Unknown Object (File)
Sun, May 5, 9:39 PM
Unknown Object (File)
Thu, May 2, 12:35 AM
Unknown Object (File)
Tue, Apr 30, 8:53 AM
Unknown Object (File)
Mon, Apr 29, 2:40 AM
Subscribers

Details

Summary

Instead of having a separate sysutils/uefi-edk2-qemu port, add qemu_x64
and qemu_i386 as FLAVORs of sysutils/edk2 and delete sysutils/uefi-edk2-qemu.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 45868
Build 42756: arc lint + arc unit

Event Timeline

bcran requested review of this revision.Jun 5 2022, 8:51 PM
bcran created this revision.
bcran retitled this revision from sysutils/edk2: Add edk2 as a new FLAVOR of edk2 to sysutils/edk2: Add qemu as a new FLAVOR of edk2.Jun 5 2022, 8:55 PM

I'm not sure how to handle the fact that qemu (OVMF) firmware can be built for both x86_64 and i386.
The existing port uses FLAVOR, but since sysutils/edk2 already uses FLAVOR for the platform, I'm not sure how selecting the architecture should be done.

I'm not sure how to handle the fact that qemu (OVMF) firmware can be built for both x86_64 and i386.
The existing port uses FLAVOR, but since sysutils/edk2 already uses FLAVOR for the platform, I'm not sure how selecting the architecture should be done.

Add two flavors, one qemu_x86 and one qemu_i386

Add separate qemu_x64 and qemu_i386 flavors.

bcran retitled this revision from sysutils/edk2: Add qemu as a new FLAVOR of edk2 to sysutils/edk2: Add qemu_x64 and qemu_i386 as new FLAVORs of edk2.Jun 6 2022, 3:29 AM
bcran edited the summary of this revision. (Show Details)

Since it depends on nasm I think it needs an ONLY_FOR_ARCH and the old one seems to be restricted on amd64 only.

Add ONLY_FOR_ARCH lines and revert i386 addition at top

This revision is now accepted and ready to land.Jun 6 2022, 4:10 PM