Discussion:
[meta-arago] [rocko/master] [PATCH v2] matrix-gui: update matrix-gui-2.0.service
Hongmei Gou
2018-12-04 20:25:34 UTC
Permalink
* Add "Requires=weston.service" to make matrix-gui start only after
weston has successfully started

* Match matrix-gui-2.0.service with SysVinit auto-generated service
to ensure reliable matrxi-gui startup from booting

Signed-off-by: Hongmei Gou <***@ti.com>
---
V2 change:
Remove "Before=thermal-zone-init.service" per the review

.../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 8 ++++++++
meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service b/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
index 3ce1024..40a8f0f 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
@@ -1,9 +1,17 @@
[Unit]
Description=Matrix GUI
After=weston.service
+Requires=weston.service
+Before=graphical.target

[Service]
Type=forking
+Restart=no
+TimeoutSec=5min
+IgnoreSIGPIPE=no
+KillMode=process
+GuessMainPID=no
+RemainAfterExit=yes
ExecStart=/etc/init.d/matrix-gui-2.0 start

[Install]
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
index c79f13e..064e27d 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"

SECTION = "multimedia"

-PR = "r36"
+PR = "r37"

SYSTEMD_SERVICE_${PN} = "matrix-gui-2.0.service"
--
1.9.1
Denys Dmytriyenko
2018-12-04 21:05:50 UTC
Permalink
Ack
Post by Hongmei Gou
* Add "Requires=weston.service" to make matrix-gui start only after
weston has successfully started
* Match matrix-gui-2.0.service with SysVinit auto-generated service
to ensure reliable matrxi-gui startup from booting
---
Remove "Before=thermal-zone-init.service" per the review
.../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 8 ++++++++
meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service b/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
index 3ce1024..40a8f0f 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-2.0.service
@@ -1,9 +1,17 @@
[Unit]
Description=Matrix GUI
After=weston.service
+Requires=weston.service
+Before=graphical.target
[Service]
Type=forking
+Restart=no
+TimeoutSec=5min
+IgnoreSIGPIPE=no
+KillMode=process
+GuessMainPID=no
+RemainAfterExit=yes
ExecStart=/etc/init.d/matrix-gui-2.0 start
[Install]
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
index c79f13e..064e27d 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"
SECTION = "multimedia"
-PR = "r36"
+PR = "r37"
SYSTEMD_SERVICE_${PN} = "matrix-gui-2.0.service"
--
1.9.1
_______________________________________________
meta-arago mailing list
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
Loading...