* [PATCH 0/5] Various bits related to pulseaudio
@ 2015-08-24 22:19 Christopher Larson
2015-08-24 22:19 ` [PATCH 1/5] libunwind: always depend on libatomics-ops Christopher Larson
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw)
To: openembedded-core; +Cc: Christopher Larson
From: Christopher Larson <chris_larson@mentor.com>
The following changes since commit c38acd720b3f6ffbeb544063692eb471dada8593:
binconfig-disabled: write an message to stderr to help confused developers (2015-08-19 17:57:58 +0100)
are available in the git repository at:
git://github.com/kergoth/openembedded-core pulseaudio-related-bits
https://github.com/kergoth/openembedded-core/tree/pulseaudio-related-bits
Christopher Larson (5):
libunwind: always depend on libatomics-ops
libatomic-ops: move to recipes-support, fix recipe name
packagegroup-core-x11-sato: obey the pulseaudio distro feature
pulseaudio: pass --with-systemduserunitdir
pulseaudio: add 'autospawn-for-root' PACKAGECONFIG
meta/recipes-devtools/guile/guile_2.0.11.bb | 2 +-
meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 22 ++++++++++++++++++++--
.../packagegroups/packagegroup-core-x11-sato.bb | 3 +--
meta/recipes-support/bdwgc/bdwgc_7.4.2.bb | 2 +-
.../libatomic-ops/libatomic-ops_7.4.2.bb} | 1 +
meta/recipes-support/libunwind/libunwind.inc | 11 +----------
6 files changed, 25 insertions(+), 16 deletions(-)
rename meta/{recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb => recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb} (96%)
--
2.2.1
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH 1/5] libunwind: always depend on libatomics-ops 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson @ 2015-08-24 22:19 ` Christopher Larson 2015-08-24 22:19 ` [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name Christopher Larson ` (3 subsequent siblings) 4 siblings, 0 replies; 11+ messages in thread From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw) To: openembedded-core - Depend on libatomics-ops unconditionally for all archs, as it's used by more than just arm, and currently the configure script auto-detects and enables its use based on the existence of its header, resulting in non-deterministic builds. - Drop the ldflags bits which link against libatomic_ops, because libunwind doesn't actually link against it, nor does it need to, as it currently uses all inline functions from the header file. Signed-off-by: Christopher Larson <kergoth@gmail.com> --- meta/recipes-support/libunwind/libunwind.inc | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/meta/recipes-support/libunwind/libunwind.inc b/meta/recipes-support/libunwind/libunwind.inc index 406bbe4..c8eab58 100644 --- a/meta/recipes-support/libunwind/libunwind.inc +++ b/meta/recipes-support/libunwind/libunwind.inc @@ -2,6 +2,7 @@ DESCRIPTION = "a portable and efficient C programming interface (API) to determi HOMEPAGE = "http://www.nongnu.org/libunwind" LICENSE = "MIT" LIC_FILES_CHKSUM = "file://LICENSE;md5=3fced11d6df719b47505837a51c16ae5" +DEPENDS += "libatomics-ops" SRC_URI = "${SAVANNAH_NONGNU_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz" @@ -18,11 +19,6 @@ ATOMICOPS_armv5 = "-DAO_USE_PTHREAD_DEFS=1" ATOMICOPS_armv4 = "-DAO_USE_PTHREAD_DEFS=1" ATOMICOPS ?= "" -LDFLAGS += "${LIBATOMICS}" -LIBATOMICS_armv5 = "-latomic_ops" -LIBATOMICS_armv4 = "-latomic_ops" -LIBATOMICS ?= "" - LDFLAGS_append_x86 = " -fuse-ld=gold" LDFLAGS_append_x86-64 = " -fuse-ld=gold" @@ -32,9 +28,4 @@ LDFLAGS_append_aarch64 = " -fuse-ld=gold" LDFLAGS_append_powerpc = " -fuse-ld=gold" LDFLAGS_append_powerpc64 = " -fuse-ld=gold" -DEPENDS += "${DEPLIBATOMICS}" -DEPLIBATOMICS_armv5 = "libatomics-ops" -DEPLIBATOMICS_armv4 = "libatomics-ops" -DEPLIBATOMICS ?= "" - BBCLASSEXTEND = "native" -- 2.2.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson 2015-08-24 22:19 ` [PATCH 1/5] libunwind: always depend on libatomics-ops Christopher Larson @ 2015-08-24 22:19 ` Christopher Larson 2015-08-25 0:32 ` Khem Raj 2015-08-24 22:19 ` [PATCH 3/5] packagegroup-core-x11-sato: obey the pulseaudio distro feature Christopher Larson ` (2 subsequent siblings) 4 siblings, 1 reply; 11+ messages in thread From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw) To: openembedded-core - This recipe is useful for more than just pulseaudio, so move it to recipes-support. - Rename to the correct upstream name, which corresponds to the library name. Keep a PROVIDES of libatomics-ops for compatibility. Signed-off-by: Christopher Larson <kergoth@gmail.com> --- meta/recipes-devtools/guile/guile_2.0.11.bb | 2 +- meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 2 +- meta/recipes-support/bdwgc/bdwgc_7.4.2.bb | 2 +- .../libatomic-ops/libatomic-ops_7.4.2.bb} | 1 + meta/recipes-support/libunwind/libunwind.inc | 2 +- 5 files changed, 5 insertions(+), 4 deletions(-) rename meta/{recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb => recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb} (96%) diff --git a/meta/recipes-devtools/guile/guile_2.0.11.bb b/meta/recipes-devtools/guile/guile_2.0.11.bb index 71f696e..4869694 100644 --- a/meta/recipes-devtools/guile/guile_2.0.11.bb +++ b/meta/recipes-devtools/guile/guile_2.0.11.bb @@ -37,7 +37,7 @@ BBCLASSEXTEND = "native" DEPENDS = "libunistring bdwgc gmp libtool libffi ncurses readline" # add guile-native only to the target recipe's DEPENDS -DEPENDS_append_class-target = " guile-native libatomics-ops" +DEPENDS_append_class-target = " guile-native libatomic-ops" RDEPENDS_${PN}_append_libc-glibc_class-target = "glibc-gconv-iso8859-1" diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc index 3389ec9..a098473 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://GPL;md5=4325afd396febcb659c36b49533135d4 \ file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \ file://src/pulsecore/resampler.h;beginline=4;endline=21;md5=09794012ae16912c0270f3280cc8ff84" -DEPENDS = "libatomics-ops liboil libsndfile1 libtool" +DEPENDS = "libatomic-ops liboil libsndfile1 libtool" # optional DEPENDS += "udev alsa-lib glib-2.0 dbus gconf" DEPENDS += "json-c gdbm speexdsp libxml-parser-perl-native libcap" diff --git a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb index 23ec954..1ca8cf7 100644 --- a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb +++ b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb @@ -36,6 +36,6 @@ inherit autotools pkgconfig # by default use external libatomic-ops PACKAGECONFIG ??= "libatomic-ops" -PACKAGECONFIG[libatomic-ops] = "--with-libatomic-ops=yes,--with-libatomic-ops=no,libatomics-ops" +PACKAGECONFIG[libatomic-ops] = "--with-libatomic-ops=yes,--with-libatomic-ops=no,libatomic-ops" BBCLASSEXTEND = "native nativesdk" diff --git a/meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb b/meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb similarity index 96% rename from meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb rename to meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb index 26faabb..4d0e299 100644 --- a/meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb +++ b/meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb @@ -1,6 +1,7 @@ SUMMARY = "A library for atomic integer operations" HOMEPAGE = "https://github.com/ivmai/libatomic_ops/" SECTION = "optional" +PROVIDES += "libatomics-ops" LICENSE = "GPLv2 & MIT" LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \ file://doc/LICENSING.txt;md5=e00dd5c8ac03a14c5ae5225a4525fa2d \ diff --git a/meta/recipes-support/libunwind/libunwind.inc b/meta/recipes-support/libunwind/libunwind.inc index c8eab58..6660af2 100644 --- a/meta/recipes-support/libunwind/libunwind.inc +++ b/meta/recipes-support/libunwind/libunwind.inc @@ -2,7 +2,7 @@ DESCRIPTION = "a portable and efficient C programming interface (API) to determi HOMEPAGE = "http://www.nongnu.org/libunwind" LICENSE = "MIT" LIC_FILES_CHKSUM = "file://LICENSE;md5=3fced11d6df719b47505837a51c16ae5" -DEPENDS += "libatomics-ops" +DEPENDS += "libatomic-ops" SRC_URI = "${SAVANNAH_NONGNU_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz" -- 2.2.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name 2015-08-24 22:19 ` [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name Christopher Larson @ 2015-08-25 0:32 ` Khem Raj 2015-08-25 15:39 ` Christopher Larson 0 siblings, 1 reply; 11+ messages in thread From: Khem Raj @ 2015-08-25 0:32 UTC (permalink / raw) To: Christopher Larson; +Cc: openembedded-core [-- Attachment #1: Type: text/plain, Size: 4930 bytes --] > On Aug 24, 2015, at 3:19 PM, Christopher Larson <kergoth@gmail.com> wrote: > > - This recipe is useful for more than just pulseaudio, so move it to > recipes-support. > - Rename to the correct upstream name, which corresponds to the library name. > Keep a PROVIDES of libatomics-ops for compatibility. > There have been few fixes especially for gcc5 see https://github.com/ivmai/libatomic_ops/commit/437c10b73ae4b83306f77c851cf8385172ae3759 can you also bring those in while you are at it ? > Signed-off-by: Christopher Larson <kergoth@gmail.com> > --- > meta/recipes-devtools/guile/guile_2.0.11.bb | 2 +- > meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 2 +- > meta/recipes-support/bdwgc/bdwgc_7.4.2.bb | 2 +- > .../libatomic-ops/libatomic-ops_7.4.2.bb} | 1 + > meta/recipes-support/libunwind/libunwind.inc | 2 +- > 5 files changed, 5 insertions(+), 4 deletions(-) > rename meta/{recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb => recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb} (96%) > > diff --git a/meta/recipes-devtools/guile/guile_2.0.11.bb b/meta/recipes-devtools/guile/guile_2.0.11.bb > index 71f696e..4869694 100644 > --- a/meta/recipes-devtools/guile/guile_2.0.11.bb > +++ b/meta/recipes-devtools/guile/guile_2.0.11.bb > @@ -37,7 +37,7 @@ BBCLASSEXTEND = "native" > > DEPENDS = "libunistring bdwgc gmp libtool libffi ncurses readline" > # add guile-native only to the target recipe's DEPENDS > -DEPENDS_append_class-target = " guile-native libatomics-ops" > +DEPENDS_append_class-target = " guile-native libatomic-ops" > > RDEPENDS_${PN}_append_libc-glibc_class-target = "glibc-gconv-iso8859-1" > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > index 3389ec9..a098473 100644 > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://GPL;md5=4325afd396febcb659c36b49533135d4 \ > file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \ > file://src/pulsecore/resampler.h;beginline=4;endline=21;md5=09794012ae16912c0270f3280cc8ff84" > > -DEPENDS = "libatomics-ops liboil libsndfile1 libtool" > +DEPENDS = "libatomic-ops liboil libsndfile1 libtool" > # optional > DEPENDS += "udev alsa-lib glib-2.0 dbus gconf" > DEPENDS += "json-c gdbm speexdsp libxml-parser-perl-native libcap" > diff --git a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb > index 23ec954..1ca8cf7 100644 > --- a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb > +++ b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb > @@ -36,6 +36,6 @@ inherit autotools pkgconfig > > # by default use external libatomic-ops > PACKAGECONFIG ??= "libatomic-ops" > -PACKAGECONFIG[libatomic-ops] = "--with-libatomic-ops=yes,--with-libatomic-ops=no,libatomics-ops" > +PACKAGECONFIG[libatomic-ops] = "--with-libatomic-ops=yes,--with-libatomic-ops=no,libatomic-ops" > > BBCLASSEXTEND = "native nativesdk" > diff --git a/meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb b/meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb > similarity index 96% > rename from meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb > rename to meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb > index 26faabb..4d0e299 100644 > --- a/meta/recipes-multimedia/pulseaudio/libatomics-ops_7.4.2.bb > +++ b/meta/recipes-support/libatomic-ops/libatomic-ops_7.4.2.bb > @@ -1,6 +1,7 @@ > SUMMARY = "A library for atomic integer operations" > HOMEPAGE = "https://github.com/ivmai/libatomic_ops/" > SECTION = "optional" > +PROVIDES += "libatomics-ops" > LICENSE = "GPLv2 & MIT" > LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \ > file://doc/LICENSING.txt;md5=e00dd5c8ac03a14c5ae5225a4525fa2d \ > diff --git a/meta/recipes-support/libunwind/libunwind.inc b/meta/recipes-support/libunwind/libunwind.inc > index c8eab58..6660af2 100644 > --- a/meta/recipes-support/libunwind/libunwind.inc > +++ b/meta/recipes-support/libunwind/libunwind.inc > @@ -2,7 +2,7 @@ DESCRIPTION = "a portable and efficient C programming interface (API) to determi > HOMEPAGE = "http://www.nongnu.org/libunwind" > LICENSE = "MIT" > LIC_FILES_CHKSUM = "file://LICENSE;md5=3fced11d6df719b47505837a51c16ae5" > -DEPENDS += "libatomics-ops" > +DEPENDS += "libatomic-ops" > > SRC_URI = "${SAVANNAH_NONGNU_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz" > > -- > 2.2.1 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core [-- Attachment #2: Message signed with OpenPGP using GPGMail --] [-- Type: application/pgp-signature, Size: 211 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name 2015-08-25 0:32 ` Khem Raj @ 2015-08-25 15:39 ` Christopher Larson 0 siblings, 0 replies; 11+ messages in thread From: Christopher Larson @ 2015-08-25 15:39 UTC (permalink / raw) To: Khem Raj; +Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 860 bytes --] On Mon, Aug 24, 2015 at 5:32 PM, Khem Raj <raj.khem@gmail.com> wrote: > > On Aug 24, 2015, at 3:19 PM, Christopher Larson <kergoth@gmail.com> > wrote: > > > > - This recipe is useful for more than just pulseaudio, so move it to > > recipes-support. > > - Rename to the correct upstream name, which corresponds to the library > name. > > Keep a PROVIDES of libatomics-ops for compatibility. > > > > There have been few fixes especially for gcc5 see > > https://github.com/ivmai/libatomic_ops/commit/437c10b73ae4b83306f77c851cf8385172ae3759 > can you also bring those in while you are at it ? I'll add a todo to do that as an additional patch so it doesn't block this going in. -- Christopher Larson kergoth at gmail dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics [-- Attachment #2: Type: text/html, Size: 1452 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/5] packagegroup-core-x11-sato: obey the pulseaudio distro feature 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson 2015-08-24 22:19 ` [PATCH 1/5] libunwind: always depend on libatomics-ops Christopher Larson 2015-08-24 22:19 ` [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name Christopher Larson @ 2015-08-24 22:19 ` Christopher Larson 2015-08-24 22:19 ` [PATCH 4/5] pulseaudio: pass --with-systemduserunitdir Christopher Larson 2015-08-24 22:19 ` [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG Christopher Larson 4 siblings, 0 replies; 11+ messages in thread From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw) To: openembedded-core; +Cc: Christopher Larson From: Christopher Larson <chris_larson@mentor.com> Signed-off-by: Christopher Larson <chris_larson@mentor.com> --- meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb b/meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb index 04d1e99..e0b1f51 100644 --- a/meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb +++ b/meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb @@ -39,8 +39,7 @@ RDEPENDS_${PN}-base = "\ libsdl \ ${NETWORK_MANAGER} \ udev-extraconf \ - pulseaudio-server \ - pulseaudio-client-conf-sato \ + ${@bb.utils.contains('DISTRO_FEATURES', 'pulseaudio', 'pulseaudio-server pulseaudio-client-conf-sato', '', d)} \ " # pcmanfm doesn't work on mips -- 2.2.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 4/5] pulseaudio: pass --with-systemduserunitdir 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson ` (2 preceding siblings ...) 2015-08-24 22:19 ` [PATCH 3/5] packagegroup-core-x11-sato: obey the pulseaudio distro feature Christopher Larson @ 2015-08-24 22:19 ` Christopher Larson 2015-08-24 22:19 ` [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG Christopher Larson 4 siblings, 0 replies; 11+ messages in thread From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw) To: openembedded-core; +Cc: Christopher Larson From: Christopher Larson <chris_larson@mentor.com> We want to make sure our path variables are obeyed. Signed-off-by: Christopher Larson <chris_larson@mentor.com> --- meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc index a098473..6199dcd 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc @@ -43,7 +43,7 @@ PACKAGECONFIG[bluez4] = "--enable-bluez4,--disable-bluez4,bluez4 sbc" PACKAGECONFIG[bluez5] = "--enable-bluez5,--disable-bluez5,bluez5 sbc" PACKAGECONFIG[ofono] = "--enable-bluez5-ofono-headset,--disable-bluez5-ofono-headset,ofono" PACKAGECONFIG[gtk] = "--enable-gtk3,--disable-gtk3,gtk+3" -PACKAGECONFIG[systemd] = "--enable-systemd-daemon --enable-systemd-login --enable-systemd-journal,--disable-systemd-daemon --disable-systemd-login --disable-systemd-journal,systemd" +PACKAGECONFIG[systemd] = "--enable-systemd-daemon --enable-systemd-login --enable-systemd-journal --with-systemduserunitdir=${systemd_user_unitdir},--disable-systemd-daemon --disable-systemd-login --disable-systemd-journal,systemd" PACKAGECONFIG[x11] = "--enable-x11,--disable-x11,virtual/libx11 libxtst libice libsm libxcb" PACKAGECONFIG[avahi] = "--enable-avahi,--disable-avahi,avahi" PACKAGECONFIG[jack] = "--enable-jack,--disable-jack,jack" -- 2.2.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson ` (3 preceding siblings ...) 2015-08-24 22:19 ` [PATCH 4/5] pulseaudio: pass --with-systemduserunitdir Christopher Larson @ 2015-08-24 22:19 ` Christopher Larson 2015-08-25 8:35 ` Tanu Kaskinen 4 siblings, 1 reply; 11+ messages in thread From: Christopher Larson @ 2015-08-24 22:19 UTC (permalink / raw) To: openembedded-core; +Cc: Christopher Larson From: Christopher Larson <chris_larson@mentor.com> Since many embedded systems don't have non-root users, it's useful to be able to use pulseaudio autospawn for root as well. Signed-off-by: Christopher Larson <chris_larson@mentor.com> --- meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc index 6199dcd..70009d0 100644 --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc @@ -47,6 +47,11 @@ PACKAGECONFIG[systemd] = "--enable-systemd-daemon --enable-systemd-login --enabl PACKAGECONFIG[x11] = "--enable-x11,--disable-x11,virtual/libx11 libxtst libice libsm libxcb" PACKAGECONFIG[avahi] = "--enable-avahi,--disable-avahi,avahi" PACKAGECONFIG[jack] = "--enable-jack,--disable-jack,jack" +# Since many embedded systems don't have non-root users, it's useful to be +# able to use pulseaudio autospawn for root as well. Autospawn will only be +# used if the systemd packageconfig is not enabled, otherwise it will rely on +# systemd's user services to spawn it. +PACKAGECONFIG[autospawn-for-root] = ",,," EXTRA_OECONF_append_arm = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " --enable-neon-opt=no", d)}" EXTRA_OECONF_append_armeb = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " --enable-neon-opt=no", d)}" @@ -59,6 +64,19 @@ export TARGET_PFPU = "${TARGET_FPU}" OE_LT_RPATH_ALLOW = "any" OE_LT_RPATH_ALLOW[export]="1" +set_cfg_value () { + sed -i -e "s/\(; *\)\?$2 =.*/$2 = $3/" "$1" + if ! grep -q "^$2 = $3\$" "$1"; then + die "Use of sed to set '$2' to '$3' in '$1' failed" + fi +} + +do_compile_append () { + if ${@bb.utils.contains('PACKAGECONFIG', 'autospawn-for-root', 'true', 'false', d)}; then + set_cfg_value src/client.conf allow-autospawn-for-root yes + fi +} + do_install_append() { install -d ${D}${sysconfdir}/default/volatiles install -m 0644 ${WORKDIR}/volatiles.04_pulse ${D}${sysconfdir}/default/volatiles/volatiles.04_pulse -- 2.2.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG 2015-08-24 22:19 ` [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG Christopher Larson @ 2015-08-25 8:35 ` Tanu Kaskinen 2015-08-25 14:26 ` Christopher Larson 0 siblings, 1 reply; 11+ messages in thread From: Tanu Kaskinen @ 2015-08-25 8:35 UTC (permalink / raw) To: Christopher Larson, openembedded-core; +Cc: Christopher Larson On Mon, 2015-08-24 at 15:19 -0700, Christopher Larson wrote: > From: Christopher Larson <chris_larson@mentor.com> > > Since many embedded systems don't have non-root users, it's useful to be able > to use pulseaudio autospawn for root as well. > > Signed-off-by: Christopher Larson <chris_larson@mentor.com> > --- > meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 18 > ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > index 6199dcd..70009d0 100644 > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > @@ -47,6 +47,11 @@ PACKAGECONFIG[systemd] = "--enable-systemd-daemon --enable-systemd-login --enabl > PACKAGECONFIG[x11] = "--enable-x11,--disable-x11,virtual/libx11 libxtst libice libsm libxcb" > PACKAGECONFIG[avahi] = "--enable-avahi,--disable-avahi,avahi" > PACKAGECONFIG[jack] = "--enable-jack,--disable-jack,jack" > +# Since many embedded systems don't have non-root users, it's useful to be > +# able to use pulseaudio autospawn for root as well. Autospawn will only be > +# used if the systemd packageconfig is not enabled, otherwise it will rely on > +# systemd's user services to spawn it. The latter sentence is incorrect. Having systemd enabled does not currently have any effect on autospawning. The systemd service and socket files get installed, but not enabled by default, because the D -Bus session socket is not available to services that are started by systemd, which causes some trouble for PulseAudio. Once D-Bus works better out of the box, systemd socket activation will be enabled for PulseAudio, but until then we will rely on PulseAudio's own autospawning. > +PACKAGECONFIG[autospawn-for-root] = ",,," > > EXTRA_OECONF_append_arm = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " --enable-neon-opt=no", d)}" > EXTRA_OECONF_append_armeb = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " --enable-neon-opt=no", d)}" > @@ -59,6 +64,19 @@ export TARGET_PFPU = "${TARGET_FPU}" > OE_LT_RPATH_ALLOW = "any" > OE_LT_RPATH_ALLOW[export]="1" > > +set_cfg_value () { > + sed -i -e "s/\(; *\)\?$2 =.*/$2 = $3/" "$1" > + if ! grep -q "^$2 = $3\$" "$1"; then > + die "Use of sed to set '$2' to '$3' in '$1' failed" > + fi > +} > + > +do_compile_append () { > + if ${@bb.utils.contains('PACKAGECONFIG', 'autospawn-for-root', 'true', 'false', d)}; then > + set_cfg_value src/client.conf allow-autospawn-for-root yes > + fi > +} I think a configure option like --allow-autospawn-for-root-by-default would be nicer than patching configuration files like this. What do you (and others) think? I'm not against applying this patch for now, but if I'm not alone with this preference, I'll add the configure option later myself. -- Tanu ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG 2015-08-25 8:35 ` Tanu Kaskinen @ 2015-08-25 14:26 ` Christopher Larson 2015-08-25 16:08 ` Tanu Kaskinen 0 siblings, 1 reply; 11+ messages in thread From: Christopher Larson @ 2015-08-25 14:26 UTC (permalink / raw) To: Tanu Kaskinen; +Cc: Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 3838 bytes --] On Tue, Aug 25, 2015 at 1:35 AM, Tanu Kaskinen <tanuk@iki.fi> wrote: > On Mon, 2015-08-24 at 15:19 -0700, Christopher Larson wrote: > > From: Christopher Larson <chris_larson@mentor.com> > > > > Since many embedded systems don't have non-root users, it's useful to be > able > > to use pulseaudio autospawn for root as well. > > > > Signed-off-by: Christopher Larson <chris_larson@mentor.com> > > --- > > meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 18 > > ++++++++++++++++++ > > 1 file changed, 18 insertions(+) > > > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > > index 6199dcd..70009d0 100644 > > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc > > @@ -47,6 +47,11 @@ PACKAGECONFIG[systemd] = "--enable-systemd-daemon > --enable-systemd-login --enabl > > PACKAGECONFIG[x11] = "--enable-x11,--disable-x11,virtual/libx11 libxtst > libice libsm libxcb" > > PACKAGECONFIG[avahi] = "--enable-avahi,--disable-avahi,avahi" > > PACKAGECONFIG[jack] = "--enable-jack,--disable-jack,jack" > > +# Since many embedded systems don't have non-root users, it's useful to > be > > +# able to use pulseaudio autospawn for root as well. Autospawn will > only be > > +# used if the systemd packageconfig is not enabled, otherwise it will > rely on > > +# systemd's user services to spawn it. > > The latter sentence is incorrect. Having systemd enabled does not > currently have any effect on autospawning. The systemd service and > socket files get installed, but not enabled by default, because the D > -Bus session socket is not available to services that are started by > systemd, which causes some trouble for PulseAudio. Once D-Bus works > better out of the box, systemd socket activation will be enabled for > PulseAudio, but until then we will rely on PulseAudio's own > autospawning. > Ah, thanks for the clarification. I'll just remove the second sentence entirely. > +PACKAGECONFIG[autospawn-for-root] = ",,," > > > > EXTRA_OECONF_append_arm = "${@bb.utils.contains("TUNE_FEATURES", > "neon", "", " --enable-neon-opt=no", d)}" > > EXTRA_OECONF_append_armeb = "${@bb.utils.contains("TUNE_FEATURES", > "neon", "", " --enable-neon-opt=no", d)}" > > @@ -59,6 +64,19 @@ export TARGET_PFPU = "${TARGET_FPU}" > > OE_LT_RPATH_ALLOW = "any" > > OE_LT_RPATH_ALLOW[export]="1" > > > > +set_cfg_value () { > > + sed -i -e "s/\(; *\)\?$2 =.*/$2 = $3/" "$1" > > + if ! grep -q "^$2 = $3\$" "$1"; then > > + die "Use of sed to set '$2' to '$3' in '$1' failed" > > + fi > > +} > > + > > +do_compile_append () { > > + if ${@bb.utils.contains('PACKAGECONFIG', 'autospawn-for-root', > 'true', 'false', d)}; then > > + set_cfg_value src/client.conf allow-autospawn-for-root yes > > + fi > > +} > > I think a configure option like --allow-autospawn-for-root-by-default > would be nicer than patching configuration files like this. What do you > (and others) think? I'm not against applying this patch for now, but if > I'm not alone with this preference, I'll add the configure option later > myself. It needs to stay a runtime configuration file option for administrators, no? We're just changing the default. So I'm not sure that having a configure option do it or us do it really changes all that much, particularly since it's probably unlikely that upstream would be interested in such a patch. That said, I don't feel at all strongly about it, so if others would rather see a configure option, go for it. -- Christopher Larson kergoth at gmail dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics [-- Attachment #2: Type: text/html, Size: 5128 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG 2015-08-25 14:26 ` Christopher Larson @ 2015-08-25 16:08 ` Tanu Kaskinen 0 siblings, 0 replies; 11+ messages in thread From: Tanu Kaskinen @ 2015-08-25 16:08 UTC (permalink / raw) To: Christopher Larson; +Cc: Patches and discussions about the oe-core layer On Tue, 2015-08-25 at 07:26 -0700, Christopher Larson wrote: > On Tue, Aug 25, 2015 at 1:35 AM, Tanu Kaskinen <tanuk@iki.fi> wrote: > > > I think a configure option like --allow-autospawn-for-root-by-default > > would be nicer than patching configuration files like this. What do you > > (and others) think? I'm not against applying this patch for now, but if > > I'm not alone with this preference, I'll add the configure option later > > myself. > > > It needs to stay a runtime configuration file option for administrators, > no? We're just changing the default. So I'm not sure that having a > configure option do it or us do it really changes all that much, > particularly since it's probably unlikely that upstream would be interested > in such a patch. Yes, the configure option would not change things much, the only point would be to make it slightly easier for embedded system builders to change the configuration. Setting a configure option should be simpler than patching the configuration file. Having this in upstream would make sense, because this issue is not specific to OpenEmbedded. -- Tanu ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-08-25 16:08 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-08-24 22:19 [PATCH 0/5] Various bits related to pulseaudio Christopher Larson 2015-08-24 22:19 ` [PATCH 1/5] libunwind: always depend on libatomics-ops Christopher Larson 2015-08-24 22:19 ` [PATCH 2/5] libatomic-ops: move to recipes-support, fix recipe name Christopher Larson 2015-08-25 0:32 ` Khem Raj 2015-08-25 15:39 ` Christopher Larson 2015-08-24 22:19 ` [PATCH 3/5] packagegroup-core-x11-sato: obey the pulseaudio distro feature Christopher Larson 2015-08-24 22:19 ` [PATCH 4/5] pulseaudio: pass --with-systemduserunitdir Christopher Larson 2015-08-24 22:19 ` [PATCH 5/5] pulseaudio: add 'autospawn-for-root' PACKAGECONFIG Christopher Larson 2015-08-25 8:35 ` Tanu Kaskinen 2015-08-25 14:26 ` Christopher Larson 2015-08-25 16:08 ` Tanu Kaskinen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox