* [poky][PATCH v4] pulseaudio
@ 2015-07-29 23:49 Yuqing Zhu
2015-07-29 23:49 ` [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7 Yuqing Zhu
0 siblings, 1 reply; 5+ messages in thread
From: Yuqing Zhu @ 2015-07-29 23:49 UTC (permalink / raw)
To: openembedded-core
Fix no process to open pulseaudio in yocto.
Yuqing Zhu (1):
pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7
...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 ++++++++++++++++++++++
.../pulseaudio/pulseaudio_6.0.bb | 1 +
2 files changed, 32 insertions(+)
create mode 100644 meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
--
1.9.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7
2015-07-29 23:49 [poky][PATCH v4] pulseaudio Yuqing Zhu
@ 2015-07-29 23:49 ` Yuqing Zhu
2015-07-29 15:57 ` Otavio Salvador
2015-07-29 20:49 ` Tanu Kaskinen
0 siblings, 2 replies; 5+ messages in thread
From: Yuqing Zhu @ 2015-07-29 23:49 UTC (permalink / raw)
To: openembedded-core
Revert "launch: Avoid specifically starting PA and rely on autospawn/socket activation"
or the pulseaudio can't be opened in yocto.
Signed-off-by: Yuqing Zhu <b54851@freescale.com>
---
...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 ++++++++++++++++++++++
.../pulseaudio/pulseaudio_6.0.bb | 1 +
2 files changed, 32 insertions(+)
create mode 100644 meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
new file mode 100644
index 0000000..669c355
--- /dev/null
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
@@ -0,0 +1,31 @@
+From 5760c6b1a3c9b6dd8e70c9d1ea78fcf192250b4a Mon Sep 17 00:00:00 2001
+From: Shengjiu Wang <shengjiu.wang@freescale.com>
+Date: Mon, 16 Mar 2015 15:30:11 +0800
+Subject: [PATCH] Revert "launch: Avoid specifically starting PA and rely on
+ autospawn/socket activation"
+
+This reverts commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7.
+
+Upstream-Status: Pending
+
+Signed-off-by: shengjiu.wang <b02247@freescale.com>
+---
+ src/daemon/start-pulseaudio-x11.in | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/src/daemon/start-pulseaudio-x11.in b/src/daemon/start-pulseaudio-x11.in
+index 16e46b1..e108645 100755
+--- a/src/daemon/start-pulseaudio-x11.in
++++ b/src/daemon/start-pulseaudio-x11.in
+@@ -17,6 +17,8 @@
+
+ set -e
+
++@PA_BINARY@ --start "$@"
++
+ if [ x"$DISPLAY" != x ] ; then
+
+ @PACTL_BINARY@ load-module module-x11-publish "display=$DISPLAY" > /dev/null
+--
+1.7.9.5
+
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
index 31e9096..10c3d7a 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
@@ -6,6 +6,7 @@ SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \
file://0001-conf-parser-add-support-for-.d-directories.patch \
file://fix-git-version-gen.patch \
file://volatiles.04_pulse \
+ file://0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch \
"
SRC_URI[md5sum] = "b691e83b7434c678dffacfa3a027750e"
SRC_URI[sha256sum] = "b50640e0b80b1607600accfad2e45aabb79d379bf6354c9671efa2065477f6f6"
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7
2015-07-29 23:49 ` [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7 Yuqing Zhu
@ 2015-07-29 15:57 ` Otavio Salvador
2015-07-29 20:49 ` Tanu Kaskinen
1 sibling, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2015-07-29 15:57 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: Patches and discussions about the oe-core layer
On Wed, Jul 29, 2015 at 8:49 PM, Yuqing Zhu <b54851@freescale.com> wrote:
> Revert "launch: Avoid specifically starting PA and rely on autospawn/socket activation"
> or the pulseaudio can't be opened in yocto.
>
> Signed-off-by: Yuqing Zhu <b54851@freescale.com>
Can you explain what this revert tries to fix?
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7
2015-07-29 23:49 ` [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7 Yuqing Zhu
2015-07-29 15:57 ` Otavio Salvador
@ 2015-07-29 20:49 ` Tanu Kaskinen
2015-07-29 20:52 ` Otavio Salvador
1 sibling, 1 reply; 5+ messages in thread
From: Tanu Kaskinen @ 2015-07-29 20:49 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: openembedded-core
On Thu, 2015-07-30 at 07:49 +0800, Yuqing Zhu wrote:
> Revert "launch: Avoid specifically starting PA and rely on autospawn/socket activation"
> or the pulseaudio can't be opened in yocto.
>
> Signed-off-by: Yuqing Zhu <b54851@freescale.com>
> ---
> ...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 ++++++++++++++++++++++
> .../pulseaudio/pulseaudio_6.0.bb | 1 +
> 2 files changed, 32 insertions(+)
> create mode 100644 meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
>
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
> new file mode 100644
> index 0000000..669c355
> --- /dev/null
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
> @@ -0,0 +1,31 @@
> +From 5760c6b1a3c9b6dd8e70c9d1ea78fcf192250b4a Mon Sep 17 00:00:00 2001
> +From: Shengjiu Wang <shengjiu.wang@freescale.com>
> +Date: Mon, 16 Mar 2015 15:30:11 +0800
> +Subject: [PATCH] Revert "launch: Avoid specifically starting PA and rely on
> + autospawn/socket activation"
> +
> +This reverts commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7.
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: shengjiu.wang <b02247@freescale.com>
> +---
> + src/daemon/start-pulseaudio-x11.in | 2 ++
> + 1 file changed, 2 insertions(+)
> +
> +diff --git a/src/daemon/start-pulseaudio-x11.in b/src/daemon/start-pulseaudio-x11.in
> +index 16e46b1..e108645 100755
> +--- a/src/daemon/start-pulseaudio-x11.in
> ++++ b/src/daemon/start-pulseaudio-x11.in
> +@@ -17,6 +17,8 @@
> +
> + set -e
> +
> ++@PA_BINARY@ --start "$@"
> ++
> + if [ x"$DISPLAY" != x ] ; then
> +
> + @PACTL_BINARY@ load-module module-x11-publish "display=$DISPLAY" > /dev/null
> +--
> +1.7.9.5
> +
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> index 31e9096..10c3d7a 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> @@ -6,6 +6,7 @@ SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \
> file://0001-conf-parser-add-support-for-.d-directories.patch \
> file://fix-git-version-gen.patch \
> file://volatiles.04_pulse \
> + file://0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch \
> "
> SRC_URI[md5sum] = "b691e83b7434c678dffacfa3a027750e"
> SRC_URI[sha256sum] = "b50640e0b80b1607600accfad2e45aabb79d379bf6354c9671efa2065477f6f6"
> --
> 1.9.1
This is safe to apply, but I'd like to understand why you think this is
needed. The commit message says that without this patch, "pulseaudio
can't be opened in yocto". PulseAudio works fine in core-image-sato at
least, so what is different in your setup? Have you disabled
autospawning for some reason?
--
Tanu
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7
2015-07-29 20:49 ` Tanu Kaskinen
@ 2015-07-29 20:52 ` Otavio Salvador
0 siblings, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2015-07-29 20:52 UTC (permalink / raw)
To: Tanu Kaskinen; +Cc: Patches and discussions about the oe-core layer
On Wed, Jul 29, 2015 at 5:49 PM, Tanu Kaskinen
<tanu.kaskinen@linux.intel.com> wrote:
> This is safe to apply, but I'd like to understand why you think this is
> needed. The commit message says that without this patch, "pulseaudio
> can't be opened in yocto". PulseAudio works fine in core-image-sato at
> least, so what is different in your setup? Have you disabled
> autospawning for some reason?
I prefer to not divert from upstream without a reasoning for it. This
is what needs to be clarified.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-07-29 20:52 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-29 23:49 [poky][PATCH v4] pulseaudio Yuqing Zhu
2015-07-29 23:49 ` [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7 Yuqing Zhu
2015-07-29 15:57 ` Otavio Salvador
2015-07-29 20:49 ` Tanu Kaskinen
2015-07-29 20:52 ` Otavio Salvador
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.