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

Commit ab149cbd authored by Chih-Hung Hsieh's avatar Chih-Hung Hsieh
Browse files

No need to dump modules added -Werror. 

* This reduces size of output file wall_werror.txt.
* Stop collecting $(MODULES_ADDED_WERROR)

Bug: 66996870
Test: normal build
Change-Id: Id750cc7fe6489a7654b20740d503226ec6941bbb
parent b8e08222

core/Makefile

+0 −2
Original line number Diff line number Diff line
@@ -536,8 +536,6 @@ $(WALL_WERROR): 
	@rm -f $@

	echo "# Modules using -Wno-error" >> $@

	for m in $(sort $(SOONG_MODULES_USING_WNO_ERROR) $(MODULES_USING_WNO_ERROR)); do echo $$m >> $@; done

	echo "# Modules added default -Wall -Werror" >> $@

	for m in $(sort $(SOONG_MODULES_ADDED_WERROR) $(MODULES_ADDED_WERROR)); do echo $$m >> $@; done

	echo "# Modules added default -Wall" >> $@

	for m in $(sort $(SOONG_MODULES_ADDED_WALL) $(MODULES_ADDED_WALL)); do echo $$m >> $@; done

core/binary.mk

+0 −1
Original line number Diff line number Diff line
@@ -1696,7 +1696,6 @@ ifneq ($(LOCAL_MODULE_MAKEFILE),$(SOONG_ANDROID_MK)) 
        ifeq (,$(filter -Werror,$(my_all_cflags)))

          # Add -Wall -Werror unless the project is in the WARNING_ALLOWED project list.

          ifeq (,$(strip $(call find_warning_allowed_projects,$(LOCAL_PATH))))

            $(eval MODULES_ADDED_WERROR := $(MODULES_ADDED_WERROR) $(LOCAL_MODULE_MAKEFILE):$(LOCAL_MODULE))

            my_cflags := -Wall -Werror $(my_cflags)

          else

            $(eval MODULES_ADDED_WALL := $(MODULES_ADDED_WALL) $(LOCAL_MODULE_MAKEFILE):$(LOCAL_MODULE))