----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18957/#review36634 -----------------------------------------------------------
Patch looks great! Reviews applied: [18957] All tests passed. - Mesos ReviewBot On March 10, 2014, 10:36 a.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18957/ > ----------------------------------------------------------- > > (Updated March 10, 2014, 10:36 a.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Niklas > Nielsen, Timothy St. Clair, and Vinod Kone. > > > Bugs: MESOS-1071 > https://issues.apache.org/jira/browse/MESOS-1071 > > > Repository: mesos-git > > > Description > ------- > > Allows preventing the use of the currently bundled leveldb in favor of a > preinstalled version. > > > Diffs > ----- > > 3rdparty/Makefile.am 8e1d915 > configure.ac 390f11b > src/Makefile.am 384b312 > src/python/setup.py.in 02f00ef > > Diff: https://reviews.apache.org/r/18957/diff/ > > > Testing > ------- > > ../configure > make check > > OSX: brew installed leveldb comes with activated libsnappy > ../configure --without-included-leveldb CPPFLAGS=-I/usr/local/include > LDFLAGS=-L/usr/local/lib LIBS="-lleveldb -lsnappy" > make check > > linux: manually installed leveldb without libsnappy > ../configure --without-included-leveldb CPPFLAGS=-I/usr/local/include > LDFLAGS=-L/usr/local/lib LIBS=-lleveldb > make check > > > Thanks, > > Till Toenshoff > >
