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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux