New dependency for kdelibs and kdebase: dbusmenu-qt

Christoph Feck christoph at maxiom.de
Thu Apr 29 12:01:25 BST 2010


On Thursday 29 April 2010 11:38:25 Aurélien Gâteau wrote:
> On 26/04/2010 13:12, Christoph Feck wrote:
> > In FindQJSON.cmake, you try to find the include file "parser.h", but
> > later include <qjson/parser.h>. This then fails with the error "include
> > file qjson/parser.h not found", because it apparently tries to look at
> > "qjson/qjson/parser.h". I had to change FindQJSON.cmake. Diff below.
> 
> Including "parser.h" felt a bit ugly anyway, or at least too generic.
> Thanks for fixing. FindQJSON.cmake was created by libqjson author, so I
> will report the fix to him as well.

Note that I did not commit the change, as it needs to be discussed with QJson 
author first. I am not even sure if I have commit rights to gitorious, so 
please apply the change if it is correct.

Christoph Feck (kdepepo)




More information about the kde-core-devel mailing list