Renamed MANUAL to DOCS after discussion with Zirias in IRC.

Great job!

Removed double REINPLACE_CMD for src/VBox/HostDrivers/adpctl/VBoxNetAdpCtl.cpp - 1st is in patch.

After discussion in IRC did several improvements.

Replaced all "/usr/local" with %%LOCALBASE%% and "/usr/local/lib/virtualbox" with %%VBOX_DIR%%.

In D40119#914019, @arrowd wrote:

Ideally, we'd want to use %%LOCALBASE%% substitution there, but meh. I think this is fine as it is.

Ideally, we'd want to use %%LOCALBASE%% substitution there, but meh. I think this is fine as it is.

LGTM given that you did the build and then make check-plist.

Can we just reuse the patch from www/phpvirtualbox via EXTRA_PATCHES?

PATH changed to: /sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin

In D40088#913176, @arrowd wrote:

It builds for me on 14-CURRENT, but doesn't work: I'm getting INTERNAL_ERROR 3 on a VM start. Does it work for you on 13?

It builds for me on 14-CURRENT, but doesn't work: I'm getting INTERNAL_ERROR 3 on a VM start. Does it work for you on 13?

Looks fine to me, however please let it be further reviewed by and obtain approvals from ports committers.

https://lists.freebsd.org/archives/freebsd-virtualization/2022-November/001004.html

@lwhsu thanks for the correction. I'll attempt the commit, subject to approval of the review.

In D37074#841877, @grahamperrin wrote:

I don't have a ports commit bit, so – if approved – this will require a commit by someone else.

Please, is the summary reasonable?

You've got my approval to commit, @nc!

Update PORTREVISION for current Ports tree.

LGTM

It's not just required if you happen to be bridging. ether_demux() expects to be called in epoch context. (Its first call is to NET_EPOCH_ASSERT().)

@emaste : https://svnweb.freebsd.org/changeset/base/288176 will not MFH, right?