[PATCH] ezx: Use pulseaudio 0.9.15

[PATCH] ezx: Use pulseaudio 0.9.15

[Antonio Ospite]

Pulseaudio versions newer than 0.9.15 don't build on ezx machines (and
presumably on any armv5te), stick to this version until the problem is
handled upstream and a patch is available.

Cfr: http://bugs.openembedded.org/show_bug.cgi?id=5404

Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
---
 conf/machine/include/motorola-ezx-base.inc |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/conf/machine/include/motorola-ezx-base.inc b/conf/machine/include/motorola-ezx-base.inc
index 75c0f42..4d17211 100644
--- a/conf/machine/include/motorola-ezx-base.inc
+++ b/conf/machine/include/motorola-ezx-base.inc
@@ -2,6 +2,7 @@ TARGET_ARCH = "arm"
 
 PREFERRED_PROVIDER_xserver = "xserver-kdrive"
 PREFERRED_PROVIDER_virtual/kernel ?= "openezx-kernel"
+PREFERRED_VERSION_pulseaudio = "0.9.15"
 
 #cat /proc/mtd
 #dev:    size   erasesize  name
-- 
1.7.0

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Koen Kooi]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 28-02-10 19:57, Antonio Ospite wrote:
> Pulseaudio versions newer than 0.9.15 don't build on ezx machines (and
> presumably on any armv5te), stick to this version until the problem is
> handled upstream and a patch is available.
> 
> Cfr: http://bugs.openembedded.org/show_bug.cgi?id=5404
> 
> Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
> ---
>  conf/machine/include/motorola-ezx-base.inc |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/conf/machine/include/motorola-ezx-base.inc b/conf/machine/include/motorola-ezx-base.inc
> index 75c0f42..4d17211 100644
> --- a/conf/machine/include/motorola-ezx-base.inc
> +++ b/conf/machine/include/motorola-ezx-base.inc
> @@ -2,6 +2,7 @@ TARGET_ARCH = "arm"
>  
>  PREFERRED_PROVIDER_xserver = "xserver-kdrive"
>  PREFERRED_PROVIDER_virtual/kernel ?= "openezx-kernel"
> +PREFERRED_VERSION_pulseaudio = "0.9.15"

NAK: machines have no business setting versions (except for kernels and
even that is debatable).

If it doesn't build, fix your distro.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFLiso5MkyGM64RGpERAlimAKCeTk+/aznFhmSWt+jrrARo3pU7/ACfbTVe
23RWoqCv4bOfPPRzj/0fKfg=
=v16E
-----END PGP SIGNATURE-----

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Holger Hans Peter Freyther]

On Sunday 28 February 2010 20:55:37 Koen Kooi wrote:

> NAK: machines have no business setting versions (except for kernels and
> even that is debatable).
> 
> If it doesn't build, fix your distro.

Hi Koen,

you are right on the content but you sound rather harsh. It would be nice if 
you could use some fill words like "please" and sharing ideas how he can 
resolve his issue. E.g. are you aware of a patch to pulseaudio to work on 
armv5t and lower?


z.

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Philip Balister]

On 02/28/2010 02:55 PM, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 28-02-10 19:57, Antonio Ospite wrote:
>> Pulseaudio versions newer than 0.9.15 don't build on ezx machines (and
>> presumably on any armv5te), stick to this version until the problem is
>> handled upstream and a patch is available.
>>
>> Cfr: http://bugs.openembedded.org/show_bug.cgi?id=5404
>>
>> Signed-off-by: Antonio Ospite<ospite@studenti.unina.it>
>> ---
>>   conf/machine/include/motorola-ezx-base.inc |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/conf/machine/include/motorola-ezx-base.inc b/conf/machine/include/motorola-ezx-base.inc
>> index 75c0f42..4d17211 100644
>> --- a/conf/machine/include/motorola-ezx-base.inc
>> +++ b/conf/machine/include/motorola-ezx-base.inc
>> @@ -2,6 +2,7 @@ TARGET_ARCH = "arm"
>>
>>   PREFERRED_PROVIDER_xserver = "xserver-kdrive"
>>   PREFERRED_PROVIDER_virtual/kernel ?= "openezx-kernel"
>> +PREFERRED_VERSION_pulseaudio = "0.9.15"
>
> NAK: machines have no business setting versions (except for kernels and
> even that is debatable).
>
> If it doesn't build, fix your distro.

I'm with Koen here, machines may be used by other distros so forcing 
revision selection for recipes (other than directly impact the machine) 
has no business in machine files.

Philip

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Michael 'Mickey' Lauer]

Am Sonntag, den 28.02.2010, 16:49 -0500 schrieb Philip Balister:
> On 02/28/2010 02:55 PM, Koen Kooi wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > On 28-02-10 19:57, Antonio Ospite wrote:
> >> Pulseaudio versions newer than 0.9.15 don't build on ezx machines (and
> >> presumably on any armv5te), stick to this version until the problem is
> >> handled upstream and a patch is available.
> >>
> >> Cfr: http://bugs.openembedded.org/show_bug.cgi?id=5404
> >>
> >> Signed-off-by: Antonio Ospite<ospite@studenti.unina.it>
> >> ---
> >>   conf/machine/include/motorola-ezx-base.inc |    1 +
> >>   1 files changed, 1 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/conf/machine/include/motorola-ezx-base.inc b/conf/machine/include/motorola-ezx-base.inc
> >> index 75c0f42..4d17211 100644
> >> --- a/conf/machine/include/motorola-ezx-base.inc
> >> +++ b/conf/machine/include/motorola-ezx-base.inc
> >> @@ -2,6 +2,7 @@ TARGET_ARCH = "arm"
> >>
> >>   PREFERRED_PROVIDER_xserver = "xserver-kdrive"
> >>   PREFERRED_PROVIDER_virtual/kernel ?= "openezx-kernel"
> >> +PREFERRED_VERSION_pulseaudio = "0.9.15"
> >
> > NAK: machines have no business setting versions (except for kernels and
> > even that is debatable).
> >
> > If it doesn't build, fix your distro.
> 
> I'm with Koen here, machines may be used by other distros so forcing 
> revision selection for recipes (other than directly impact the machine) 
> has no business in machine files.

Of course one could argue that there is a bug in the OE recipes metadata
rather than the distro, since the pulseaudio recipes are known broken
for certain architectures -- however they don't contain any blacklisting
or whitelisting except for om-gta01/om-gta02 which I added when I became
aware of the problem. Antonio, lets just blacklist the broken versions
for the machines you're interested in.

:M:

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Antonio Ospite]

[-- Attachment #1: Type: text/plain, Size: 2086 bytes --]

On Sun, 28 Feb 2010 23:04:58 +0100
"Michael 'Mickey' Lauer" <mickey@vanille-media.de> wrote:

> Am Sonntag, den 28.02.2010, 16:49 -0500 schrieb Philip Balister:
> > On 02/28/2010 02:55 PM, Koen Kooi wrote:
> > >
> > > On 28-02-10 19:57, Antonio Ospite wrote:
[...]
> > >> Cfr: http://bugs.openembedded.org/show_bug.cgi?id=5404
[...]
> > > NAK: machines have no business setting versions (except for kernels and
> > > even that is debatable).
> > >
> > > If it doesn't build, fix your distro.
> > 
> > I'm with Koen here, machines may be used by other distros so forcing 
> > revision selection for recipes (other than directly impact the machine) 
> > has no business in machine files.
>

Be patient but my knowledge of OE is still limited, the point here is
that distros can override PREFERRED_VERSIONs set in machine files, isn't
it?

> Of course one could argue that there is a bug in the OE recipes metadata
> rather than the distro, since the pulseaudio recipes are known broken
> for certain architectures -- however they don't contain any blacklisting
> or whitelisting except for om-gta01/om-gta02 which I added when I became
> aware of the problem. Antonio, lets just blacklist the broken versions
> for the machines you're interested in.
>
> :M:

I can well add DEFAULT_PREFERENCE_om-a780 = "-1"
in both pulseaudio_0.9.21.bb and pulseaudio_0.9.19.bb
Is that what you mean Mickey?
Way better than my solution but still a bit hackish.
If that's OK for you, it'll be fine with me as well.

About fixing distro, isn't 'minimal' supposed to build all the latest
recipes? If so a  conf/distro/include/preferred-minimal-versions.inc
would not have much sense if we can't set preference rules per
machines (or archs).

Thanks,
   Antonio

-- 
Antonio Ospite
http://ao2.it

PGP public key ID: 0x4553B001

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

[-- Attachment #2: Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Dr. Michael Lauer]

Hi Antonio,

> Be patient but my knowledge of OE is still limited, the point here is
> that distros can override PREFERRED_VERSIONs set in machine files, isn't
> it?

Machines should not limit versions, only providers; distros and local configurations are
the ones supposed to specify versions. Then again, this assumption relies on the
"default" configuration being working, which -- in case of pulseaudio -- it obviously
is not.

>> Of course one could argue that there is a bug in the OE recipes metadata
>> rather than the distro, since the pulseaudio recipes are known broken
>> for certain architectures -- however they don't contain any blacklisting
>> or whitelisting except for om-gta01/om-gta02 which I added when I became
>> aware of the problem. Antonio, lets just blacklist the broken versions
>> for the machines you're interested in.
>> 
>> :M:
> 
> I can well add DEFAULT_PREFERENCE_om-a780 = "-1"
> in both pulseaudio_0.9.21.bb and pulseaudio_0.9.19.bb
> Is that what you mean Mickey?

Yes.

> Way better than my solution but still a bit hackish.

Not necessarily. It would be better if we could limit it for armv4(t) and armv5(t),
however we don't have such fine granular overrides, hence we need to blacklist
on machine basis.

> If that's OK for you, it'll be fine with me as well.

:)

> About fixing distro, isn't 'minimal' supposed to build all the latest
> recipes? If so a  conf/distro/include/preferred-minimal-versions.inc
> would not have much sense if we can't set preference rules per
> machines (or archs).

We can set preference rules per machine or arch, however the general
consensus is that this is being done in the distro or local conf.

Yes that codifies machine knowledge in distro and may have to be revisited,
but at least it has been our working style until now.

:M:

Re: [PATCH] ezx: Use pulseaudio 0.9.15

[Marcin Juszkiewicz]

Dnia niedziela, 28 lutego 2010 o 20:55:37 Koen Kooi napisał(a):
> > diff --git a/conf/machine/include/motorola-ezx-base.inc
> > b/conf/machine/include/motorola-ezx-base.inc index 75c0f42..4d17211
> > 100644
> > --- a/conf/machine/include/motorola-ezx-base.inc
> > +++ b/conf/machine/include/motorola-ezx-base.inc
> > @@ -2,6 +2,7 @@ TARGET_ARCH = "arm"
> >
> >  PREFERRED_PROVIDER_xserver = "xserver-kdrive"
> >  PREFERRED_PROVIDER_virtual/kernel ?= "openezx-kernel"
> > +PREFERRED_VERSION_pulseaudio = "0.9.15"
> 
> NAK: machines have no business setting versions (except for kernels and
> even that is debatable).
> 
> If it doesn't build, fix your distro.

Agreed with Koen - what if my Umbaumba distro really, really require other 
version of pulseaudio because using 0.9.15 makes device explode? Ok, it is a 
bit hardcode description but it is to show that machines should not set any 
versions.

Regards, 
-- 
JID:      hrw@jabber.org
Website:  http://marcin.juszkiewicz.com.pl/
LinkedIn: http://www.linkedin.com/in/marcinjuszkiewicz