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

[rocko/master] [PATCH v2] 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 <a0271529@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

Re: [rocko/master] [PATCH v2] matrix-gui: update matrix-gui-2.0.service

[Denys Dmytriyenko]

Ack

On Tue, Dec 04, 2018 at 03:25:34PM -0500, Hongmei Gou wrote:
> * 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 <a0271529@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
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago