Freeze exemption for PA integration into Phonon

[Helio Chissini de Castro]

On Wednesday 18 November 2009 18:29:19 Sebastian Kügler wrote:
> On Wednesday 18 November 2009 18:41:36 Tom Albers wrote:
> > Op Wednesday 18 November 2009 18:10 schreef u:
> > > Hi!
> > >
> > > Colin Guthrie has been working on better integration of PulseAudio in
> > > Phonon, but AFAIK it is now too late for getting it into 4.4, without
> > > getting and exemption from the freeze.
> > >
> > > But IMHO (and Eike Hein, who reminded me about this issue :-), distros
> > > will most probably just go ahead and patch in coling's work anyways, if
> > > it isn't available in KDE itself.
> > >
> > > The KDE-module in question is kdebase-runtime (and also Phonon itself,
> > > but that's another story).
> > >
> > > Another tiny snag is that it depends on an unreleased version of
> > > PulseAudio (because whoever packaged up the last PA release forgot to
> > > include coling's work), but I assume spring-release distros will have
> > > an up-to-date version, so if we just check for the right PA version
> > > while building, we should be OK.
> > >
> > > Mandriva's latest release already includes coling's work, so it has
> > > already received a fair amount of testing.
> > >
> > > So before you all start going around in circles screaming "pulseaudio
> > > prrrrfffft", if PA isn't available at build-time, all the PA-specific
> > > stuff becomes nops (and possibly completely optimized away? I don't
> > > have enough compiler-fu). And if Phonon is built with PA integration,
> > > but the PA daemon isn't available for some reason (it crashed, or isn't
> > > installed, for example), it degrades gracefully.
> > >
> > > I think this is the truth, the whole truth and nothing but the truth,
> > > so help me God, but if I'm wrong, please correct me. :-)
> > >
> > > Or ask. Or grant an exemption, if you find the kindness in your hearts
> > > to do so.
> >
> > A link to the patches would have been nice ;-)
> >
> > If there are good, working and tested cmake and ifdefs, I suggest we
> >  include it now, ship it in the beta and possibily in the first release
> >  candidate. If we get complains or unsolvable issues we can remove it for
> >  the final release if needed.
> >
> > Please wait 24h for other opinions, if noone objects or raises possible
> >  issues, please merge it in asap, so we can at least have some basic
> >  testing before we package the beta.
> 
> Good suggestion. +1
> 


Colin work have my 100% approval. I was there in the integration in Mandriva.
I would say "go".

-- 
Helio Chissini de Castro
South America and Brazil Primary Contact
KDE Developer since 2002