[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: New dependency for kdelibs and kdebase: dbusmenu-qt
From:       Aurélien_Gâteau <agateau () kde ! org>
Date:       2010-05-07 10:25:07
Message-ID: 4BE3EA83.4010306 () kde ! org
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]