----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67985/#review206262 -----------------------------------------------------------
3rdparty/stout/configure.ac Lines 496 (patched) <https://reviews.apache.org/r/67985/#comment289159> On my machine (Ubuntu 18.04, but probably the same for all debian derivatives), the rapidjson headers get installed under `/usr/include/rapidjson/rapidjson.h`. So I suspect `AC_CHECK_HEADERS()` wouldn't be able to find it when trying to build against an unbundled version of rapidjson. - Benno Evers On July 20, 2018, 3:38 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67985/ > ----------------------------------------------------------- > > (Updated July 20, 2018, 3:38 a.m.) > > > Review request for mesos, Alexander Rukletsov and Benno Evers. > > > Bugs: MESOS-9092 > https://issues.apache.org/jira/browse/MESOS-9092 > > > Repository: mesos > > > Description > ------- > > This follows the approach used for picojson. > > > Diffs > ----- > > 3rdparty/stout/3rdparty/Makefile.am > cc831a93a57d970bc28d754acd04569780154bee > 3rdparty/stout/CMakeLists.txt 9cbb6f2a13fe9201fdebb9e9994d7725e53af083 > 3rdparty/stout/Makefile.am 44710c21d3c5a2bc2309b0fe10419712523d3187 > 3rdparty/stout/configure.ac 7e85c7efccdcbca8f3ef1173075384b0060a15c7 > > > Diff: https://reviews.apache.org/r/67985/diff/1/ > > > Testing > ------- > > Tested at the end of this chain, since this is split across > stout/libprocess/mesos. > > > Thanks, > > Benjamin Mahler > >
