> On Oct. 23, 2014, 9:50 p.m., Dominic Hamon wrote: > > configure.ac, line 289 > > <https://reviews.apache.org/r/26426/diff/3/?file=731214#file731214line289> > > > > might be simpler to set a default then override: > > > > debug="-g1" > > if ... > > debug="-g" > > else if ... > > debug="" > > fi > > > > optimize="-O0" > > if ... > > optimize="-O2" > > fi
Updated for the debug case where there is a defualt then a "in other cases do this, in other cases the default should be different". For optimize I like that I see I will definitively get either -O0 or -O2 based on which way the if goes. - Cody ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26426/#review58118 ----------------------------------------------------------- On Oct. 24, 2014, 9:29 p.m., Cody Maloney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26426/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2014, 9:29 p.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, 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 > >
