From 65f28793ab74a13d088ab6c11661fab878e1609f Mon Sep 17 00:00:00 2001 From: Alix Espino Date: Mon, 12 Sep 2022 18:51:23 +0000 Subject: [PATCH] Revert "Default BuildBrokenClangCFlags & BuildBrokenClangAsFlag..." Revert "deletion of clang_cflags & clang_asflags from Soong" Revert submission 2154033-delete clang_cflags clang_asflags Reason for revert: Broke build on git_tm-dev-plus-aosp and downstream branches Reverted Changes: I5c499a37e:Default BuildBrokenClangCFlags & BuildBrokenClang... Ic1d90b72d:deletion of clang_cflags & clang_asflags from Soon... Change-Id: I62480972e5cd6cfe01782068b25893222a8edf79 --- Changes.md | 21 --------------------- core/board_config.mk | 2 -- core/soong_config.mk | 2 -- 3 files changed, 25 deletions(-) diff --git a/Changes.md b/Changes.md index 3ad26417b2..27e52f2630 100644 --- a/Changes.md +++ b/Changes.md @@ -827,27 +827,6 @@ go run bpmodify.go -w -m=module_name -remove-property=true -property=clang filep ``` `BUILD_BROKEN_CLANG_PROPERTY` can be used as temporarily workaround - - -### Stop using clang_cflags and clang_asflags - -clang_cflags and clang_asflags are deprecated. -To fix any build errors, use bpmodify to either - - move the contents of clang_asflags/clang_cflags into asflags/cflags or - - delete clang_cflags/as_flags as necessary - -To Move the contents: -``` make -go run bpmodify.go -w -m=module_name -move-property=true -property=clang_cflags -new-location=cflags filepath -``` - -To Delete: -``` make -go run bpmodify.go -w -m=module_name -remove-property=true -property=clang_cflags filepath -``` - -`BUILD_BROKEN_CLANG_ASFLAGS` and `BUILD_BROKEN_CLANG_CFLAGS` can be used as temporarily workarounds - ### Other envsetup.sh variables {#other_envsetup_variables} * ANDROID_TOOLCHAIN diff --git a/core/board_config.mk b/core/board_config.mk index 2218162d1b..192e96b5a5 100644 --- a/core/board_config.mk +++ b/core/board_config.mk @@ -175,8 +175,6 @@ _board_strip_list += ODM_MANIFEST_SKUS _build_broken_var_list := \ BUILD_BROKEN_CLANG_PROPERTY \ - BUILD_BROKEN_CLANG_ASFLAGS \ - BUILD_BROKEN_CLANG_CFLAGS \ BUILD_BROKEN_DEPFILE \ BUILD_BROKEN_DUP_RULES \ BUILD_BROKEN_DUP_SYSPROP \ diff --git a/core/soong_config.mk b/core/soong_config.mk index 64d1308bb2..2ff064a408 100644 --- a/core/soong_config.mk +++ b/core/soong_config.mk @@ -276,8 +276,6 @@ $(call add_json_str, PrebuiltHiddenApiDir, $(BOARD_PREBUILT_HIDDENAPI_DIR)) $(call add_json_str, ShippingApiLevel, $(PRODUCT_SHIPPING_API_LEVEL)) $(call add_json_bool, BuildBrokenClangProperty, $(filter true,$(BUILD_BROKEN_CLANG_PROPERTY))) -$(call add_json_bool, BuildBrokenClangAsFlags, $(filter true,$(BUILD_BROKEN_CLANG_ASFLAGS))) -$(call add_json_bool, BuildBrokenClangCFlags, $(filter true,$(BUILD_BROKEN_CLANG_CFLAGS))) $(call add_json_bool, BuildBrokenDepfile, $(filter true,$(BUILD_BROKEN_DEPFILE))) $(call add_json_bool, BuildBrokenEnforceSyspropOwner, $(filter true,$(BUILD_BROKEN_ENFORCE_SYSPROP_OWNER))) $(call add_json_bool, BuildBrokenTrebleSyspropNeverallow, $(filter true,$(BUILD_BROKEN_TREBLE_SYSPROP_NEVERALLOW)))