diff options
author | jpaetzel <jpaetzel@FreeBSD.org> | 2015-01-29 01:28:58 +0800 |
---|---|---|
committer | jpaetzel <jpaetzel@FreeBSD.org> | 2015-01-29 01:28:58 +0800 |
commit | 3ce864c11e38886557e8961310c3027ebbc9fe87 (patch) | |
tree | f4fc74ead499c8cb5ad200cf43247fe6ed287d80 /emulators | |
parent | 224035f12fe311d0fcd415e59a5a909736591bee (diff) | |
download | freebsd-ports-gnome-3ce864c11e38886557e8961310c3027ebbc9fe87.tar.gz freebsd-ports-gnome-3ce864c11e38886557e8961310c3027ebbc9fe87.tar.zst freebsd-ports-gnome-3ce864c11e38886557e8961310c3027ebbc9fe87.zip |
Fix circular dep issues so this script can DTRT at boot.
This script needs to run after ldconfig, however it's deps cause:
a) A circular dependancy in rcorder
b) It starts before ldconfig
This causes the VBoxManage command to bail out when it's run by
this script because it can't find it's required libs. The consequence of
this is host only networks aren't set up properly at boot.
By removing the explicit dependancy on ldconfig we get rid of the circular
dependancy issues. However since the VBoxManage command does actually need
libraries in /usr/local/lib we set that explicitly with LD_LIBRARY_PATH.
There's not really a good solution to this issue. The real "problem" is that /usr
could be an NFS mount and you have to run ldconfig after /usr is mounted. While
LD_LIBRARY_PATH is kind of ugly, and removing an explicit dependancy is ugly,
the other alternatives are uglier. (Break NFS mounted /usr for example)
I'd also argue that having this just not work (which is the state of it before
this commit) is pretty ugly too.
Also, I apologize in advance for not discussing this on emulation@ and just doing
a drive by commit, however hopefully the fact that "This fixes obvious breakage"
makes up for the fact that I skipped peer review, proper approval, and most importantly
giving the community to bikeshed this ad naseum.
Diffstat (limited to 'emulators')
-rw-r--r-- | emulators/virtualbox-ose-kmod/files/vboxnet.in | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/emulators/virtualbox-ose-kmod/files/vboxnet.in b/emulators/virtualbox-ose-kmod/files/vboxnet.in index d2745daf7457..6145137295f9 100644 --- a/emulators/virtualbox-ose-kmod/files/vboxnet.in +++ b/emulators/virtualbox-ose-kmod/files/vboxnet.in @@ -5,7 +5,7 @@ # PROVIDE: vboxnet # BEFORE: NETWORKING -# REQUIRE: FILESYSTEMS ldconfig +# REQUIRE: FILESYSTEMS # KEYWORD: nojail # @@ -43,7 +43,7 @@ vboxnet_start() fi # initialize configured host-only interfaces - %%PREFIX%%/bin/VBoxManage list hostonlyifs >/dev/null + LD_LIBRARY_PATH=%%PREFIX%%/lib %%PREFIX%%/bin/VBoxManage list hostonlyifs >/dev/null } vboxnet_stop() |