[PATCH] kdeplasma-addons - only build rss/news plasmoids if rss dataengine is guessed to be available

Sune Vuorela debian at pusling.com
Thu May 6 18:05:27 CEST 2010

On Wednesday 05 May 2010 08:10:06 Thiago Macieira wrote:
> Em Quarta-feira 5. Maio 2010, às 03.11.42, Maciej Mrozowski escreveu:
> > Ok to commit?
> I'd say no. A runtime dependency could be installed after the code is
> built, so your change is technically wrong.

I agree. Please no more build time checks for runtime dependencies. We have 
enough to patch away of these already.

Instead of not building it, you can do a warning.

message(Kdepimlibs not found. if this is the same build environment as 
kdebase/workspace, you will get a nonfunctional foo and bar plasma widget. if 
you are distributor, you can probably ignore this)

And if there still is some all around, let's get rid of them.


More information about the Kde-buildsystem mailing list