Review Request: FindQJSON.cmake: Look for parser.h in the right location.

Raphael Kubo da Costa rakuco at freebsd.org
Sat May 12 20:18:51 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104925/
-----------------------------------------------------------

Review request for KDevelop.


Description
-------

QJSON headers are installed into qjson/qjson/<HEADER>.  The find_path()
call, however, was looking for headers in qjson/<HEADER>, which failed.

Passing PATH_SUFFIX is not enough, and given how the headers are used in
the code, we need to look for qjson/parser.h, not parser.h.


Diffs
-----

  cmake/modules/FindQJSON.cmake 002084aefd8e2c1506a0674439cc37bc2539074b 

Diff: http://git.reviewboard.kde.org/r/104925/diff/


Testing
-------

qjson was not found here before, and it is after the patch is applied.


Thanks,

Raphael Kubo da Costa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120512/9fc58e89/attachment.html>


More information about the KDevelop-devel mailing list