[ptxdist] [PATCH] pulseaudio: version bump 8.0 -> 11.0

[ptxdist] [PATCH] pulseaudio: version bump 8.0 -> 11.0

[Lucas Stach]

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
---
I'm unsure about the with-system-user, with-system-group and
with-access-group configure settings, so someone should probably
validate them.
---
 rules/pulseaudio.make | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/rules/pulseaudio.make b/rules/pulseaudio.make
index 34b6aae16a32..8b527608248c 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	:= 11.0
+PULSEAUDIO_MD5		:= c5fe1cd05f5306b1a0fba5099f0e9e37
 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 \
@@ -68,7 +69,7 @@ PULSEAUDIO_CONF_OPT	:= \
 	--disable-jack \
 	--disable-asyncns \
 	--disable-tcpwrap \
-	--disable-tcpwrap \
+	--disable-lirc \
 	--disable-dbus \
 	--disable-bluez4 \
 	--disable-bluez5 \
@@ -78,7 +79,6 @@ PULSEAUDIO_CONF_OPT	:= \
 	--disable-hal-compat \
 	$(GLOBAL_IPV6_OPTION) \
 	--disable-openssl \
-	--disable-xen \
 	--disable-gcov \
 	--enable-orc \
 	--$(call ptx/endis, PTXCONF_PULSEAUDIO_SYSTEMD)-systemd-daemon \
@@ -95,10 +95,20 @@ PULSEAUDIO_CONF_OPT	:= \
 	--disable-force-preopen \
 	--with-caps \
 	--with-database=simple \
+	--with-pulsedsp-location= \
 	--without-fftw \
 	--without-speex \
 	--without-soxr \
-	--with-systemduserunitdir=/usr/lib/systemd/user
+	--with-systemduserunitdir=/usr/lib/systemd/user \
+	--with-system-user=root \
+	--with-system-group=root \
+	--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
 
-- 
2.11.0


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] pulseaudio: version bump 8.0 -> 11.0

[Clemens Gruber]

Hi,

On Fri, Sep 08, 2017 at 05:15:18PM +0200, Lucas Stach wrote:
> Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> ---
> I'm unsure about the with-system-user, with-system-group and
> with-access-group configure settings, so someone should probably
> validate them.

Passing --with-system-user=root is not a good idea. I think we should
use a system user like nobody or a special "pulse" user (the default),
because PulseAudio can then drop privileges and change to that user
after startup.
Not sure if a tmpfiles.d file is necessary for /run/pulse or if
PulseAudio creates the directory with correct permissions.

Cheers,
Clemens

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] pulseaudio: version bump 8.0 -> 11.0

[Juergen Borleis]

Hi Lucas,

On Friday 08 September 2017 17:15:18 Lucas Stach wrote:
> [...]
> +	--with-system-user=root \
> +	--with-system-group=root \
> [...9

Like Clemens already stated: forcing to "root" is a no-go.

jb

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCH] pulseaudio: version bump 8.0 -> 11.0

[Lucas Stach]

Am Freitag, den 08.09.2017, 17:51 +0200 schrieb Clemens Gruber:
> Hi,
> 
> On Fri, Sep 08, 2017 at 05:15:18PM +0200, Lucas Stach wrote:
> > > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> > ---
> > I'm unsure about the with-system-user, with-system-group and
> > with-access-group configure settings, so someone should probably
> > validate them.
> 
> Passing --with-system-user=root is not a good idea. I think we should
> use a system user like nobody or a special "pulse" user (the default),
> because PulseAudio can then drop privileges and change to that user
> after startup.
> Not sure if a tmpfiles.d file is necessary for /run/pulse or if
> PulseAudio creates the directory with correct permissions.

Michael, I would really appreciate some guidance on how you would like
to have this handled.

Regards,
Lucas

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de