summaryrefslogtreecommitdiff
path: root/coreconf/Werror.mk
blob: f1f8dc9ec58e31dc4a1e218a738d6c8f14721c44 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
#
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

# This sets WARNING_CFLAGS for unix-like operating systems.

ifndef CC_NAME
  CC_NAME := $(shell $(CC) -? 2>&1 >/dev/null | sed -e 's/:.*//;1q')
  export CC_NAME
endif

ifndef WARNING_CFLAGS
  # This tests to see if enabling the warning is possible before
  # setting an option to disable it.
  disable_warning = $(shell $(CC) -x c -E -Werror -W$(1) /dev/null >/dev/null 2>&1 && echo -Wno-$(1))

  WARNING_CFLAGS = -Wall
  ifeq ($(CC_NAME),clang)
    # -Qunused-arguments : clang objects to arguments that it doesn't understand
    #    and fixing this would require rearchitecture
    WARNING_CFLAGS += -Qunused-arguments
    # -Wno-parentheses-equality : because clang warns about macro expansions
    WARNING_CFLAGS += $(call disable_warning,parentheses-equality)
    ifdef BUILD_OPT
      # clang is unable to handle glib's expansion of strcmp and similar for optimized
      # builds, so ignore the resulting errors.
      # See https://llvm.org/bugs/show_bug.cgi?id=20144
      WARNING_CFLAGS += $(call disable_warning,array-bounds)
      WARNING_CFLAGS += $(call disable_warning,unevaluated-expression)
    endif
  endif # if clang

  ifndef NSS_ENABLE_WERROR
    ifeq ($(OS_TARGET),Android)
      # Android lollipop generates the following warning:
      # error: call to 'sprintf' declared with attribute warning:
      #   sprintf is often misused; please use snprintf [-Werror]
      # So, just suppress -Werror entirely on Android
      NSS_ENABLE_WERROR = 0
      $(warning OS_TARGET is Android, disabling -Werror)
    else
      ifeq ($(CC_NAME),clang)
        # Clang reports its version as an older gcc, but it's OK
        NSS_ENABLE_WERROR = 1
      else
        CC_VERSION := $(subst ., ,$(shell $(CC) -dumpversion))
        ifneq (,$(filter 4.8 4.9,$(word 1,$(CC_VERSION)).$(word 2,$(CC_VERSION))))
          NSS_ENABLE_WERROR = 1
        endif
        ifeq (,$(filter 0 1 2 3 4,$(word 1,$(CC_VERSION))))
          NSS_ENABLE_WERROR = 1
        endif
        ifndef NSS_ENABLE_WERROR
          $(warning Unable to find gcc 4.8 or greater, disabling -Werror)
          NSS_ENABLE_WERROR = 0
        endif
      endif
    endif
  endif #ndef NSS_ENABLE_WERROR

  ifeq ($(NSS_ENABLE_WERROR),1)
    WARNING_CFLAGS += -Werror
  else
    # Old versions of gcc (< 4.8) don't support #pragma diagnostic in functions.
    # Use this to disable use of that #pragma and the warnings it suppresses.
    WARNING_CFLAGS += -DNSS_NO_GCC48
  endif
  export WARNING_CFLAGS
endif # ndef WARNING_CFLAGS