[pulseaudio-discuss] Patch review status wiki page updated

[Tanu Kaskinen]

On Thu, 2014-04-17 at 19:34 +0600, Alexander E. Patrakov wrote:
> 17.04.2014 18:51, Tanu Kaskinen wrote:
> > On Thu, 2014-04-17 at 17:09 +0600, Alexander E. Patrakov wrote:
> >> 17.04.2014 15:14, Tanu Kaskinen wrote:
> >>> Patch review status updated:
> >>> http://www.freedesktop.org/wiki/Software/PulseAudio/PatchStatus/
> >>
> >> I cannot find this patch in any section of the wiki page:
> >>
> >> http://comments.gmane.org/gmane.comp.audio.pulseaudio.general/19868
> >>
> >> The patch is still relevant.
> >
> > Hmm, I thought it wasn't relevant, because it turned out that people do
> > care about module-equalizer-sink. In my understanding the point of the
> > patch was that it would be the first step in getting the code removed
> > from the tree, the next step would happen if we didn't get any
> > complaints about this (but we already did). The warning text in the
> > patch suggests this intention too:
> >
> > "The module will be removed if nobody steps up to rewrite it."
> >
> > I don't think that statement is true.
> 
> Thanks for the explanation of your viewpoint. OK, let's throw that patch 
> off the table and give the author a chance to improve the code 
> (especially since he knows about the objections), but I will resubmit it 
> after the next release if nothing changes.
> 
> My viewpoint is still that the current code is broken beyond repair, and 
> I cannot fix it, either.

I reviewed the original discussion, and I tried (unsuccessfully) to use
Veromix to see how good replacement it is. My conclusion is that I don't
want to remove module-equalizer-sink. If I'm going to break the audio
setup of people who use module-equalizer-sink, I want to be able to
comfortably say "we removed the module, because X is clearly a better
alternative". I'm not comfortable with saying that if "X" is substituted
by "Veromix". Veromix is unmaintained. It's not available in Fedora.
Trying to use it from source resulted in an empty window and a bunch of
errors in the console:

ERROR:root:Could not find any typelib for AppIndicator3
ERROR:root:Could not find any typelib for Unity
ERROR:root:Could not find any typelib for Dbusmenu

-- 
Tanu