[oe] [PATCH] pulseaudio: Switch to version version 0.9.22 for Angstrom 2011.03

[Paul Menzel]

Am Freitag, den 08.07.2011, 19:21 -0500 schrieb Joel A Fernandes:

I do not know if the commit summary is correct and if it should be
prepended with »angstrom«.

> This fixes all the issues noticed earlier with pulseaudio 0.9.15:
>  * Pulseaudio keeping audio devices busy when run as per-user instance
>  * Failure of pactl, pacmd etc to connect to the pulseaudio server

Please always include links to corresponding list threads if they are
available.

> Signed-off-by: Joel A Fernandes <agnel.joel at gmail.com>
> ---
> Notes:
> * This has to go into the OpenEmbedded 2011.03-maintenance branch as there is no such configration file in the master branch

        $ git grep pulse conf/distro
        conf/distro/include/angstrom-2008-preferred-versions.inc:PREFERRED_VERSION_gst-pulse    = "0.9.7"
        conf/distro/include/angstrom-2008-preferred-versions.inc:PREFERRED_VERSION_pulseaudio   = "0.9.15"
        conf/distro/include/angstrom-2010-preferred-versions.inc:PREFERRED_VERSION_gst-pulse    = "0.9.7"
        conf/distro/include/angstrom-2010-preferred-versions.inc:PREFERRED_VERSION_pulseaudio   = "0.9.22"
        […]

So you your change could actually be applied to master?

> * Please ignore the earlier patch "pulseaudio: Run as system-wide instance instead of per-user"
> 
> Applying this and the patch titled "beagleboard-test-scripts: Add a script flash-fs.sh for flashing NAND" will mark the
> "Alpha" build of the C5 image. Can this be tagged?
> 
> Things to go into "Beta":
> * User button fixes (kernel and flashing scripts)
> * U-boot fixes for EHCI timeouts when doing a tftpboot

Great, but this should have gone to a separate thread.

>  conf/distro/angstrom-2008.1.conf                   |    3 +++
>  .../include/angstrom-2008-preferred-versions.inc   |    2 +-
>  2 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/conf/distro/angstrom-2008.1.conf b/conf/distro/angstrom-2008.1.conf
> index ae1ed4c..7b4f25a 100644
> --- a/conf/distro/angstrom-2008.1.conf
> +++ b/conf/distro/angstrom-2008.1.conf
> @@ -121,6 +121,9 @@ PREFERRED_VERSION_binutils_avr32 = "2.17"
>  PREFERRED_VERSION_binutils-cross_avr32 = "2.17"
>  PREFERRED_VERSION_binutils-cross-sdk_avr32 = "2.17"
>  
> +ANGSTROM_PULSEAUDIO_VERSION		= "0.9.15"
> +ANGSTROM_PULSEAUDIO_VERSION_armv7a	= "0.9.22"
> +

1. Where did you find this syntax?

	$ git grep PULSE conf/distro/

does not find anything in branch master.

2. Could you add an explanation, why 0.9.22 is not used for everything
and only for armv7a. Is that because of commit cce5e648 [1]?

>  PREFERRED_PROVIDER_dbus-glib             = "dbus-glib"
>  PREFERRED_PROVIDER_hotplug               = "udev"
>  PREFERRED_PROVIDER_opkg                 ?= "opkg"
> diff --git a/conf/distro/include/angstrom-2008-preferred-versions.inc b/conf/distro/include/angstrom-2008-preferred-versions.inc
> index b5b524a..e4b8bf1 100644
> --- a/conf/distro/include/angstrom-2008-preferred-versions.inc
> +++ b/conf/distro/include/angstrom-2008-preferred-versions.inc
> @@ -44,7 +44,7 @@ PREFERRED_VERSION_pango-native     = "1.24.4"
>  #PREFERRED_VERSION_pixman	= "0.13.2"
>  PREFERRED_VERSION_pkgconfig	= "0.23"
>  PREFERRED_VERSION_pkgconfig-native = "0.23"
> -PREFERRED_VERSION_pulseaudio 	= "0.9.15"
> +PREFERRED_VERSION_pulseaudio 	= "${ANGSTROM_PULSEAUDIO_VERSION}"
>  PREFERRED_VERSION_python-pycairo = "1.8.0"
>  PREFERRED_VERSION_qt4-tools-native = "${ANGSTROM_QT_VERSION}"
>  PREFERRED_VERSION_qt4-tools-sdk = "${ANGSTROM_QT_VERSION}"

So I would prefer, if you would change the version just in this file and
such a patch could go into branch master first.

Even better would be the following.

1. Package PA 0.9.23 for oe-dev. If I remember correctly I send all
patches upstream and it should work for all architectures(?).
2. Prefer PA 0.9.23 for Ȧngström and other distributions.
3. Remove older PA versions.
4. Cherry pick these changes into branch 2011.03-maintenance.

Joel, do you have time and motivation to do that?


Thanks,

Paul


[1] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=cce5e648222525a0619f454ddc4f2253afca2821
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110709/ccf4a1b6/attachment-0002.sig>