[X2Go-Commits] [buildscripts] 01/01: bin/slave-start-prepare.sh: sg does not support "-" as the group name, pass in ${USER} directly.

[git-admin at x2go.org]

This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch master
in repository buildscripts.

commit c13179d3f8e28b62ee322eaf0c9f56c2cc117cdb
Author: Mihai Moldovan <ionic at ionic.de>
Date:   Sat Nov 25 09:46:08 2017 +0100

    bin/slave-start-prepare.sh: sg does not support "-" as the group name, pass in ${USER} directly.
---
 bin/slave-start-prepare.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/bin/slave-start-prepare.sh b/bin/slave-start-prepare.sh
index d7608ab..53be85b 100755
--- a/bin/slave-start-prepare.sh
+++ b/bin/slave-start-prepare.sh
@@ -28,7 +28,10 @@ else
       exec sg "${cur_group}" "${0} --set-groups ${@}"
     else
       # No more groups in list, make the primary group actually primary.
-      exec sg - "${0} --skip-groups"
+      # Note that while the sg man page says that it supports the "-"
+      # parameter just as newgrp does, in fact this is not supported.
+      # Let's hope the primary group is always called like the user.
+      exec sg "${USER}" "${0} --skip-groups"
     fi
   else
     if [[ "${1}" == "--skip-groups" ]]; then

--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/buildscripts.git