My fault, sorry... I just forgot about that additional dependency. I think I'll be able to fix it today or tomorrow, by simply disabling chrome support when QJSON is not found.<div>Small problem: there is no builtin  CMake script for QJSON, there's one around that works quite fine, however, I've been using it for other project.</div>

<div>Should I add this script to kde-workspace? where exactly?</div><div>Thanks<br><br><div class="gmail_quote">On Thu, Aug 30, 2012 at 10:06 PM, Marco Martin <span dir="ltr"><<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thursday 30 August 2012, Martin Gräßlin wrote:<br>
> Hi all,<br>
><br>
> since 8f0f84d9575812ee1832d7fabb2c8ee1a9058112 kde-workspace depends on<br>
> QJSON. In plasma/generic/runners/bookmarks/CMakeLists.txt there is the<br>
> following check:<br>
> find_package(QJSON REQUIRED)<br>
><br>
> unfortunately the macro_log_feature is not used, so instead of getting a<br>
> nice summary of what is missing one has to search for something like [1].<br>
><br>
> Now fixing this is easy, but before doing so, I wanted to check whether it<br>
> is really intended to have QJSON as a hard requirement for kde-workspace.<br>
><br>
> To me this looks like an optional dependency and if not present the<br>
> bookmarks runner is just excluded.<br>
><br>
> Opinions?<br>
<br>
</div></div>yeah, seems an oversight.<br>
it should definitely be an optional dependency<br>
<br>
<br>
Cheers,<br>
Marco Martin<br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</blockquote></div><br></div>