Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c33f240d authored by Colin Cross's avatar Colin Cross Committed by Gerrit Code Review
Browse files

Merge "Forbid -Weverything"

parents 4f51fba5 0d1f54e2

core/binary.mk

+8 −1
Original line number Diff line number Diff line
@@ -1441,10 +1441,10 @@ endif 


# Check if -Werror or -Wno-error is used in C compiler flags.

# Header libraries do not need cflags.

my_all_cflags := $(my_cflags) $(my_cppflags) $(my_cflags_no_override)

ifneq (HEADER_LIBRARIES,$(LOCAL_MODULE_CLASS))

  # Prebuilt modules do not need cflags.

  ifeq (,$(LOCAL_PREBUILT_MODULE_FILE))

    my_all_cflags := $(my_cflags) $(my_cppflags) $(my_cflags_no_override)

    # Issue warning if -Wno-error is used.

    ifneq (,$(filter -Wno-error,$(my_all_cflags)))

      $(eval MODULES_USING_WNO_ERROR := $(MODULES_USING_WNO_ERROR) $(LOCAL_MODULE_MAKEFILE):$(LOCAL_MODULE))
@@ -1463,6 +1463,13 @@ ifneq (HEADER_LIBRARIES,$(LOCAL_MODULE_CLASS)) 
  endif

endif



ifneq (,$(filter -Weverything,$(my_all_cflags)))

  ifeq (,$(ANDROID_TEMPORARILY_ALLOW_WEVERYTHING))

    $(call pretty-error, -Weverything is not allowed in Android.mk files.\

      Build with `m ANDROID_TEMPORARILY_ALLOW_WEVERYTHING=true` to experiment locally with -Weverything.)

  endif

endif



# Disable clang-tidy if it is not found.

ifeq ($(PATH_TO_CLANG_TIDY),)

  my_tidy_enabled := false