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

Ship it!



src/Makefile.am (lines 108 - 109)
<https://reviews.apache.org/r/40327/#comment165327>

    I think the emphasis is slightly off here; we aren't working around noisy 
warnings (which are all valid), but the fact that these warnings are useless to 
us as we have no intention of fixing them. I'd also drop the ref to the jira 
issue since it can be reached via the commit message.
    
    How about:
    
    > We use "-isystem" instead of "-I" to add Boost to the include search 
path. This disables compiler warnings inside Boost headers since we have no 
intention of fixing those.



src/Makefile.am (line 110)
<https://reviews.apache.org/r/40327/#comment165328>

    I think it would make sense to also update the mesos `CMakeLists.txt`.


- Benjamin Bannier


On Nov. 15, 2015, 10:31 a.m., Neil Conway wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40327/
> -----------------------------------------------------------
> 
> (Updated Nov. 15, 2015, 10:31 a.m.)
> 
> 
> Review request for mesos, Benjamin Bannier and Joris Van Remoortere.
> 
> 
> Bugs: MESOS-3799
>     https://issues.apache.org/jira/browse/MESOS-3799
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This instructs the compiler to ignore warnings in the Boost headers. This is
> useful, because those headers often contain hard-to-silence warnings that are
> not otherwise useful to display. See MESOS-3799.
> 
> 
> Diffs
> -----
> 
>   src/Makefile.am 92d07c3da897c7664c63e2af91d45921d66f64aa 
> 
> Diff: https://reviews.apache.org/r/40327/diff/
> 
> 
> Testing
> -------
> 
> Re-bootstrapped and tested compilation with Ubuntu Wily and OSX 10.10.
> 
> Note that the cmake build still emits boost-related warnings: I didn't try to 
> fix this, since cmake emits other warnings that autotools suppresses (e.g., 
> no-unused-local-typedefs).
> 
> 
> Thanks,
> 
> Neil Conway
> 
>

Reply via email to