Page MenuHomeFreeBSD

sysutils/uefi-edk2-bhyve: add slave port with CSM enabled
AbandonedPublic

Authored by novel on Jul 7 2016, 12:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 6 2024, 4:24 AM
Unknown Object (File)
Jan 4 2024, 3:52 PM
Unknown Object (File)
Dec 20 2023, 12:54 AM
Unknown Object (File)
Nov 28 2023, 10:03 AM
Unknown Object (File)
Nov 25 2023, 9:00 PM
Unknown Object (File)
Nov 13 2023, 10:57 AM
Unknown Object (File)
Oct 24 2023, 9:19 PM
Unknown Object (File)
Sep 5 2023, 3:55 AM

Details

Summary

Both slave and master could be installed at the same time as they install firmware with different name depending if CSM was activated or not.

While here, change direct dependency on gcc to USE_GCC in uefi-edk2-bhyve.

Test Plan

Both ports tested with poudriere.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

novel retitled this revision from to sysutils/uefi-edk2-bhyve: add slave port with CSM disabled.
novel updated this object.
novel edited the test plan for this revision. (Show Details)
novel set the repository for this revision to rP FreeBSD ports repository.
fabian.freyer_physik.tu-berlin.de edited edge metadata.

tbh, I'd argue for a -csm port, and make no csm the default - but I guess this should be discussed. But this would also be fine for me.

This revision is now accepted and ready to land.Jul 7 2016, 9:34 AM
novel retitled this revision from sysutils/uefi-edk2-bhyve: add slave port with CSM disabled to sysutils/uefi-edk2-bhyve: add slave port with CSM enabled.
novel edited edge metadata.

Reworked to have -csm slave port and the main port without CSM.

Also, I noticed that I need to drop the CSM option at all to avoid conflicts with the slave port.

This revision now requires review to proceed.Jul 8 2016, 12:40 AM
This revision is now accepted and ready to land.Jul 8 2016, 3:02 PM

TBH, we should also bump the port version here.

uefi-edk2-bhyve/Makefile
9

PORTREVISION=1

fabian.freyer_physik.tu-berlin.de added inline comments.
uefi-edk2-bhyve/Makefile
20

@editor_callfortesting.org suggested we change this to ${PREFIX}/share/uefi-firmware/ since firmware is considered plural

48

s/uefi-firmwares/uefi-firmware/

uefi-edk2-bhyve/Makefile
30

I think these could be removed

fabian.freyer_physik.tu-berlin.de edited edge metadata.
fabian.freyer_physik.tu-berlin.de added inline comments.
uefi-edk2-bhyve-csm/Makefile
5

s/with/without

This revision now requires changes to proceed.Jul 8 2016, 5:08 PM

I can't help but notice that there is a /usr/share/firmware directory.

Should this simply be used?

I can't help but notice that there is a /usr/share/firmware directory.

Should this simply be used?

I don't think so; we should respect ${PREFIX}. Also it's useful to have a folder containing only UEFI Firmware since bhyve managers (e.g. libvirt) would list these.