Details
-
Bug
-
Status: Product Review
-
Trivial
-
Resolution: Fixed
-
-
-
Code Review
-
Description
On standalone, indra/cmake/FindJsonCpp.cmake sets JSONCPP_INCLUDE_DIR. However indra/newview/CMakeLists.txt declares not the path(s) in that variable, but instead those in JSONCPP_INCLUDE_DIRS an include dir, although that variable isn't set on standalone. (On non-standalone, JSONCPP_INCLUDE_DIRS is set in indra/cmake/JsonCpp.cmake.
So even when the JsonCpp headers are found alright, the corresponding dir isn't used as an include dir. Even on standalone, this didn't have any effect most of the time, because that dir happened to be one of the default include dirs, which don't have to be explicitly passed for headers in them being found.