From: ChenQi <Qi.Chen@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] psplash: Adding support for systemd service files
Date: Mon, 13 Oct 2014 17:05:52 +0800 [thread overview]
Message-ID: <543B95F0.8010707@windriver.com> (raw)
In-Reply-To: <CADyYWLma=JqZF=dJRysw7H4HHWSSKdgcpJe4kL3-As9pcwbK5Q@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 5547 bytes --]
Does anybody know what's the current state of this issue?
Thanks,
Chen Qi
On 08/11/2014 06:00 PM, sujith h wrote:
>
>
>
> On Fri, Aug 8, 2014 at 11:37 PM, Saul Wold <sgw@linux.intel.com
> <mailto:sgw@linux.intel.com>> wrote:
>
> On 08/08/2014 02:09 AM, Sujith H wrote:
>
> From: Sujith H <Sujith_Haridasan@mentor.com
> <mailto:Sujith_Haridasan@mentor.com>>
>
> If the DISTRO_FEATURES has systemd, then its better
> to have corresponding service files associated with
> the psplash. This change will help to do the same.
>
> Can you propose a patch to the psplash upstream which is part of
> the Yocto Project (send the patch to yocto@yoctoproject.org
> <mailto:yocto@yoctoproject.org>) and then we can do a psplash
> upgrade to include these files directly?
>
>
> Ok then I will submit the same patch to Yocto Project.
>
> Thanks,
> Sujith H
>
>
> Thanks
> Sau!
>
>
> Signed-off-by: Muzaffar Mahmood <muzaffar_mahmood@mentor.com
> <mailto:muzaffar_mahmood@mentor.com>>
> Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com
> <mailto:Sujith_Haridasan@mentor.com>>
> ---
> meta/recipes-core/psplash/files/psplash-quit.service | 11
> +++++++++++
> meta/recipes-core/psplash/files/psplash-start.service | 11
> +++++++++++
> meta/recipes-core/psplash/psplash_git.bb
> <http://psplash_git.bb> | 12 +++++++++++-
> 3 files changed, 33 insertions(+), 1 deletion(-)
> create mode 100644
> meta/recipes-core/psplash/files/psplash-quit.service
> create mode 100644
> meta/recipes-core/psplash/files/psplash-start.service
>
> diff --git
> a/meta/recipes-core/psplash/files/psplash-quit.service
> b/meta/recipes-core/psplash/files/psplash-quit.service
> new file mode 100644
> index 0000000..14bd499
> --- /dev/null
> +++ b/meta/recipes-core/psplash/files/psplash-quit.service
> @@ -0,0 +1,11 @@
> +[Unit]
> +Description=Terminate Psplash Boot Screen
> +After=psplash-start.service
> +
> +[Service]
> +Type=oneshot
> +ExecStart=/usr/bin/psplash-write QUIT
> +TimeoutSec=20
> +
> +[Install]
> +WantedBy=multi-user.target
> diff --git
> a/meta/recipes-core/psplash/files/psplash-start.service
> b/meta/recipes-core/psplash/files/psplash-start.service
> new file mode 100644
> index 0000000..502b150
> --- /dev/null
> +++ b/meta/recipes-core/psplash/files/psplash-start.service
> @@ -0,0 +1,11 @@
> +[Unit]
> +Description=Starts Psplash Boot screen
> +Wants=systemd-vconsole-setup.service
> +After=systemd-vconsole-setup.service
> systemd-udev-trigger.service systemd-udevd.service
> +DefaultDependencies=no
> +
> +[Service]
> +ExecStart=/usr/bin/psplash
> +
> +[Install]
> +WantedBy=sysinit.target
> diff --git a/meta/recipes-core/psplash/psplash_git.bb
> <http://psplash_git.bb>
> b/meta/recipes-core/psplash/psplash_git.bb <http://psplash_git.bb>
> index 628ced4..16fa0ae 100644
> --- a/meta/recipes-core/psplash/psplash_git.bb
> <http://psplash_git.bb>
> +++ b/meta/recipes-core/psplash/psplash_git.bb
> <http://psplash_git.bb>
> @@ -11,6 +11,8 @@ PR = "r15"
>
> SRC_URI = "git://git.yoctoproject.org/${BPN}
> <http://git.yoctoproject.org/$%7BBPN%7D> \
> file://psplash-init \
> + file://psplash-start.service \
> + file://psplash-quit.service \
> ${SPLASH_IMAGES}"
>
> SPLASH_IMAGES = "file://psplash-poky-img.h;outsuffix=default"
> @@ -65,11 +67,14 @@ python __anonymous() {
>
> S = "${WORKDIR}/git"
>
> -inherit autotools pkgconfig update-rc.d update-alternatives
> +inherit autotools pkgconfig update-rc.d update-alternatives
> systemd
>
> ALTERNATIVE_PRIORITY = "100"
> ALTERNATIVE_LINK_NAME[psplash] = "${bindir}/psplash"
>
> +SYSTEMD_PACKAGES =
> "${@base_contains('DISTRO_FEATURES','systemd','${PN}','',d)}"
> +SYSTEMD_SERVICE_${PN} =
> "${@base_contains('DISTRO_FEATURES','systemd','psplash-start.service
> psplash-quit.service','',d)}"
> +
> python do_compile () {
> import shutil
>
> @@ -103,6 +108,11 @@ do_install_append() {
> install -m 0755 $i ${D}${bindir}/$i
> done
> rm -f ${D}${bindir}/psplash
> +
> + if
> ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)};
> then
> + install -d ${D}${systemd_unitdir}/system
> + install -m 644 ${WORKDIR}/*.service
> ${D}/${systemd_unitdir}/system
> + fi
> }
>
> FILES_${PN} += "/mnt/.psplash"
>
>
>
>
> --
> സുജിത് ഹരിദാസന്
> Bangalore
> <Project>Contributor to KDE project
> http://fci.wikia.com/wiki/Anti-DRM-Campaign
> <Blog> http://sujithh.info
>
>
[-- Attachment #2: Type: text/html, Size: 11255 bytes --]
prev parent reply other threads:[~2014-10-13 9:04 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-08 9:09 [PATCH] psplash: Adding support for systemd service files Sujith H
2014-08-08 18:07 ` Saul Wold
2014-08-11 10:00 ` sujith h
2014-10-13 9:05 ` ChenQi [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=543B95F0.8010707@windriver.com \
--to=qi.chen@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox