diff options
| author | 2024-03-15 16:03:22 +0800 | |
|---|---|---|
| committer | 2024-03-21 10:47:32 +0800 | |
| commit | 3fe7b7f66efb4de8fee61b19d53c91118dde2d33 (patch) | |
| tree | c3123d961e30d73d75c339bcd141d41c6839998c /androidmk | |
| parent | 8b0192d031276bcfbd0e409acd43f8e8d2a9474b (diff) | |
Disable "__builtin_func" when converting mk to bp
The "__builtin_func" doesn't work in soong. The converting tool
androidmk should disable this feature.
Bug: 329770862
Test: 1. cd build/soong/androidmk/androidmk
2. go test -run TestEndToEnd
Change-Id: I1de04e4e1a34fb7335e0013f59b33f7f2673722a
Diffstat (limited to 'androidmk')
| -rw-r--r-- | androidmk/androidmk/androidmk_test.go | 16 | ||||
| -rw-r--r-- | androidmk/parser/scope.go | 6 |
2 files changed, 18 insertions, 4 deletions
diff --git a/androidmk/androidmk/androidmk_test.go b/androidmk/androidmk/androidmk_test.go index 460f1ffac..1dd479ca5 100644 --- a/androidmk/androidmk/androidmk_test.go +++ b/androidmk/androidmk/androidmk_test.go @@ -511,6 +511,22 @@ endif # b==false `, }, { + // Unsupported function case because that doesn't work in bp + desc: "error for unsupported functions", + in: ` +include $(CLEAR_VARS) +LOCAL_SRC_FILES := $(filter-out filter-out-file.java ,$(call all-java-files-under, src)) +LOCAL_PACKAGE_NAME := foo +include $(BUILD_PACKAGE) +`, + expected: ` +android_app { + name: "foo", + srcs: ["UNSUPPORTED FUNCTION:filter-out filter-out-file.java src/**/*.java"], +} + `, + }, + { desc: "ignore all-makefiles-under", in: ` include $(call all-makefiles-under,$(LOCAL_PATH)) diff --git a/androidmk/parser/scope.go b/androidmk/parser/scope.go index 8111c89e1..e1a523a6b 100644 --- a/androidmk/parser/scope.go +++ b/androidmk/parser/scope.go @@ -14,9 +14,7 @@ package parser -import ( - "strings" -) +import "strings" type Scope interface { Get(name string) string @@ -88,7 +86,7 @@ func (v Variable) EvalFunction(scope Scope) ([]string, bool) { if fname == "call" { return scope.Call(argVals[0], argVals[1:]), true } else { - return []string{"__builtin_func:" + fname + " " + strings.Join(argVals, " ")}, true + return []string{"UNSUPPORTED FUNCTION:" + fname + " " + strings.Join(argVals, " ")}, true } } } |