virtualbox-ose — Commit History

Thes is ~430-640 files, ~30-55MB total for different platforms.
We are using devel/kBuild for build VirtualBox ports.
devel/kBuild is a newer version than the one included in the VirtualBox
tarballs.

PR:	289019

After commit
https://github.com/freebsd/freebsd-src/commit/93b4a54456589e78dcd37f2db8333baff3f3f0b2
  kernel: linker_load_file: /boot/modules/vboxnetflt.ko - unsupported file type
  kernel: KLD vboxnetflt.ko: depends on netgraph - not available or version mismatch

Thanks to Benjamin Close <Benjamin.Close@clearchain.com> and
rkoberman@gmail.com for researching the issue and identifying its cause.

PR:	286204 286717
MFH:	2025Q2

Apply the same fix as 8b659812a21508795778a345dbbf7d08b7fcbb38 to allow
cross buidling and make it appear in kmods repositories

PR:	284439

PR:	271146

Fixes:	b4a6bc8744f4 (Improve ports)

* Replace PORTVERSION with DISTVERSION.
* Fix CONFLICTS_INSTALL.
* Fix PYTHON in DEFAULT_OPTIONS in emulators/virtualbox-ose.
* Pet portclippy and portlint.
* Formatting and etc.

PR:	258159

Simplify expressions for FreeBSD 13.X

Reviewed by:	many

Differential Revision:	https://reviews.freebsd.org/D46601

* Fix build with llvm15+ from ports and from base:
  - ISO C++17 does not allow 'register' storage class specifier;
  - require -fPIC on i386.
* Fix load kernel modules on host and guest i386 if kernel compiled with
PAE option: add non-default option PAE to ports.
Based on patches from mi@.

PR:		279257 224616
Approved by:	vbox (maintainer, timeout 1 month and 6 years)

llvm15 was also merged into 13.2-STABLE effective from OSVERSION
1302505. Fix the ports that failed to build with llvm15.

Approved by:	portmgr (blanket)

Approved by:	portmgr (blanket)

Commit b7f05445c00f has added WWW entries to port Makefiles based on
WWW: lines in pkg-descr files.

This commit removes the WWW: lines of moved-over URLs from these
pkg-descr files.

Approved by:		portmgr (tcberner)

It has been common practice to have one or more URLs at the end of the
ports' pkg-descr files, one per line and prefixed with "WWW:". These
URLs should point at a project website or other relevant resources.

Access to these URLs required processing of the pkg-descr files, and
they have often become stale over time. If more than one such URL was
present in a pkg-descr file, only the first one was tarnsfered into
the port INDEX, but for many ports only the last line did contain the
port specific URL to further information.

There have been several proposals to make a project URL available as
a macro in the ports' Makefiles, over time.

This commit implements such a proposal and moves one of the WWW: entries
of each pkg-descr file into the respective port's Makefile. A heuristic
attempts to identify the most relevant URL in case there is more than
one WWW: entry in some pkg-descr file. URLs that are not moved into the
Makefile are prefixed with "See also:" instead of "WWW:" in the pkg-descr
files in order to preserve them.

There are 1256 ports that had no WWW: entries in pkg-descr files. These
ports will not be touched in this commit.

The portlint port has been adjusted to expect a WWW entry in each port
Makefile, and to flag any remaining "WWW:" lines in pkg-descr files as
deprecated.

Approved by:		portmgr (tcberner)

Use a more restricted range of supported __FreeBSD_version values for
the virtualbox kernel module since it uses internal KPIs for which KBI
stability is not guaranteed.  DECLARE_MODULE_TIED would tie the module
to a specific __FreeBSD_version which would cause the module to stop
loading when an SA or EN is pushed on a release branch.  Instead, use
DECLARE_MODULE_MAXVER with an upper bound that is limited to the range
of a releng branch so that a module built on X.Y will load on X.Y with
SAs or ENs applied but will fail to load on X.Y-1 or X.Y+1.

Reviewed by:	kevans, imp, madpilot, lwhsu (earlier version)
Differential Revision:	https://reviews.freebsd.org/D16119

PR:		262656

There have been lots of missing CONFLICTS_INSTALL entries, either
because conflicting ports were added without updating existing ports,
due to name changes of generated packages, due to mis-understanding
the format and semantics of the conflicts entries, or just due to
typoes in package names.

This patch is the result of a comparison of all files contained in
the official packages with each other. This comparison was based on
packages built with default options and may therefore have missed
further conflicts with optionally installed files.

Where possible, version numbers in conflicts entries have been
generalized, some times taking advantage of the fact that a port
cannot conflict with itself (due to logic in bsd.port.mk that
supresses the pattern match result in that case).

A few ports that set the conflicts variables depending on complex
conditions (e.g. port options), have been left unmodified, despite
probably containing outdated package names.

These changes should only affect the installation of locally built
ports, not the package building with poudriere. They should give an
early indication of the install conflict in cases where currently
the pkg command aborts an installation when it detects that an
existing file would be overwritten,

Approved by:	portmgr (implicit)

The conflict checks compare the patterns first against the package
names without version (as reported by "pkg query "%n"), then - if
there was no match - agsinst the full package names including the
version (as reported by "pkg query "%n-%v").

Approved by: portmgr (blanket)

After src commit e266a0f7f001 a module is not allowed to call
kern_kldload while being loaded. The virtualbox module is doing
exactly this though. virutalbox used to do this because the ng_ether
netgraph node did not provide version information in the past and
there was no way to properly depend on it.

ng_ether has gained versioning information in base r238844 more
than ten years ago, so we can now unconditionally properly depend
on it, since this revision is now included in all supported FreeBSD
versions.

Thanks to Tomoaki AOKI for identifying the base commuit exposing
the issue.

Backport fix to legacy port.

WHile here also merge a double patch for the smae file.

PR:		256505
MFH:		2021Q2

The VirtualBox legacy ports track version 5.x, limit portscout
reporting accordingly.

Please note that this new version supports only amd64 CPUs. If you need
to use older hardware please install the legacy ports.

Note that moving VM snapshots across major updates is unsupported, it's
strongly suggested to properly shutdown VMs before upgrading, please
check UPDATING for further details.

This update is the result of work from many people, and thanks to all
who gave feedback and tested things.

Patch based on work from: Mario Lobo <lobo@bsd.com.br> and jkim.

PR:			234878
Submitted by:		kunda <chitty_cloud@me.com>
Approved by:		vbox (implicit)
Reviewed by:		decke
Tested by:		jwb,
			martin ilavsky <ilavsky.martin@gmail.com>,
			Mario Lobo <lobo@bsd.com.br>
Relnotes:		https://www.virtualbox.org/wiki/Changelog-6.1
Differential Revision:	https://reviews.freebsd.org/D28871

The virtualbox-ose-kmod-legacy port is staged, but it has no purpose
without the unstaged virtualbox-ose-legacy port, so both ports have
been removed at the same time.

Tested by:	Douglas Berry (doug at bitnix dot ca)
PR:		ports/187580

Reviewed by:	vbox@ (decke)

PR:		ports/185579 [2]
Security:	CVE-2013-5892
Submitted by:	Jung-uk Kim <jkim@FreeBSD.org> [1]
		Jimmy Olgeni <olgeni@FreeBSD.org> [2]

Submitted by:	Gabor Pali <pgj@freebsd.org>

Obtained from:	https://www.virtualbox.org/ticket/10940

Discussed with:	beat (for chruetertee mirrors)

Submitted by:	bapt

Obtained from:		redports.org/~virtualbox/

In cases where I had to hand-edit unusual instances also
modify formatting slightly to be more uniform (and in
some cases, correct). This includes adding some $FreeBSD$
tags, and most importantly moving rcvar= to right after
name= so it's clear that one is derived from the other.

Reported by:	avg

Remove explicit initialization of m->flags and m->oflags because per
default vm_phys_alloc_contig() already initializes the page as unmanaged.
Chase move of PG_UNMANAGED flag from m->flags to m->oflags and renaming
to VPO_UNMANAGED for an additional assert.

PR:		ports/159675
Reported by:	Kyryll A Mirnenko <xmirya@gmail.com>,
		Michael Butler <imb@protected-networks.net>
Thanks to:	beat

Submitted by:	Jung-uk Kim <jkim@FreeBSD.org>

PR:		ports/156699 (based on) [3]
Submitted by:	Stefan Tollkuehn <stefan.tollkuehn@editura.de>, [1][2]
		Raul Gigea <raul.gigea@editura.de>, [1]
		Tobias Oberstein <tobias.oberstein@tavendo.de>, [1]
		Emil <the_mix_room@hotmail.com> [3]
Thanks to:	dougb@, [1][2]
		Cr4zi3/EFNet, [2]
		crsd [2]

Submitted by:	Michael Butler <imb at protected-networks dot net>

Feature safe:	yes

Approved by:	beat (co-mentor)

Reported by:	scf@ [1]
Submitted by:	imp@ [2]

Many thanks to the VirtualBox developers (especially Alexander Eichner
for all his work and help), all tester and patch submitter and the whole
vbox@ team especially decke@.

Obtained from:	http://www.virtualbox.org/changeset/25953 [1]
Reported by:	William DiNoia <william.dinoia AT gmail.com> [2]
On behalf of:	vbox@ (decke, dhn, itetcu, miwi, nox)

Changelog from VirtualBox is available here:
http://www.virtualbox.org/wiki/Changelog

Changes in the port:
  - VirtualBox and Guest Additions iso updated to 3.1.4
  - Add UNIQUENAME to fix options file clashes between the virtualbox ports [0]
  - Fixed vtophys problems on CURRENT and STABLE kernes newer than
    January 23th [1]
  - Fixed page double wiring issue in rtR0MemObjNativeAllocPage [2]

PR: 		ports/143361 [0]
Submitted by:	Martin Birgmeier <martin.birgmeier AT aon.at> [0]
		Alexander Eichner <Alexander.Eichner AT Sun.COM> [1]
Thanks To:	All testers on emulation@ and current@ [1]
           	Giovanni Trematerra <giovanni.trematerra AT gmail.com> [1]
		Ian Freislich <ianf AT clue.co.za> [1]
Reported by: 	Alan Cox <alc AT cs dot rice dot edu> [2]

Reported by:	Alexander Kabaev <kabaev AT gmail.com>,
		Rainer Hurling <rhurlin AT gwdg.de>,
		Nakata Maho <chat95 AT mac.com>,
		Andrzej Tobola <ato AT iem.pw.edu.pl>
On behalf of:   vbox@ (decke, dhn, itetcu, miwi, nox)

emulators/virtualbox-ose:
- Update to 3.1.2
- Update guest additions to 3.1.2
- Port has been renamed to virtualbox-ose to reflect that we are using
  the OSE version. [1]
- Added proper PulseAudio support for FreeBSD [2]
- procfs is not required anymore because vbox uses sysctl(3) now [3]
- Update pkg-message to reflect recent changes
- Add nox's FreeBSD host networking patches that are now also in the
  upstream vbox svn (modulo vbox variable naming style adjustments:)
	http://lists.freebsd.org/pipermail/freebsd-emulation/2010-January/007260.html
	http://www.virtualbox.org/changeset/25698
  1. Allow direct tap networking again (for users that need the best
  network performance and/or need more complex network setups, like when
  they want to use routing instead of bridging to e.g. protect from guests
  messing with the lan's arp tables; a tap + routing + proxy arp example
  is in the above freebsd-emulation posting.)
  2. Enable vbox' shared mac feature when using bridged mode on a wifi
  interface, together with the virtualbox-ose-kmod change this
  should fix bridged mode for wifi users. [4]

emulators/virtualbox-ose-kmod:
- Update to 3.1.2
- Add rc.d script to load kernel modules
- Fix build with a non-standard location for the system source [5]
- Merge aeichner's vboxnetflt fix committed to upstream vbox svn
  (thanks!) that makes the shared mac feature enabled above actually work
  on FreeBSD hosts. [6]

Please see http://wiki.freebsd.org/VirtualBox for update instructions.

Many thanks to the VirtualBox developers, all tester, patch submitter and
the whole vbox@ team.

PR:		ports/141630 [2]
Noticed by:	mm@ [1]
Submitted by:	Noriyoshi Kawano <bowie AT nrik.jp> [2],
		Baptiste Daroussin <baptiste.daroussin AT gmail.com> [3]
		and Bernhard Froehlich <decke AT bluelife.at> [3],
		nox@ [4], scf@ [5]
Obtained from:	http://www.virtualbox.org/changeset/25699 [6]
On behalf of:	vbox@ (decke, dhn, itetcu, miwi, nox)

PR:			ports/141013
Reported / tested by:	Scot Hetzel <swhetzel AT gmail.com>
Submitted by:		Fredrik Lindberg <fli AT shapeshifter.se>

PR:		ports/140171
Submitted by:	Hubert Tournier <hubert AT tournier.org>
Reported by:	Stanislav Dulis <stasd AT xnx.ru> (via private mail)

PR:		ports/139391 [1]
Submitted by:	"Y.A." <nexther0 AT gmail.com> [1]
Tested by:	Yuri Gorchakov <yuri.gorchakov AT point-group.ru> [1]
Reported by:	Maxim <maksims.k AT gmail.com> [2]
On behalf of:	vbox@ (decke, dhn, itetcu, miwi, nox)

Reported by:	QATty
Submitted by:	Bernhard Froehlich <decke AT bluelife.at>
On behalf of:	vbox@ (decke, dhn, itetcu, miwi, nox)

Security:	http://www.vuxml.org/freebsd/ebeed063-b328-11de-b6a5-0030843d3802.html
On behalf of:	vbox@ (decke, dhn, itetcu, beat, nox)

Approved by:	vbox@ (miwi, beat)
Feature safe:	yes

PR:		ports/138935 [1] (based on)
Reported by:	scf@ [3]
Submitted by:	Julian H. Stacey <jhs AT berklix.com> [1],
		Bernhard Froehlich <decke AT bluelife.at> [2],
		Fredrik Lindberg <fli AT shapeshifter.se> [3]
On behalf of:	vbox@ (decke, dhn, itetcu, miwi, nox)
Feature safe:	yes

Reported by:	Jonathan Chen <jonc AT chen.org.nz> [1]
On behalf of:	vbox@ (decke, dhn, beat, nox)

PR:		ports/138063 [1]
Submitted by:	Anonymous <swell dot k at gmail dot com> [1],
		Bernhard Froehlich <decke AT bluelife.at> [2]
On behalf of:	vbox@ (decke, dhn, miwi, nox)

Submitted by:	Guido Falsi <mad AT madpilot.net> [1], nork [2],
		Bernhard Froehlich <decke AT bluelife.at> [3], nox [4]
Thanks to:	All testers and patch submitters their patch already
		included upstream
On behalf of:	vbox@ (decke, dhn, miwi, nox)

Reported by:	Mike Harding <mvh AT ix.netcom.com>

Submitted by:	miwi

Approved by:	vbox@ (miwi)

WWW:	http://www.virtualbox.org/

Note:

Please READ pkg-messages carefully, also please
take a look in the Wiki page (http://wiki.freebsd.org/VirtualBox)

Thanks to: thank you to everyone who helped make this possible.