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 DEVELOPER_CFLAGS += -Wno-sign-compare
  57 endif
  58 endif
  59
  60 # uninitialized warnings on gcc 4.9.2 in xdiff/xdiffi.c and config.c
  61 # not worth fixing since newer compilers correctly stop complaining
  62 #
  63 # Likewise, gcc older than 4.9 complains about initializing a
  64 # struct-within-a-struct using just "{ 0 }"
  65 ifneq ($(filter gcc4,$(COMPILER_FEATURES)),)
  66 ifeq ($(filter gcc5,$(COMPILER_FEATURES)),)
  67 DEVELOPER_CFLAGS += -Wno-uninitialized
  68 DEVELOPER_CFLAGS += -Wno-missing-braces
  69 endif
  70 endif
  71
  72 # Old versions of clang complain about initializaing a
  73 # struct-within-a-struct using just "{0}" rather than "{{0}}".  This
  74 # error is considered a false-positive and not worth fixing, because
  75 # new clang versions do not, so just disable it.
  76 #
  77 # The "bug" was fixed in upstream clang 9.
  78 #
  79 # Complicating this is that versions of clang released by Apple have
  80 # their own version numbers (associated with the corresponding version
  81 # of XCode) unrelated to the official clang version numbers.
  82 #
  83 # The bug was fixed in Apple clang 12.
  84 #
  85 ifneq ($(filter clang1,$(COMPILER_FEATURES)),)     # if we are using clang
  86 ifeq ($(uname_S),Darwin)                           # if we are on darwin
  87 ifeq ($(filter clang12,$(COMPILER_FEATURES)),)     # if version < 12
  88 DEVELOPER_CFLAGS += -Wno-missing-braces
  89 endif
  90 else                                               # not darwin
  91 ifeq ($(filter clang9,$(COMPILER_FEATURES)),)      # if version < 9
  92 DEVELOPER_CFLAGS += -Wno-missing-braces
  93 endif
  94 endif
  95 endif
  96
  97 # https://bugzilla.redhat.com/show_bug.cgi?id=2075786
  98 ifneq ($(filter gcc12,$(COMPILER_FEATURES)),)
  99 DEVELOPER_CFLAGS += -Wno-error=stringop-overread
 100 endif
 101
 102 GIT_TEST_PERL_FATAL_WARNINGS = YesPlease