config.mak.dev

   1 ifndef COMPILER_FEATURES
   2 COMPILER_FEATURES := $(shell ./detect-compiler $(CC))
   3 endif
   4
   5 ifeq ($(filter no-error,$(DEVOPTS)),)
   6 DEVELOPER_CFLAGS += -Werror
   7 SPARSE_FLAGS += -Wsparse-error
   8 endif
   9
  10 DEVELOPER_CFLAGS += -Wall
  11 ifeq ($(filter no-pedantic,$(DEVOPTS)),)
  12 DEVELOPER_CFLAGS += -pedantic
  13 ifneq ($(or $(filter gcc5,$(COMPILER_FEATURES)),$(filter clang4,$(COMPILER_FEATURES))),)
  14 DEVELOPER_CFLAGS += -Wpedantic
  15 ifneq ($(filter gcc10,$(COMPILER_FEATURES)),)
  16 ifeq ($(uname_S),MINGW)
  17 DEVELOPER_CFLAGS += -Wno-pedantic-ms-format
  18 endif
  19 endif
  20 endif
  21 endif
  22
  23 ifneq ($(uname_S),FreeBSD)
  24 ifneq ($(or $(filter gcc6,$(COMPILER_FEATURES)),$(filter clang7,$(COMPILER_FEATURES))),)
  25 DEVELOPER_CFLAGS += -std=gnu99
  26 endif
  27 else
  28 # FreeBSD cannot limit to C99 because its system headers unconditionally
  29 # rely on C11 features.
  30 endif
  31
  32 DEVELOPER_CFLAGS += -Wdeclaration-after-statement
  33 DEVELOPER_CFLAGS += -Wformat-security
  34 DEVELOPER_CFLAGS += -Wold-style-definition
  35 DEVELOPER_CFLAGS += -Woverflow
  36 DEVELOPER_CFLAGS += -Wpointer-arith
  37 DEVELOPER_CFLAGS += -Wstrict-prototypes
  38 DEVELOPER_CFLAGS += -Wunused
  39 DEVELOPER_CFLAGS += -Wvla
  40 DEVELOPER_CFLAGS += -Wwrite-strings
  41 DEVELOPER_CFLAGS += -fno-common
  42
  43 ifneq ($(filter clang4,$(COMPILER_FEATURES)),)
  44 DEVELOPER_CFLAGS += -Wtautological-constant-out-of-range-compare
  45 endif
  46
  47 ifneq ($(or $(filter gcc6,$(COMPILER_FEATURES)),$(filter clang4,$(COMPILER_FEATURES))),)
  48 DEVELOPER_CFLAGS += -Wextra
  49 # if a function is public, there should be a prototype and the right
  50 # header file should be included. If not, it should be static.
  51 DEVELOPER_CFLAGS += -Wmissing-prototypes
  52 ifeq ($(filter extra-all,$(DEVOPTS)),)
  53 # These are disabled because we have these all over the place.
  54 DEVELOPER_CFLAGS += -Wno-empty-body
  55 DEVELOPER_CFLAGS += -Wno-missing-field-initializers
  56 endif
  57 endif
  58
  59 # uninitialized warnings on gcc 4.9.2 in xdiff/xdiffi.c and config.c
  60 # not worth fixing since newer compilers correctly stop complaining
  61 #
  62 # Likewise, gcc older than 4.9 complains about initializing a
  63 # struct-within-a-struct using just "{ 0 }"
  64 ifneq ($(filter gcc4,$(COMPILER_FEATURES)),)
  65 ifeq ($(filter gcc5,$(COMPILER_FEATURES)),)
  66 DEVELOPER_CFLAGS += -Wno-uninitialized
  67 DEVELOPER_CFLAGS += -Wno-missing-braces
  68 endif
  69 endif
  70
  71 # Old versions of clang complain about initializing a
  72 # struct-within-a-struct using just "{0}" rather than "{{0}}".  This
  73 # error is considered a false-positive and not worth fixing, because
  74 # new clang versions do not, so just disable it.
  75 #
  76 # The "bug" was fixed in upstream clang 9.
  77 #
  78 # Complicating this is that versions of clang released by Apple have
  79 # their own version numbers (associated with the corresponding version
  80 # of XCode) unrelated to the official clang version numbers.
  81 #
  82 # The bug was fixed in Apple clang 12.
  83 #
  84 ifneq ($(filter clang1,$(COMPILER_FEATURES)),)     # if we are using clang
  85 ifeq ($(uname_S),Darwin)                           # if we are on darwin
  86 ifeq ($(filter clang12,$(COMPILER_FEATURES)),)     # if version < 12
  87 DEVELOPER_CFLAGS += -Wno-missing-braces
  88 endif
  89 else                                               # not darwin
  90 ifeq ($(filter clang9,$(COMPILER_FEATURES)),)      # if version < 9
  91 DEVELOPER_CFLAGS += -Wno-missing-braces
  92 endif
  93 endif
  94 endif
  95
  96 # https://bugzilla.redhat.com/show_bug.cgi?id=2075786
  97 ifneq ($(filter gcc12,$(COMPILER_FEATURES)),)
  98 DEVELOPER_CFLAGS += -Wno-error=stringop-overread
  99 endif
 100
 101 GIT_TEST_PERL_FATAL_WARNINGS = YesPlease