----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50455/#review167507 -----------------------------------------------------------
Closing this review due to inactivity. Please see our [guidelines](https://github.com/apache/mesos/blob/master/docs/reopening-reviews.md) for reopening reviews. - Joris Van Remoortere On Dec. 10, 2016, 7:18 p.m., Srinivas Brahmaroutu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50455/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2016, 7:18 p.m.) > > > Review request for mesos, Alex Clemmer and Joseph Wu. > > > Bugs: MESOS-5902 > https://issues.apache.org/jira/browse/MESOS-5902 > > > Repository: mesos > > > Description > ------- > > Using FindJava and FindJNI cmake packages to detect java on the host, > set JNI include paths and > set HAS_JAVA global variable to conditionally build Java support. > > > Diffs > ----- > > cmake/MesosConfigure.cmake 6a9ed9dc02f5c9a6d1fce0866f19ffeafec35cdc > > > Diff: https://reviews.apache.org/r/50455/diff/5/ > > > Testing > ------- > > cmake -DENABLE_JAVA=ON > check build variables set for Java. > cmake should output > -- ************************************************************ > -- Java Found and will be enabled > -- ************************************************************ > > > Thanks, > > Srinivas Brahmaroutu > >
