800401 - Stop forcing -Werror on everybody

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Description

[(no longer active)]

Attachment: Patch (v1)

This is regularly breaking people's builds.  WebRTC should switch to using FAIL_ON_WARNINGS at some point, but for now this unbreaks people's builds.

Comment 1

[Ethan Hugg [:ehugg]]

Comment on attachment 670418 [details] [diff] [review]
Patch (v1)

Review of attachment 670418 [details] [diff] [review]:
-----------------------------------------------------------------

::: media/webrtc/signaling/signaling.gyp
@@ -216,1 @@
>              '-Wno-error=conversion'

You should probably take out these '-Wno-error=conversion' lines as well.

@@ -756,1 @@
>              '-Wno-error=conversion'

And here.

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 670418 [details] [diff] [review]
Patch (v1)

Review of attachment 670418 [details] [diff] [review]:
-----------------------------------------------------------------

With the changes to parts we don't use removed (to reduce update-from-upstream fun), r+

::: media/webrtc/trunk/build/common.gypi
@@ -1670,5 @@
>            }],
>            # TODO: Fix all warnings on chromeos too.
>            [ 'os_posix==1 and OS!="mac" and OS!="ios" and (clang!=1 or chromeos==1)', {
>              'cflags!': [
> -              '-Werror',

We purposely ignore cflags in mozmake.py, and only use cflags_mozilla

@@ +2793,5 @@
>            'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES',
>            'GCC_OBJC_CALL_CXX_CDTORS': 'YES',        # -fobjc-call-cxx-cdtors
>            'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES',      # -fvisibility=hidden
>            'GCC_THREADSAFE_STATICS': 'NO',           # -fno-threadsafe-statics
> +          'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO',    # -Werror

I don't believe we use the xcode_settings var when we build; that's used for xcode gyp output.

::: media/webrtc/trunk/build/nocompile.gypi
@@ +80,5 @@
>              'python',
>              '<(nocompile_driver)',
>              '4', # number of compilers to invoke in parallel.
>              '<(RULE_INPUT_PATH)',
> +            '-Wall -Wfatal-errors -I<(DEPTH)',

We don't use nocompile.gypi

::: media/webrtc/trunk/peerconnection_client.target.mk
@@ +45,5 @@
>  	'-DWTF_USE_DYNAMIC_ANNOTATIONS=1' \
>  	'-D_DEBUG'
>  
>  # Flags passed to all source files.
>  CFLAGS_Debug := \

We don't use peerconnection_client.target.mk

Comment 3

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bc8cbe29588d

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/bc8cbe29588d