Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH v1] connman: fix systemd support for connman-* packages.
@ 2013-08-12 16:28 Yevhen Kyriukha
  2013-08-12 22:52 ` Saul Wold
  0 siblings, 1 reply; 2+ messages in thread
From: Yevhen Kyriukha @ 2013-08-12 16:28 UTC (permalink / raw)
  To: openembedded-core; +Cc: Yevhen Kyriukha

This patch fixes systemd's postinst/postrm script generation.

Signed-off-by: Yevhen Kyriukha <kirgene@gmail.com>
---
 meta/recipes-connectivity/connman/connman.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index 6b5ea0b..c5014b7 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -56,6 +56,7 @@ PACKAGECONFIG[wispr] = "--enable-wispr,--disable-wispr,gnutls,"
 INITSCRIPT_NAME = "connman"
 INITSCRIPT_PARAMS = "start 05 5 2 3 . stop 22 0 1 6 ."
 
+SYSTEMD_PACKAGES = "${PN} ${PN}-vpn"
 SYSTEMD_SERVICE_${PN} = "connman.service"
 SYSTEMD_SERVICE_${PN}-vpn = "connman-vpn.service"
 SYSTEMD_WIRED_SETUP = "ExecStartPre=-${libdir}/connman/wired-setup"
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v1] connman: fix systemd support for connman-* packages.
  2013-08-12 16:28 [PATCH v1] connman: fix systemd support for connman-* packages Yevhen Kyriukha
@ 2013-08-12 22:52 ` Saul Wold
  0 siblings, 0 replies; 2+ messages in thread
From: Saul Wold @ 2013-08-12 22:52 UTC (permalink / raw)
  To: Yevhen Kyriukha; +Cc: openembedded-core

On 08/12/2013 09:28 AM, Yevhen Kyriukha wrote:
> This patch fixes systemd's postinst/postrm script generation.
>
> Signed-off-by: Yevhen Kyriukha <kirgene@gmail.com>
> ---
>   meta/recipes-connectivity/connman/connman.inc | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
> index 6b5ea0b..c5014b7 100644
> --- a/meta/recipes-connectivity/connman/connman.inc
> +++ b/meta/recipes-connectivity/connman/connman.inc
> @@ -56,6 +56,7 @@ PACKAGECONFIG[wispr] = "--enable-wispr,--disable-wispr,gnutls,"
>   INITSCRIPT_NAME = "connman"
>   INITSCRIPT_PARAMS = "start 05 5 2 3 . stop 22 0 1 6 ."
>
> +SYSTEMD_PACKAGES = "${PN} ${PN}-vpn"

This needs to be guarded with a check for the vpn in PACKAGECONFIG, 
otherwise a packaging error can occur since the service file is not 
available:

ERROR: Function failed: SYSTEMD_SERVICE_connman-vpn value 
connman-vpn.service does not exist

Please correct this.

Thanks
	Sau!

>   SYSTEMD_SERVICE_${PN} = "connman.service"
>   SYSTEMD_SERVICE_${PN}-vpn = "connman-vpn.service"
>   SYSTEMD_WIRED_SETUP = "ExecStartPre=-${libdir}/connman/wired-setup"
>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-08-12 22:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-12 16:28 [PATCH v1] connman: fix systemd support for connman-* packages Yevhen Kyriukha
2013-08-12 22:52 ` Saul Wold

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox