[meta-arago] [rocko/master] [PATCH] matrix-gui: update matrix-gui-2.0.service

[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

Signed-off-by: Hongmei Gou <***@ti.com>
---
.../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 9 +++++++++
meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 10 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..cd87d8b 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,18 @@
[Unit]
Description=Matrix GUI
After=weston.service
+Requires=weston.service
+Before=graphical.target
+Before=thermal-zone-init.service

[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"

[Denys Dmytriyenko]

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
---
.../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 9 +++++++++
meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 10 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..cd87d8b 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,18 @@
[Unit]
Description=Matrix GUI
After=weston.service
+Requires=weston.service
+Before=graphical.target
+Before=thermal-zone-init.service

I don't believe this one should be added - there's no dependency between the
two and thermal-zone-init may be gone in the future or on some platforms.

Post by Hongmei Gou
[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

[Gou, Hongmei]

-----Original Message-----
From: Dmytriyenko, Denys
Sent: Tuesday, December 04, 2018 2:51 PM
To: Gou, Hongmei
Subject: Re: [meta-arago] [rocko/master] [PATCH] matrix-gui: update matrix-
gui-2.0.service

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
---
.../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 9

+++++++++

Post by Hongmei Gou
meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 10 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

Post by Hongmei Gou
index 3ce1024..cd87d8b 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-

2.0.service

Post by Hongmei Gou
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/matrix-gui-

2.0.service

Post by Hongmei Gou
@@ -1,9 +1,18 @@
[Unit]
Description=Matrix GUI
After=weston.service
+Requires=weston.service
+Before=graphical.target
+Before=thermal-zone-init.service

I don't believe this one should be added - there's no dependency between the
two and thermal-zone-init may be gone in the future or on some platforms.

Yes, you are right. Will submit a V2 patch to remove this.

Post by Hongmei Gou
[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

Post by Hongmei Gou
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"

Post by Hongmei Gou
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