[kdepim] [Bug 402185] >=kdepim-18.12.0: Make speech dependency optional again

[bugzilla_noreply at kde.org]

https://bugs.kde.org/show_bug.cgi?id=402185

--- Comment #5 from holgersson at posteo.de ---
(In reply to Laurent Montel from comment #4)
> (In reply to Christophe Giboudeaux from comment #3)
> > Laurent could answer better. I see a couple reasons:
> > 
> > - it was optional when the TTS support in Qt wasn't part of the releases.
> > Nowadays, it's considered stable and is part .
> > 
> > - In the pim repositories, we avoid optional dependencies. The code is
> > cleaner if we don't have to disable chunks of code.
> 
> Hi,
> not necessary to give you more info as Christophe resumed correctly why I
> removed optional dependancy.
> I keep it from long time as optional until this module was stable.
> And I wanted to clean up my code.
> 
> So it's really intentional.
> 
> Regards.

OK, is there another possible way to implement it optional while keeping your
code reasonable clean, maybe as some kind of plugin? I’d be a happy tester for
patches.

-- 
You are receiving this mail because:
You are the assignee for the bug.