From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Lucas Stach <l.stach@pengutronix.de>
Subject: Re: [ptxdist] [PATCH 3/3] pulseaudio: version bump 8.0 -> 12.2
Date: Thu, 22 Nov 2018 11:12:09 +0100 [thread overview]
Message-ID: <20181122101209.sgiqu7zfdlvvul5w@pengutronix.de> (raw)
In-Reply-To: <20181122100640.u5cdkv5hg52wrmoj@pengutronix.de>
On Thu, Nov 22, 2018 at 11:06:40AM +0100, Roland Hieber wrote:
> On Mon, Nov 05, 2018 at 04:01:17PM +0100, Lucas Stach wrote:
> > - version bump
> > - add glib support
> > - add speex resampler support
> > - add bluetooth support
> >
> > Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> > ---
> > rules/pulseaudio.in | 25 +++++++++++++++++++++++--
> > rules/pulseaudio.make | 41 ++++++++++++++++++++++++++++++++---------
> > 2 files changed, 55 insertions(+), 11 deletions(-)
> >
> > diff --git a/rules/pulseaudio.in b/rules/pulseaudio.in
> > index f59ad470e6df..13ec74b87cd5 100644
> > --- a/rules/pulseaudio.in
> > +++ b/rules/pulseaudio.in
> > @@ -8,12 +8,16 @@ config PULSEAUDIO
> > select LIBC_PTHREAD
> > select ALSA_LIB
> > select ALSA_LIB_MIXER
> > - select JSON_C
> > select LIBCAP
> > select LIBLTDL
> > select LIBSNDFILE
> > select ORC
> > - select SYSTEMD if PULSEAUDIO_SYSTEMD
> > + select SYSTEMD if PULSEAUDIO_SYSTEMD
> > + select GLIB if PULSEAUDIO_GLIB
> > + select SBC if PULSEAUDIO_BLUETOOTH
> > + select DBUS if PULSEAUDIO_BLUETOOTH
> > + select SPEEXDSP if PULSEAUDIO_SPEEX
> > + select SPEEXDSP_FLOAT_API if PULSEAUDIO_SPEEX
> > select UDEV
> > select UDEV_LIBUDEV
> > prompt "pulseaudio"
> > @@ -29,4 +33,21 @@ config PULSEAUDIO_SYSTEMD
> > bool
> > default INITMETHOD_SYSTEMD
> >
> > +config PULSEAUDIO_BLUETOOTH
> > + bool
> > + prompt "enable bluetooth support"
> > +
> > +config PULSEAUDIO_GLIB
> > + bool
> > + prompt "enable glib support"
> > +
> > +config PULSEAUDIO_SPEEX
> > + bool
> > + default y
> > + prompt "enable speex based resamplers"
> > + help
> > + The speex based resamplers provide the best cost/quality
> > + trade-off on most platforms. It's recommended to only disable
> > + this when no resampling is required.
> > +
> > endif
> > diff --git a/rules/pulseaudio.make b/rules/pulseaudio.make
> > index 8dda3501a0a9..2e3d2167aa3c 100644
> > --- a/rules/pulseaudio.make
> > +++ b/rules/pulseaudio.make
> > @@ -16,8 +16,8 @@ PACKAGES-$(PTXCONF_PULSEAUDIO) += pulseaudio
> > #
> > # Paths and names
> > #
> > -PULSEAUDIO_VERSION := 8.0
> > -PULSEAUDIO_MD5 := 8678442ba0bb4b4c33ac6f62542962df
> > +PULSEAUDIO_VERSION := 12.2
> > +PULSEAUDIO_MD5 := c42f1f1465e8df9859d023dc184734bf
> > PULSEAUDIO := pulseaudio-$(PULSEAUDIO_VERSION)
> > PULSEAUDIO_SUFFIX := tar.xz
> > PULSEAUDIO_URL := http://freedesktop.org/software/pulseaudio/releases/$(PULSEAUDIO).$(PULSEAUDIO_SUFFIX)
> > @@ -51,6 +51,7 @@ PULSEAUDIO_CONF_OPT := \
> > --enable-atomic-arm-memory-barrier \
> > --$(call ptx/endis, PTXCONF_ARCH_ARM_NEON)-neon-opt \
> > $(GLOBAL_LARGE_FILE_OPTION) \
> > + --enable-memfd \
> > --disable-x11 \
> > --disable-tests \
> > --disable-samplerate \
> > @@ -61,24 +62,25 @@ PULSEAUDIO_CONF_OPT := \
> > --disable-esound \
> > --disable-solaris \
> > --disable-waveout \
> > - --disable-glib2 \
> > + --$(call ptx/endis, PTXCONF_PULSEAUDIO_GLIB)-glib2 \
> > --disable-gtk3 \
> > + --disable-gsettings \
> > --disable-gconf \
> > + --disable-schemas-compile \
> > --disable-avahi \
> > --disable-jack \
> > --disable-asyncns \
> > --disable-tcpwrap \
> > - --disable-tcpwrap \
> > - --disable-dbus \
> > + --disable-lirc \
> > + --$(call ptx/endis, PTXCONF_PULSEAUDIO_BLUETOOTH)-dbus \
> > --disable-bluez4 \
> > - --disable-bluez5 \
> > + --$(call ptx/endis, PTXCONF_PULSEAUDIO_BLUETOOTH)-bluez5 \
>
> This suggests that a "select BLUEZ" should be present in the kconfig
> snippet, but it looks like it is missing.
Maybe only 'select BLUEZ if RUNTIME'. Lucas?
Michael
> > --disable-bluez5-ofono-headset \
> > --disable-bluez5-native-headset \
> > --enable-udev \
> > --disable-hal-compat \
> > $(GLOBAL_IPV6_OPTION) \
> > --disable-openssl \
> > - --disable-xen \
> > --disable-gcov \
> > --enable-orc \
> > --$(call ptx/endis, PTXCONF_PULSEAUDIO_SYSTEMD)-systemd-daemon \
> > @@ -95,10 +97,20 @@ PULSEAUDIO_CONF_OPT := \
> > --disable-force-preopen \
> > --with-caps \
> > --with-database=simple \
> > + --with-pulsedsp-location= \
> > --without-fftw \
> > - --without-speex \
> > + --$(call ptx/wwo, PTXCONF_PULSEAUDIO_SPEEX)-speex \
> > --without-soxr \
> > - --with-systemduserunitdir=/usr/lib/systemd/user
> > + --with-systemduserunitdir=/usr/lib/systemd/user \
> > + --with-system-user= \
> > + --with-system-group= \
> > + --with-access-group=pulse-access \
> > + --with-mac-version-min= \
> > + --with-mac-sysroot= \
> > + --with-preopen-mods=all \
> > + --with-module-dir=/usr/lib/pulse-$(PULSEAUDIO_VERSION)/modules \
> > + --with-udev-rules-dir=lib/udev/rules.d \
> > + --with-zsh-completion-dir=
> >
> > PULSEAUDIO_LDFLAGS := -Wl,-rpath,/usr/lib/pulseaudio:/usr/lib/pulse-$(PULSEAUDIO_VERSION)/modules
> >
> > @@ -129,9 +141,16 @@ $(STATEDIR)/pulseaudio.targetinstall:
> > @$(call install_alternative, pulseaudio, 0, 0, 0644, /etc/pulse/system.pa)
> > @$(call install_alternative, pulseaudio, 0, 0, 0644, /etc/pulse/default.pa)
> >
> > +ifdef PTXCONF_PULSEAUDIO_BLUETOOTH
> > + @$(call install_alternative, pulseaudio, 0, 0, 0644, \
> > + /usr/share/dbus-1/system.d/org.PulseAudio1.conf)
> > +endif
> > +
> > ifdef PTXCONF_PULSEAUDIO_SYSTEMD
> > @$(call install_alternative, pulseaudio, 0, 0, 0644, \
> > /usr/lib/systemd/system/pulseaudio.service)
> > + @$(call install_link, pulseaudio, ../pulseaudio.service, \
> > + /usr/lib/systemd/system/multi-user.target.wants/pulseaudio.service)
> > @$(call install_alternative, pulseaudio, 0, 0, 0644, \
> > /usr/lib/systemd/system/pulseaudio.socket)
> > @$(call install_link, pulseaudio, ../pulseaudio.socket, \
> > @@ -153,6 +172,10 @@ endif
> > @$(call install_lib, pulseaudio, 0, 0, 0644, pulseaudio/libpulsecommon-$(PULSEAUDIO_VERSION))
> > @$(call install_tree, pulseaudio, 0, 0, -, /usr/lib/pulse-$(PULSEAUDIO_VERSION)/modules)
> >
> > +ifdef PTXCONF_PULSEAUDIO_GLIB
> > + @$(call install_lib, pulseaudio, 0, 0, 0644, libpulse-mainloop-glib)
> > +endif
> > +
> > @$(call install_tree, pulseaudio, 0, 0, -, /usr/share/pulseaudio)
> >
> > @$(call install_finish, pulseaudio)
> > --
> > 2.19.1
> >
> >
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
>
> --
> Roland Hieber | r.hieber@pengutronix.de |
> Pengutronix e.K. | https://www.pengutronix.de/ |
> Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
next prev parent reply other threads:[~2018-11-22 10:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-05 15:01 [ptxdist] [PATCH 1/3] speex: version bump 1.2rc1 -> 1.2 Lucas Stach
2018-11-05 15:01 ` [ptxdist] [PATCH 2/3] sbc: new package Lucas Stach
2018-11-05 15:01 ` [ptxdist] [PATCH 3/3] pulseaudio: version bump 8.0 -> 12.2 Lucas Stach
2018-11-22 6:24 ` Michael Olbrich
2018-11-22 10:06 ` Roland Hieber
2018-11-22 10:12 ` Michael Olbrich [this message]
2018-11-22 10:27 ` Lucas Stach
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=20181122101209.sgiqu7zfdlvvul5w@pengutronix.de \
--to=m.olbrich@pengutronix.de \
--cc=l.stach@pengutronix.de \
--cc=ptxdist@pengutronix.de \
/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