-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20119/#review39788
-----------------------------------------------------------


Patch looks great!

Reviews applied: [20119]

All tests passed.

- Mesos ReviewBot


On April 8, 2014, 2:21 p.m., Timothy St. Clair wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20119/
> -----------------------------------------------------------
> 
> (Updated April 8, 2014, 2:21 p.m.)
> 
> 
> Review request for mesos and Benjamin Hindman.
> 
> 
> Bugs: MESOS-1071 and MESOS-1168
>     https://issues.apache.org/jira/browse/MESOS-1071
>     https://issues.apache.org/jira/browse/MESOS-1168
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Allows preventing the use of bundled libraries in favor of preinstalled 
> versions, where possible.
> 
> --disable-bundled generally switches from bundled to preinstalled libraries.
> 
> If --disable-bundled is supplied or --with-leveldb[=DIR] was given but the 
> configuration phase could not locate the dependency, it stops with an error.
> If --disable-bundled is not supplied and no --with-leveldb[=DIR] was given, 
> the bundled library is built and used (DEFAULT). 
> 
> NOTE: 
> 1. This review only implements this preinstalled library usage on LevelDB & 
> ZooKeeper. Subsequent review requests for other bundled libraries will follow 
> as soon as this implementation has been accepted.
> 
> 2. This modification removed the old --without-installed-PACKAGE semantics, 
> as they conflict with well established --with-PACKAGE semantics
> 
> 
> Diffs
> -----
> 
>   3rdparty/Makefile.am 23ff1fa 
>   configure.ac c1de6d7 
>   src/Makefile.am 95f133d 
>   src/python/setup.py.in 02f00ef 
> 
> Diff: https://reviews.apache.org/r/20119/diff/
> 
> 
> Testing
> -------
> 
> make check across all combinatoric incantations. 
> 
> 
> Thanks,
> 
> Timothy St. Clair
> 
>

Reply via email to