From: Scott Murray <scott.murray@konsulko.com>
Date: Tue, 21 Dec 2021 23:37:36 +0000 (-0500)
Subject: meta-app-framework: tweak agl-driver definition
X-Git-Tag: 12.91.0~3
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=15f6be527e8c54f6695345554bbbf4500b5a83e8;p=AGL%2Fmeta-agl.git

meta-app-framework: tweak agl-driver definition

Tweak agl-driver user definition in agl-session recipe to ensure
it is a member of the video and display groups (the latter has
been added back to the static group file).  This is required to
avoid agl-compositor startup failures on rcar3.  The display
group membership potentially could be dropped if we were to
bbappend rcar3's gles-user-module recipe to tweak its udev rules.
For now, take the most straightforward approach.

Bug-AGL: SPEC-4161

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I7237ade5d8680655f17716ac048349a476eb5f29
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/27060
Tested-by: Jenkins Job builder account
ci-image-build: Jenkins Job builder account
ci-image-boot-test: Jenkins Job builder account
---

diff --git a/meta-agl-core/files/group b/meta-agl-core/files/group
index f52448920..b04ff377b 100644
--- a/meta-agl-core/files/group
+++ b/meta-agl-core/files/group
@@ -86,6 +86,7 @@ lldpd::956:
 postdrop::954:
 weston::200:
 wayland::201:
+display::202:
 agl-driver::1001:
 agl-passenger::1002:
 systemd-network::1005:
diff --git a/meta-app-framework/recipes-config/agl-session/agl-session_0.1.bb b/meta-app-framework/recipes-config/agl-session/agl-session_0.1.bb
index 696533383..dc7b5bee7 100644
--- a/meta-app-framework/recipes-config/agl-session/agl-session_0.1.bb
+++ b/meta-app-framework/recipes-config/agl-session/agl-session_0.1.bb
@@ -13,10 +13,12 @@ inherit systemd allarch useradd
 USERADD_PACKAGES = "${PN}"
 USERADDEXTENSION = "useradd-staticids"
 GROUPADD_PARAM:${PN} = "\
+	--system display ; \
+	--system video ; \
 	-g 1001 agl-driver ; \
 "
 USERADD_PARAM:${PN} = "\
-  -g 1001 -u 1001 -o -d /home/agl-driver -m -K PASS_MAX_DAYS=-1 agl-driver ; \
+  -g 1001 -u 1001 -G video,display -o -d /home/agl-driver -m -K PASS_MAX_DAYS=-1 agl-driver ; \
 "
 
 SYSTEMD_PACKAGES = "${PN}"