----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26426/#review58108 -----------------------------------------------------------
Ship it! Just noting that in our code base we've about picked either [action-if-not-given], or [action-if-give] but we should probably do both. - Timothy St. Clair On Oct. 23, 2014, 8:59 p.m., Cody Maloney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26426/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2014, 8:59 p.m.) > > > Review request for mesos, Benjamin Hindman and Timothy St. Clair. > > > Repository: mesos-git > > > Description > ------- > > Reworks buiding mesos in a "debug" vs. a "release" configuration. By default, > mesos is built in a developer-centric setup (No optimizations, minimal debug > info), in order to maximize developer productivity > > None: '-O0 -g1' > --enable-optimize == '-O2' > --enable-debug == '-g' > --enable-optimize --enable-debug == '-O2 -g' > > If a user / developer passes CXXFLAGS or CFLAGS manually, then they are not > changed / touched at all. This is important so that Mesos is a good citizen > when being built for various distributions (As well as making it so > specialized one-off groupings of flags are feasible to use). > > Adds two defines for accessing what mode things are being built in: 'DEBUG' > and 'OPTIMIZE' which can be hooked into later to enable extra logging and the > like. For release builds we may want to set 'NDEBUG' which removes > assert()'s, but that is a seperate discussion. > > > Diffs > ----- > > configure.ac 2b372e06006250b5230956ef096473e98f3fa590 > > Diff: https://reviews.apache.org/r/26426/diff/ > > > Testing > ------- > > Tested all four combinations of flags, making sure they set the right > CXXFLAGS. Also specified > > > Thanks, > > Cody Maloney > >
