Merge branch 'master' of dnbd3:openslx-ng/tm-scripts

1 files changed, 9 insertions, 4 deletions

diff --git a/remote/modules/vbox/module.build b/remote/modules/vbox/module.build
index 5a533089..f4a8610a 100644
--- a/remote/modules/vbox/module.build
+++ b/remote/modules/vbox/module.build
@@ -1,11 +1,16 @@
 #!/bin/bash
 
 patch_vbox_scripts() {
-	# patching some virtualbox utility scripts to include openslx-busybox paths. Strange sed-ing, as the added
-	# openslx paths need to be at the end of PATH to not impede with system binaries to not impede with system binaries  
+	# Patching virtualbox utility scripts to include openslx-busybox binary paths. Strange sed-ing, as the openslx
+	# binary paths need to be at the end of PATH to not impede with system binaries. 
+	# It seems sufficient to patch just VBox, as eg. vboxmanage, vboxheadless etc. are just links to VBox.
+	# If only vboxmanage, vboxheadless or such should be patched (and not the base script VBox) just use a list
+	# in the loop (eg. 'for i in virtualbox vboxmanage vboxheadless; do'). These links will be replaced by patched
+	# 'real' files. Of course it will not make sense if VBox is included in the list, then.
+	# Patched files will be saved with extension .original.
+
 	pinfo "Patching virtual box scripts to include openslx (busybox)-paths ..."
-	# vboxmanage is a link to VBox; will get unlinked. Original link will be vboxmanage.original
-	for i in virtualbox vboxmanage vboxheadless; do
+	for i in VBox; do
 		pinfo "Patching virtual box script $i ..."
 		SCRIPTPATH=$(grep -m 1 PATH "${MODULE_BUILD_DIR}/usr/bin/$i"|sed 's/"//g')		# assume first hit is real path
 		sed -i "-i.original" "/^PATH=/c ${SCRIPTPATH}:/opt/openslx/bin:/opt/openslx/usr/bin:/opt/openslx/sbin"\