----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28082/#review61755 -----------------------------------------------------------
when we have the HTTP API, there will be no need for a shared libmesos. In fact, we're considering having wholly static slave and master executables to avoid some deployment issues we have with partial updates[1]. i understand this shrinks the final size, but we will probably end up undoing this at some point. 1. libmesos updates, mesos-fetcher gets the new one, slave still uses the old one. - Dominic Hamon On Nov. 14, 2014, 6:17 p.m., Cody Maloney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28082/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2014, 6:17 p.m.) > > > Review request for mesos and Timothy St. Clair. > > > Repository: mesos-git > > > Description > ------- > > Shrinks the size resulting from a 'make install' considerably. > > On a build with the options: > ../configure --enable-optimize build > > du -h of the install directory gives: > 52MB with patch > 113MB without patch > > Debug builds the difference will be even more noticeable. > > Distribution packaging guidelines for Debian, Fedora, and Ubuntu all remove > static libraries. In the few cases where someone needs a static library, it > is still buildable. Developers should be linking against the dynamic libmesos. > > The rest of the unnecessary size comes from 2 places > > - 22MB We effectively have two copies of libmesos (libmesos.so, and one in > the python native binding) > - 5-15MB We export all symbols in the .so, rather than just the necessary > ones (Combination of lots of extra strings in the binary, and excess > non-optimizable code). > > > Diffs > ----- > > configure.ac c287a17307fb821fc04ded67ff9f33d27947e5d7 > > Diff: https://reviews.apache.org/r/28082/diff/ > > > Testing > ------- > > Built and compared install directory sizes on local machine > Make distcheck on ubuntu 14.04 > > > Thanks, > > Cody Maloney > >