[pulseaudio-discuss] [PATCH] qpaeq: Try to load equalizer module before failing, better error messages

[Tanu Kaskinen]

On Sun, 2012-03-25 at 16:47 +0200, Matěj Laitl wrote:
> On 23. 3. 2012 Colin Guthrie wrote:
> > 'Twas brillig, and Matěj Laitl at 19/03/12 21:44 did gyre and gimble:
> > > This fixes bug 38728 [1]. When equalizer features are unavailable in
> > > running pulseaudio daemon, try to load relevant module. If this fails,
> > > following error is printed on stderr instead of a confusing tracebac...
> > 
> > This is a great start thank you
> > 
> > As it's a GUI program that could be started via a menu or .desktop file,
> > it would be nice to warn the user via a GUI friendly way when things go
> > really wrong.
> > 
> > That would be a good next step if you are keen!
> 
> Okay, I will do this later, with one obvious caveat that we cannot show "Qt 
> not found" dialog using Qt. :-)

What are your plans regarding this? Should we wait for an improved
patch, or should the first version be merged?

> Also, it seem that we don't ship .desktop file for qpaeq, should we?

I guess it wouldn't at least do any harm.

-- 
Tanu