You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Chun-Hung Hsiao <ch...@apache.org> on 2018/08/30 04:31:41 UTC
Review Request 68564: Disabled the `-Wunknown-warning-option`
warnings for Clang 3.5.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/
-----------------------------------------------------------
Review request for mesos, Benjamin Bannier and James Peach.
Repository: mesos
Description
-------
Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
have to suppress `-Wunknown-warning-option` warnings if `-Werror` is on.
Diffs
-----
configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
Diff: https://reviews.apache.org/r/68564/diff/1/
Testing
-------
make with clang 3.5.
Thanks,
Chun-Hung Hsiao
Re: Review Request 68564: Set up `-Wno-inconsistent-missing-override`
conditionally.
Posted by Chun-Hung Hsiao <ch...@apache.org>.
> On Aug. 31, 2018, 11:36 a.m., Benjamin Bannier wrote:
> > configure.ac
> > Lines 639 (patched)
> > <https://reviews.apache.org/r/68564/diff/3/?file=2079131#file2079131line639>
> >
> > If you implement https://reviews.apache.org/r/68576/#comment291938 this would be another candidate to be moved to a more general place. The code here should be robust enough to support that.
Dropping since the flag names are different between GCC and Clang.
- Chun-Hung
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/#review208169
-----------------------------------------------------------
On Aug. 30, 2018, 9:11 p.m., Chun-Hung Hsiao wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68564/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2018, 9:11 p.m.)
>
>
> Review request for mesos, Benjamin Bannier and James Peach.
>
>
> Bugs: MESOS-9193
> https://issues.apache.org/jira/browse/MESOS-9193
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
> only add it if the tool chain supports this option.
>
>
> Diffs
> -----
>
> configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
>
>
> Diff: https://reviews.apache.org/r/68564/diff/3/
>
>
> Testing
> -------
>
> make with clang 3.5.0/3.6.0.
>
>
> Thanks,
>
> Chun-Hung Hsiao
>
>
Re: Review Request 68564: Set up `-Wno-inconsistent-missing-override`
conditionally.
Posted by Benjamin Bannier <be...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/#review208169
-----------------------------------------------------------
Fix it, then Ship it!
configure.ac
Lines 639 (patched)
<https://reviews.apache.org/r/68564/#comment291943>
If you implement https://reviews.apache.org/r/68576/#comment291938 this would be another candidate to be moved to a more general place. The code here should be robust enough to support that.
- Benjamin Bannier
On Aug. 30, 2018, 11:11 p.m., Chun-Hung Hsiao wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68564/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2018, 11:11 p.m.)
>
>
> Review request for mesos, Benjamin Bannier and James Peach.
>
>
> Bugs: MESOS-9193
> https://issues.apache.org/jira/browse/MESOS-9193
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
> only add it if the tool chain supports this option.
>
>
> Diffs
> -----
>
> configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
>
>
> Diff: https://reviews.apache.org/r/68564/diff/3/
>
>
> Testing
> -------
>
> make with clang 3.5.0/3.6.0.
>
>
> Thanks,
>
> Chun-Hung Hsiao
>
>
Re: Review Request 68564: Set up `-Wno-inconsistent-missing-override`
conditionally.
Posted by Chun-Hung Hsiao <ch...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/
-----------------------------------------------------------
(Updated Aug. 30, 2018, 9:11 p.m.)
Review request for mesos, Benjamin Bannier and James Peach.
Changes
-------
Used `AX_APPEND_COMPILE_FLAGS`.
Summary (updated)
-----------------
Set up `-Wno-inconsistent-missing-override` conditionally.
Bugs: MESOS-9193
https://issues.apache.org/jira/browse/MESOS-9193
Repository: mesos
Description (updated)
-------
Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
only add it if the tool chain supports this option.
Diffs (updated)
-----
configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
Diff: https://reviews.apache.org/r/68564/diff/2/
Changes: https://reviews.apache.org/r/68564/diff/1-2/
Testing (updated)
-------
make with clang 3.5.0/3.6.0.
Thanks,
Chun-Hung Hsiao
Re: Review Request 68564: Disabled the `-Wunknown-warning-option`
warnings for Clang 3.5.
Posted by James Peach <jp...@apache.org>.
> On Aug. 30, 2018, 6:12 a.m., Benjamin Bannier wrote:
> > configure.ac
> > Lines 656 (patched)
> > <https://reviews.apache.org/r/68564/diff/1/?file=2078977#file2078977line656>
> >
> > Instead of ignoring unknown warnings altogether we should check whether the current compiler supports `-Wno-inconsistent-missing-override`. See the test for `-Wno-unused-local-typedef` above.
Consider using [ax_append_compile_flags](https://www.gnu.org/software/autoconf-archive/ax_append_compile_flags.html#ax_append_compile_flags)?
- James
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/#review208120
-----------------------------------------------------------
On Aug. 30, 2018, 4:31 a.m., Chun-Hung Hsiao wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68564/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2018, 4:31 a.m.)
>
>
> Review request for mesos, Benjamin Bannier and James Peach.
>
>
> Bugs: MESOS-9193
> https://issues.apache.org/jira/browse/MESOS-9193
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
> have to suppress `-Wunknown-warning-option` warnings if `-Werror` is on.
>
>
> Diffs
> -----
>
> configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
>
>
> Diff: https://reviews.apache.org/r/68564/diff/1/
>
>
> Testing
> -------
>
> make with clang 3.5.
>
>
> Thanks,
>
> Chun-Hung Hsiao
>
>
Re: Review Request 68564: Disabled the `-Wunknown-warning-option`
warnings for Clang 3.5.
Posted by Benjamin Bannier <be...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68564/#review208120
-----------------------------------------------------------
configure.ac
Lines 656 (patched)
<https://reviews.apache.org/r/68564/#comment291896>
Instead of ignoring unknown warnings altogether we should check whether the current compiler supports `-Wno-inconsistent-missing-override`. See the test for `-Wno-unused-local-typedef` above.
- Benjamin Bannier
On Aug. 30, 2018, 6:31 a.m., Chun-Hung Hsiao wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68564/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2018, 6:31 a.m.)
>
>
> Review request for mesos, Benjamin Bannier and James Peach.
>
>
> Bugs: MESOS-9193
> https://issues.apache.org/jira/browse/MESOS-9193
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Clang 3.5 does not support `-Wno-inconsisent-missing-override` so we
> have to suppress `-Wunknown-warning-option` warnings if `-Werror` is on.
>
>
> Diffs
> -----
>
> configure.ac e4fbd8cab04707c1fe0c6d5ade7b7a7dca9a7cb7
>
>
> Diff: https://reviews.apache.org/r/68564/diff/1/
>
>
> Testing
> -------
>
> make with clang 3.5.
>
>
> Thanks,
>
> Chun-Hung Hsiao
>
>