New dependency for kdelibs and kdebase: dbusmenu-qt

Aurélien Gâteau agateau at kde.org
Fri May 7 11:25:07 BST 2010


On 29/04/2010 13:01, Christoph Feck wrote:
> 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.

Hi Christoph,

I just applied the change, thanks for the fix.

Aurélien




More information about the kde-core-devel mailing list