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