Re: [OE-core] [PATCH] pulseaudio: don't include consolekit when systemd is enabled

2019-08-13 Thread Tanu Kaskinen
On Tue, 2019-08-13 at 16:11 +0300, Tanu Kaskinen wrote:
> On Fri, 2019-07-26 at 23:03 +0800, Anuj Mittal wrote:
> > When using systemd, make sure that pulseaudio-server RDEPENDS on
> > module-systemd-login instead of module-console-kit both of which provide
> > the same functionality but for different init systems [1][2].
> > 
> > Even though both modules can co-exist, this helps avoid including
> > consolekit (which has been deprecated) in the images using systemd.
> > 
> > [1] 
> > https://www.freedesktop.org/wiki/Software/PulseAudio/Documentation/User/Modules/#index51h3
> > [2] 
> > https://github.com/pulseaudio/pulseaudio/commit/860d1cf3a76701ade38784822abb24285176227c
> > 
> > Signed-off-by: Anuj Mittal 
> 
> Acked-by: Tanu Kaskinen 

Never mind, this is already in master (I didn't find it initially,
which is why I sent that mail as a ping).

-- 
Tanu

https://www.patreon.com/tanuk
https://liberapay.com/tanuk

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] pulseaudio: don't include consolekit when systemd is enabled

2019-08-13 Thread Tanu Kaskinen
On Fri, 2019-07-26 at 23:03 +0800, Anuj Mittal wrote:
> When using systemd, make sure that pulseaudio-server RDEPENDS on
> module-systemd-login instead of module-console-kit both of which provide
> the same functionality but for different init systems [1][2].
> 
> Even though both modules can co-exist, this helps avoid including
> consolekit (which has been deprecated) in the images using systemd.
> 
> [1] 
> https://www.freedesktop.org/wiki/Software/PulseAudio/Documentation/User/Modules/#index51h3
> [2] 
> https://github.com/pulseaudio/pulseaudio/commit/860d1cf3a76701ade38784822abb24285176227c
> 
> Signed-off-by: Anuj Mittal 

Acked-by: Tanu Kaskinen 

> ---
>  meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 5 -
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc 
> b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> index e245be7..ec51d8b 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> @@ -259,9 +259,12 @@ FILES_${PN}-module-gsettings += 
> "${libexecdir}/pulse/gsettings-helper ${datadir}
>  # modules must be installed when X11 is enabled.
>  RDEPENDS_pulseaudio-server += "\
>  ${@bb.utils.contains('DISTRO_FEATURES', 'x11', '\
> -pulseaudio-module-console-kit \
>  pulseaudio-module-device-manager \
>  pulseaudio-module-x11-cork-request \
>  pulseaudio-module-x11-publish \
>  pulseaudio-module-x11-xsmp \
>  ', '', d)}"
> +
> +RDEPENDS_pulseaudio-server += "${@bb.utils.contains('DISTRO_FEATURES', 
> 'x11', \
> +  bb.utils.contains('DISTRO_FEATURES', 
> 'systemd', 'pulseaudio-module-systemd-login', 
> 'pulseaudio-module-console-kit', d), \
> +  '', d)}"
> -- 
> 2.7.4
> 
-- 
Tanu

https://www.patreon.com/tanuk
https://liberapay.com/tanuk

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core