All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-ivi][PATCH] Make pulseaudio run by default
@ 2014-07-30 10:44 Sujith H
  2014-07-30 12:46 ` Sarbu, Florin-Ionut (Florin)
  0 siblings, 1 reply; 5+ messages in thread
From: Sujith H @ 2014-07-30 10:44 UTC (permalink / raw)
  To: holger.behrens, florin.sarbu; +Cc: yocto, Sujith H

From: Sujith H <Sujith_Haridasan@mentor.com>

This change will help pulseaudio run by default on the
target. On the bootup user can see pulseaudio service
started.

Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com>
---
 meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
index e32efa7..5401c66 100644
--- a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
+++ b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
@@ -7,7 +7,7 @@ PR = "r1"
 inherit systemd
 
 SYSTEMD_PACKAGES = "${PN}-server"
-SYSTEMD_SERVICE = "pulseaudio.service"
+SYSTEMD_SERVICE_${PN}-server = "pulseaudio.service"
 
 PACKAGECONFIG_remove = "bluez4"
 PACKAGECONFIG_append = " bluez5"
-- 
1.8.4



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

* Re: [meta-ivi][PATCH] Make pulseaudio run by default
  2014-07-30 10:44 Sujith H
@ 2014-07-30 12:46 ` Sarbu, Florin-Ionut (Florin)
  0 siblings, 0 replies; 5+ messages in thread
From: Sarbu, Florin-Ionut (Florin) @ 2014-07-30 12:46 UTC (permalink / raw)
  Cc: yocto@yoctoproject.org, Sujith_Haridasan@mentor.com

[-- Attachment #1: Type: text/plain, Size: 1303 bytes --]



-----Original Message-----
From: Sujith H [sujith.h@gmail.com]
Received: Wednesday, 30 Jul 2014, 13:49
To: Behrens, Holger [Holger.Behrens@windriver.com]; Sarbu, Florin-Ionut (Florin) [Florin.Sarbu@windriver.com]
CC: yocto@yoctoproject.org [yocto@yoctoproject.org]; Sujith H [Sujith_Haridasan@mentor.com]
Subject: [meta-ivi][PATCH] Make pulseaudio run by default

From: Sujith H <Sujith_Haridasan@mentor.com>

This change will help pulseaudio run by default on the
target. On the bootup user can see pulseaudio service
started.

Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com>
---
 meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
index e32efa7..5401c66 100644
--- a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
+++ b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
@@ -7,7 +7,7 @@ PR = "r1"
 inherit systemd

 SYSTEMD_PACKAGES = "${PN}-server"
-SYSTEMD_SERVICE = "pulseaudio.service"
+SYSTEMD_SERVICE_${PN}-server = "pulseaudio.service"

 PACKAGECONFIG_remove = "bluez4"
 PACKAGECONFIG_append = " bluez5"
--
1.8.4


[-- Attachment #2: Type: text/html, Size: 2273 bytes --]

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

* [meta-ivi][PATCH] Make pulseaudio run by default
@ 2014-07-30 13:19 Sujith H
  2014-07-30 14:59 ` Andrei Gherzan
  2014-09-08 21:33 ` Andrei Gherzan
  0 siblings, 2 replies; 5+ messages in thread
From: Sujith H @ 2014-07-30 13:19 UTC (permalink / raw)
  To: holger.behrens, andrei.gherzan; +Cc: yocto, Sujith H

From: Sujith H <Sujith_Haridasan@mentor.com>

This change will help pulseaudio run by default on the
target. On the bootup user can see pulseaudio service
started.

Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com>
---
 meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
index e32efa7..5401c66 100644
--- a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
+++ b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
@@ -7,7 +7,7 @@ PR = "r1"
 inherit systemd
 
 SYSTEMD_PACKAGES = "${PN}-server"
-SYSTEMD_SERVICE = "pulseaudio.service"
+SYSTEMD_SERVICE_${PN}-server = "pulseaudio.service"
 
 PACKAGECONFIG_remove = "bluez4"
 PACKAGECONFIG_append = " bluez5"
-- 
1.8.4



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

* Re: [meta-ivi][PATCH] Make pulseaudio run by default
  2014-07-30 13:19 [meta-ivi][PATCH] Make pulseaudio run by default Sujith H
@ 2014-07-30 14:59 ` Andrei Gherzan
  2014-09-08 21:33 ` Andrei Gherzan
  1 sibling, 0 replies; 5+ messages in thread
From: Andrei Gherzan @ 2014-07-30 14:59 UTC (permalink / raw)
  To: Sujith H; +Cc: yocto, Sujith H

Hi Sujith,

First of all please format summary following:
http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines

On Wed, Jul 30, 2014 at 06:49:01PM +0530, Sujith H wrote:
> From: Sujith H <Sujith_Haridasan@mentor.com>
>
> This change will help pulseaudio run by default on the
> target. On the bootup user can see pulseaudio service
> started.
>
> Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com>
> ---
>  meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> index e32efa7..5401c66 100644
> --- a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> +++ b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> @@ -7,7 +7,7 @@ PR = "r1"
>  inherit systemd
>
>  SYSTEMD_PACKAGES = "${PN}-server"
> -SYSTEMD_SERVICE = "pulseaudio.service"
> +SYSTEMD_SERVICE_${PN}-server = "pulseaudio.service"

There is no functional change between these too. (read
http://patchwork.openembedded.org/patch/29049/). In both cases the services
ends up in pulseaudio-server package.

The autostart feature is handled by SYSTEMD_AUTO_ENABLE which has as the
default value "enable". So, pulseaudio should be up and running in the current
setup.

>
>  PACKAGECONFIG_remove = "bluez4"
>  PACKAGECONFIG_append = " bluez5"
> --
> 1.8.4
>

--
Andrei Gherzan | Wind River Systems
m: +40 734 450 615 | p: +40 236 320 307


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

* Re: [meta-ivi][PATCH] Make pulseaudio run by default
  2014-07-30 13:19 [meta-ivi][PATCH] Make pulseaudio run by default Sujith H
  2014-07-30 14:59 ` Andrei Gherzan
@ 2014-09-08 21:33 ` Andrei Gherzan
  1 sibling, 0 replies; 5+ messages in thread
From: Andrei Gherzan @ 2014-09-08 21:33 UTC (permalink / raw)
  To: Sujith H; +Cc: yocto, Sujith H

Hello,

On Wed, Jul 30, 2014 at 06:49:01PM +0530, Sujith H wrote:
> From: Sujith H <Sujith_Haridasan@mentor.com>
>
> This change will help pulseaudio run by default on the
> target. On the bootup user can see pulseaudio service
> started.
>
> Signed-off-by: Sujith H <Sujith_Haridasan@mentor.com>
> ---
>  meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> index e32efa7..5401c66 100644
> --- a/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> +++ b/meta-ivi/recipes-multimedia/pulseaudio/pulseaudio_5.0.bbappend
> @@ -7,7 +7,7 @@ PR = "r1"
>  inherit systemd
>
>  SYSTEMD_PACKAGES = "${PN}-server"
> -SYSTEMD_SERVICE = "pulseaudio.service"
> +SYSTEMD_SERVICE_${PN}-server = "pulseaudio.service"
>
>  PACKAGECONFIG_remove = "bluez4"
>  PACKAGECONFIG_append = " bluez5"
> --
> 1.8.4
>

First of all sorry for this late reply. I was able to reproduce and understand
the issue here. The idea is that the bbclass is parsing the 'per package'
SYSTEMD_SERVICE variable (SYSTEMD_SERVICE_${PN}) [systemd.bbclass line 167] and
based on this it installs the post install script.

Patch merged to master.

Thanks a lot.

--
Andrei Gherzan | Wind River Systems
m: +40 734 450 615


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

end of thread, other threads:[~2014-09-08 21:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-30 13:19 [meta-ivi][PATCH] Make pulseaudio run by default Sujith H
2014-07-30 14:59 ` Andrei Gherzan
2014-09-08 21:33 ` Andrei Gherzan
  -- strict thread matches above, loose matches on Subject: below --
2014-07-30 10:44 Sujith H
2014-07-30 12:46 ` Sarbu, Florin-Ionut (Florin)

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.