----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23086/#review48049 -----------------------------------------------------------
configure.ac <https://reviews.apache.org/r/23086/#comment84305> end with period. configure.ac <https://reviews.apache.org/r/23086/#comment84336> would PKG_CHECK_MODULES work if protobuf is installed in a non-standard location that user specified with --with-protobuf=<dir>? mpi/mpiexec-mesos.in <https://reviews.apache.org/r/23086/#comment84321> period at the end. src/examples/java/test-exception-framework.in <https://reviews.apache.org/r/23086/#comment84333> What happens when the user gives --with-protobuf=/path/to/non-standard/location? is the jar still expected in /usr/share/java? is it possible to plumb through the user provided protobuf path to these scripts? that seems better than silently trying to pick the jar from a standard location? src/examples/python/test-framework.in <https://reviews.apache.org/r/23086/#comment84335> ditto. saying "failed" and proceeding seems weird. is the egg expected to be on the python path? if yes, we should at least tell the users so? - Vinod Kone On June 26, 2014, 7:31 p.m., Timothy St. Clair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23086/ > ----------------------------------------------------------- > > (Updated June 26, 2014, 7:31 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Bugs: MESOS-1174 > https://issues.apache.org/jira/browse/MESOS-1174 > > > Repository: mesos-git > > > Description > ------- > > Enable mesos to build against system installed version of protobuf. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/Makefile.am 429c956 > 3rdparty/libprocess/configure.ac 27c6ede > configure.ac 41f3c2b > mpi/mpiexec-mesos.in da0733f > src/Makefile.am 3c8b91a > src/examples/java/test-exception-framework.in 065f3c0 > src/examples/java/test-executor.in 35559d2 > src/examples/java/test-framework.in 9af48f9 > src/examples/java/test-log.in 8afed0d > src/examples/java/test-multiple-executors-framework.in 235eb23 > src/examples/python/test-containerizer.in 569519b > src/examples/python/test-executor.in 7e8875f > src/examples/python/test-framework.in c4683b9 > src/python/setup.py.in b996dfe > > Diff: https://reviews.apache.org/r/23086/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy St. Clair > >
