Merge "Introduce BUILD_BROKEN_INPUT_DIR_MODULES" am: 16e2fc2da1
Original change: https://android-review.googlesource.com/c/platform/build/+/1966254 Change-Id: I789ab17d3f3b73e66502c90d9eb5cc32313514bd
This commit is contained in:
commit
0ac682fc9a
3 changed files with 32 additions and 0 deletions
30
Changes.md
30
Changes.md
|
@ -1,5 +1,35 @@
|
|||
# Build System Changes for Android.mk Writers
|
||||
|
||||
## Genrule starts disallowing directory inputs
|
||||
|
||||
To better specify the inputs to the build, we are restricting use of directories
|
||||
as inputs to genrules.
|
||||
|
||||
To fix existing uses, change inputs to specify the inputs and update the command
|
||||
accordingly. For example:
|
||||
|
||||
```
|
||||
genrule: {
|
||||
name: "foo",
|
||||
srcs: ["bar"],
|
||||
cmd: "cp $(location bar)/*.xml $(gendir)",
|
||||
...
|
||||
}
|
||||
```
|
||||
|
||||
would become
|
||||
|
||||
```
|
||||
genrule: {
|
||||
name: "foo",
|
||||
srcs: ["bar/*.xml"],
|
||||
cmd: "cp $(in) $(gendir)",
|
||||
...
|
||||
}
|
||||
|
||||
`BUILD_BROKEN_INPUT_DIR_MODULES` can be used to allowlist specific directories
|
||||
with genrules that have input directories.
|
||||
|
||||
## Dexpreopt starts enforcing `<uses-library>` checks (for Java modules)
|
||||
|
||||
In order to construct correct class loader context for dexpreopt, build system
|
||||
|
|
|
@ -174,6 +174,7 @@ _build_broken_var_list := \
|
|||
BUILD_BROKEN_DUP_SYSPROP \
|
||||
BUILD_BROKEN_ELF_PREBUILT_PRODUCT_COPY_FILES \
|
||||
BUILD_BROKEN_ENFORCE_SYSPROP_OWNER \
|
||||
BUILD_BROKEN_INPUT_DIR_MODULES \
|
||||
BUILD_BROKEN_MISSING_REQUIRED_MODULES \
|
||||
BUILD_BROKEN_OUTSIDE_INCLUDE_DIRS \
|
||||
BUILD_BROKEN_PREBUILT_ELF_FILES \
|
||||
|
|
|
@ -270,6 +270,7 @@ $(call add_json_str, ShippingApiLevel, $(PRODUCT_SHIPPING_API_LEVEL))
|
|||
$(call add_json_bool, BuildBrokenEnforceSyspropOwner, $(filter true,$(BUILD_BROKEN_ENFORCE_SYSPROP_OWNER)))
|
||||
$(call add_json_bool, BuildBrokenTrebleSyspropNeverallow, $(filter true,$(BUILD_BROKEN_TREBLE_SYSPROP_NEVERALLOW)))
|
||||
$(call add_json_bool, BuildBrokenVendorPropertyNamespace, $(filter true,$(BUILD_BROKEN_VENDOR_PROPERTY_NAMESPACE)))
|
||||
$(call add_json_list, BuildBrokenInputDirModules, $(BUILD_BROKEN_INPUT_DIR_MODULES))
|
||||
|
||||
$(call add_json_bool, BuildDebugfsRestrictionsEnabled, $(filter true,$(PRODUCT_SET_DEBUGFS_RESTRICTIONS)))
|
||||
|
||||
|
|
Loading…
Reference in a new issue