-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33752/#review86738
-----------------------------------------------------------
LGTM
- Cody Maloney
On June 4, 2015, 7:31 p.m., James Peach wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33752/
> -----------------------------------------------------------
>
> (Updated June 4, 2015, 7:31 p.m.)
>
>
> Review request for mesos, Cody Maloney and Timothy St. Clair.
>
>
> Bugs: MESOS-2537
> https://issues.apache.org/jira/browse/MESOS-2537
>
>
> Repository: mesos
>
>
> Description
> -------
>
> In a number of places, the Mesos configure script passes "$foo=yes"
> to the 2nd argument of AC_ARG_ENABLED. However, the 2nd argument
> is invoked when the option is provided in any form, not just when
> the --enable-foo form is used. One result of this is that
> --disable-optimize doesn't work.
>
> The correct handling of the 2nd argument is to save the value of
> "$enableval". This change sets the value of all the enable variables
> using $enableval, and sets the default value based on the option
> name.
>
> There are a number of enable options that were internally named
> "$with_foo" and "$without_foo". Rename these to "$enable_foo" for
> clarity and to remove the need for both a with_ and a without_
> version.
>
> Finally, emit the compilation flags at the end of the configure
> phase so it is easier to see the results of your configuration
> options.
>
>
> Diffs
> -----
>
> configure.ac cad7f0e92eacc86d37b3f578382946db8b466531
>
> Diff: https://reviews.apache.org/r/33752/diff/
>
>
> Testing
> -------
>
> Configure and build on CentOS 7 and Mac OS X 10.10.3. Verify that the status summary
reflects the expected compiler flags. Verify that --enable-foo and --disable-foo do different
things.
>
>
> Thanks,
>
> James Peach
>
>
|