> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, November 20, 2018 6:14 PM
> To: Gou, Hongmei
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [rocko/master] [PATCH] matrix-gui: replace
> SysVinit with systemd service
>
> On Tue, Nov 20, 2018 at 05:43:09PM -0500, Gou, Hongmei wrote:
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Tuesday, November 20, 2018 5:37 PM
> > > To: Gou, Hongmei
> > > Cc: meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [rocko/master] [PATCH] matrix-gui: replace
> > > SysVinit with systemd service
> > >
> > > On Tue, Nov 20, 2018 at 05:12:52PM -0500, Gou, Hongmei wrote:
> > > > > -----Original Message-----
> > > > > From: Dmytriyenko, Denys
> > > > > Sent: Tuesday, November 20, 2018 4:54 PM
> > > > > To: Gou, Hongmei
> > > > > Cc: meta-arago@arago-project.org
> > > > > Subject: Re: [meta-arago] [rocko/master] [PATCH] matrix-gui: replace
> > > > > SysVinit with systemd service
> > > > >
> > > > > On Tue, Nov 20, 2018 at 04:51:51PM -0500, Gou, Hongmei wrote:
> > > > > > > -----Original Message-----
> > > > > > > From: Dmytriyenko, Denys
> > > > > > > Sent: Tuesday, November 20, 2018 4:43 PM
> > > > > > > To: Gou, Hongmei
> > > > > > > Cc: meta-arago@arago-project.org
> > > > > > > Subject: Re: [meta-arago] [rocko/master] [PATCH] matrix-gui:
> replace
> > > > > > > SysVinit with systemd service
> > > > > > >
> > > > > > > On Tue, Nov 20, 2018 at 04:33:55PM -0500, Hongmei Gou wrote:
> > > > > > > > Signed-off-by: Hongmei Gou <a0271...@ti.com>
> > > > > > > > ---
> > > > > > > > .../recipes-core/matrix/matrix-gui/matrix-gui-2.0.service | 10
> > > > > ++++++++++
> > > > > > > > meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb | 15
> > > > > > > +++++++++------
> > > > > > > > 2 files changed, 19 insertions(+), 6 deletions(-)
> > > > > > > > create mode 100644 meta-arago-extras/recipes-
> > > core/matrix/matrix-
> > > > > > > gui/matrix-gui-2.0.service
> > > > > > > >
> > > > > > > > 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
> > > > > > > > new file mode 100644
> > > > > > > > index 0000000..3ce1024
> > > > > > > > --- /dev/null
> > > > > > > > +++ b/meta-arago-extras/recipes-core/matrix/matrix-
> gui/matrix-
> > > gui-
> > > > > > > 2.0.service
> > > > > > > > @@ -0,0 +1,10 @@
> > > > > > > > +[Unit]
> > > > > > > > +Description=Matrix GUI
> > > > > > > > +After=weston.service
> > > > > > > > +
> > > > > > > > +[Service]
> > > > > > > > +Type=forking
> > > > > > > > +ExecStart=/etc/init.d/matrix-gui-2.0 start
> > > > > > >
> > > > > > > Looks like it's still calling a legacy initscript - any
> > > > > > > particular reason?
> > > > > >
> > > > > > Yes, we would like to keep the original scripts, as they are also
> > > > > > used
> for
> > > > > stopping/starting matrix GUI when running GUI based demos
> > > > > (/usr/share/matrix-gui-2.0/execute_command.sh):
> > > > > >
> > > > > > if [ $program_type == gui ]
> > > > > > then
> > > > > > /etc/init.d/matrix-gui-2.0 stop
> > > > > > ..
> > > > > >
> > > > > > if [ $program_type == gui ]
> > > > > > then
> > > > > > ...
> > > > > > /etc/init.d/matrix-gui-2.0 start
> > > > > > fi
> > > > >
> > > > > Any plans to completely moving to systemd and calling systemctl
> > > start/stop?
> > > >
> > > > With the current patch, "systemctl start/stop matrix-gui-2.0 " can
> > > > start/stop matrix successfully.
> > > > execute_command.sh can be updated to use systemctl start/stop.
> > > >
> > > > As for /etc/init.d/matrix-gui-2.0 (attached), I am not seeing anything
> > > > specific to init.
> > > > We can install it to /usr/bin instead of /etc/init.d, and then modify
> > > > the
> > > > path in matrix-gui-2.0.service, if this is preferable.
> > >
> > > If you keep it in /etc/initd.d, wouldn't systemd try to automatically
> convert
> > > it and potentially conflict with your own native unit file?
> >
> > systemd is not doing the conversion automatically.
> >
> > Output of "systemctl cat matrix-gui-2.0.service" with the current patch:
> > [[0;1;34m# /lib/systemd/system/matrix-gui-2.0.service[[0m
> > [Unit]
> > Description=Matrix GUI
> > After=weston.service
> >
> > [Service]
> > Type=forking
> > ExecStart=/etc/init.d/matrix-gui-2.0 start
> >
> > [Install]
> > WantedBy=multi-user.target
> >
> > Output of "systemctl cat matrix-gui-2.0.service" before the patch:
> > [[0;1;34m# /run/systemd/generator.late/matrix-gui-2.0.service[[0m
>
> Does this file also exist with the patch?
No, /run/systemd/generator.late/matrix-gui-2.0.service does not exist with the
patch.
With the information so far, would the current patch be fine to use?
>
>
> > # Automatically generated by systemd-sysv-generator
> >
> > [Unit]
> > Documentation=man:systemd-sysv-generator(8)
> > SourcePath=/etc/init.d/matrix-gui-2.0
> > Before=multi-user.target
> > Before=multi-user.target
> > Before=multi-user.target
> > Before=thermal-zone-init.service
> > After=thttpd.service
> > After=umountnfs.service
> > After=rng-tools.service
> > After=redis-server.service
> > After=dropbear.service
> > After=telnetd.service
> > After=gdbserverproxy.service
> > After=weston.service
> > After=rc.pvr.service
> > After=uim-sysfs.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
> > ExecStop=/etc/init.d/matrix-gui-2.0 stop
> >
> > >
> > >
> > > > > > > > +
> > > > > > > > +[Install]
> > > > > > > > +WantedBy=multi-user.target
> > > > > > > > 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 d8aa00b..c79f13e 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,14 +6,13 @@ LIC_FILES_CHKSUM =
> > > > > > > "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"
> > > > > > > >
> > > > > > > > SECTION = "multimedia"
> > > > > > > >
> > > > > > > > -PR = "r35"
> > > > > > > > +PR = "r36"
> > > > > > > >
> > > > > > > > -INITSCRIPT_NAME = "matrix-gui-2.0"
> > > > > > > > -INITSCRIPT_PARAMS = "defaults 97"
> > > > > > > > +SYSTEMD_SERVICE_${PN} = "matrix-gui-2.0.service"
> > > > > > > >
> > > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > > > > > >
> > > > > > > > -inherit update-rc.d
> > > > > > > > +inherit systemd
> > > > > > > >
> > > > > > > > BRANCH ?= "master"
> > > > > > > > SRCREV = "9a2d12590102fefc5c29fc9e8d346ce6b0198468"
> > > > > > > > @@ -23,6 +22,7 @@ MATRIX_INITSCRIPT =
> > > > > > > "${@base_conditional('QT_PROVIDER', 'qt5', bb.utils.contains
> > > > > > > > SRC_URI = "git://git.ti.com/matrix-gui-v2/matrix-gui-
> > > > > > > v2.git;protocol=git;branch=${BRANCH} \
> > > > > > > > file://${MATRIX_INITSCRIPT} \
> > > > > > > > file://php.ini \
> > > > > > > > + file://matrix-gui-2.0.service \
> > > > > > > > ${@base_conditional('QT_PROVIDER', 'qt5',
> > > > > > > bb.utils.contains('DISTRO_FEATURES', 'wayland', '', 'file://0001-
> > > > > > > execute_command-Stop-matrix-when-running-a-GUI-
> demo.patch',
> > > d), '',
> > > > > d)}
> > > > > > > \
> > > > > > > > ${@bb.utils.contains('DISTRO_FEATURES', 'wayland',
> > > 'file://0001-
> > > > > > > execute_command-workaround-for-GUI-apps-with-
> weston.patch', '',
> > > d)}
> > > > > \
> > > > > > > > "
> > > > > > > > @@ -46,10 +46,13 @@ do_install(){
> > > > > > > > sed -i -e "s/__MATRIX_FLAGS__/\"${MATRIX_FLAGS}\"/"
> > > > > > > ${WORKDIR}/${MATRIX_INITSCRIPT}
> > > > > > > > sed -i -e
> > > > > > >
> "s/__SWITCH_FOREGROUND_VT__/${SWITCH_FOREGROUND_VT}/"
> > > > > > > ${WORKDIR}/${MATRIX_INITSCRIPT}
> > > > > > > >
> > > > > > > > - # Install the init script
> > > > > > > > - # TODO: replace init script with systemd files
> > > > > > > > + # Install the script
> > > > > > > > install -d ${D}${sysconfdir}/init.d
> > > > > > > > install -m 0755 ${WORKDIR}/${MATRIX_INITSCRIPT}
> > > > > > > ${D}${sysconfdir}/init.d/matrix-gui-2.0
> > > > > > > > +
> > > > > > > > + # Install the systemd unit file
> > > > > > > > + install -d ${D}${systemd_system_unitdir}
> > > > > > > > + install -m 0644 ${WORKDIR}/matrix-gui-2.0.service
> > > > > > > ${D}${systemd_system_unitdir}
> > > > > > > > }
> > > > > > > >
> > > > > > > > GUIDEPS = "matrix-gui-browser refresh-screen"
> > > > > > > > --
> > > > > > > > 1.9.1
> > > > > > > >
> > > > > > > > _______________________________________________
> > > > > > > > meta-arago mailing list
> > > > > > > > meta-arago@arago-project.org
> > > > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > >
> >
_______________________________________________
meta-arago mailing list
meta-arago@arago-project.org
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
