mini-x-session: Fix starting of x-session

We need to use sudo semantics since it does not start correctly
with su. Added RDEPENDS to ensure it's present on target.

The SESSION_DIR test was constructed incorrectly, so fixed it.

(From OE-Core rev: badf3a32b9737f2ad6f9c9ff2fe623fe17295946)

Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

diff --git a/meta/recipes-graphics/mini-x-session/files/mini-x-session b/meta/recipes-graphics/mini-x-session/files/mini-x-session
index 91f1592..dca655a 100644 (file)
--- a/meta/recipes-graphics/mini-x-session/files/mini-x-session
+++ b/meta/recipes-graphics/mini-x-session/files/mini-x-session
@@ -17,13 +17,14 @@ exec /etc/mini_x/session
 fi
 
 MINI_X_SESSION_DIR=/etc/mini_x/session.d
-if [ -d "$MINI_X_SESSION_DIR"]; then
+if [ -d "$MINI_X_SESSION_DIR" ]; then
        # Execute session file on behalf of file owner
        find $MINI_X_SESSION_DIR -type f | while read SESSIONFILE; do
                set +e
                USERNAME=`stat -c %U $SESSIONFILE`
                # Using su rather than sudo as latest 1.8.1 cause failure [YOCTO #1211]
-               su -l -c '$SESSIONFILE&' $USERNAME
+#              su -l -c '$SESSIONFILE&' $USERNAME
+               sudo -b -i -u $USERNAME $SESSIONFILE&
                set -e
        done
 fi

diff --git a/meta/recipes-graphics/mini-x-session/mini-x-session_0.1.bb b/meta/recipes-graphics/mini-x-session/mini-x-session_0.1.bb
index 8106cbb..3819ee3 100644 (file)
--- a/meta/recipes-graphics/mini-x-session/mini-x-session_0.1.bb
+++ b/meta/recipes-graphics/mini-x-session/mini-x-session_0.1.bb
@@ -13,6 +13,8 @@ RCONFLICTS_${PN} = "matchbox-common"
 SRC_URI = "file://mini-x-session"
 S = "${WORKDIR}"
 
+RDEPENDS_${PN} = "sudo"
+
 inherit update-alternatives
 
 ALTERNATIVE_${PN} = "x-session-manager"