diff options
author | 2019-11-08 14:17:49 -0800 | |
---|---|---|
committer | 2019-11-11 15:44:09 -0800 | |
commit | ce23942f3c485232d49a8ed06ba0c443a2568a9d (patch) | |
tree | b74f9b70e1634d29a14aa164b67e4e1a02611c03 /partner | |
parent | 2d5ce8538b74b983b3b58d67a1c5272258d6bf26 (diff) |
Move partner androidmk and bpfix files to match their package path
Using a gomod-aware editor with build/soong requires that files
in build/soong can be mapped to the android/soong package path.
Move the partner androidmk and bpfix files such that their path
matches the package path when the android/soong package prefix is
replaced with the build/soong path prefix.
Test: go test ./...
Test: m bpfix androidmk partner_bpfix partner_androidmk
Change-Id: Ic7f7aad9e5eb9178eef0383f0b37e4fb93ce8314
Diffstat (limited to 'partner')
-rw-r--r-- | partner/Android.bp | 49 | ||||
-rw-r--r-- | partner/androidmk/androidmk.go | 58 | ||||
-rw-r--r-- | partner/androidmk/androidmk_test.go | 73 | ||||
-rw-r--r-- | partner/bpfix/bpfix.go | 27 | ||||
-rw-r--r-- | partner/bpfix/extensions/headers.go | 120 |
5 files changed, 327 insertions, 0 deletions
diff --git a/partner/Android.bp b/partner/Android.bp new file mode 100644 index 000000000..f2ced8dd3 --- /dev/null +++ b/partner/Android.bp @@ -0,0 +1,49 @@ +// Copyright 2019 Google Inc. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +// +// Sample project for creating an extended androidmk +// + +blueprint_go_binary { + name: "partner_androidmk", + srcs: [ + "androidmk/androidmk.go", + ], + testSrcs: [ + "androidmk/androidmk_test.go", + ], + deps: [ + "androidmk-lib", + "partner-bpfix-extensions", + ], +} + +blueprint_go_binary { + name: "partner_bpfix", + srcs: [ + "bpfix/bpfix.go", + ], + deps: [ + "bpfix-cmd", + "partner-bpfix-extensions", + ], +} + +bootstrap_go_package { + name: "partner-bpfix-extensions", + pkgPath: "android/soong/partner/bpfix/extensions", + srcs: ["bpfix/extensions/headers.go"], + deps: ["bpfix-lib"], +} diff --git a/partner/androidmk/androidmk.go b/partner/androidmk/androidmk.go new file mode 100644 index 000000000..f49981b09 --- /dev/null +++ b/partner/androidmk/androidmk.go @@ -0,0 +1,58 @@ +// Copyright 2017 Google Inc. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package main + +import ( + "bytes" + "flag" + "fmt" + "io/ioutil" + "os" + + "android/soong/androidmk/androidmk" + + _ "android/soong/partner/bpfix/extensions" +) + +var usage = func() { + fmt.Fprintf(os.Stderr, "usage: %s [flags] <inputFile>\n"+ + "\n%s parses <inputFile> as an Android.mk file and attempts to output an analogous Android.bp file (to standard out)\n", os.Args[0], os.Args[0]) + flag.PrintDefaults() + os.Exit(1) +} + +func main() { + flag.Usage = usage + flag.Parse() + if len(flag.Args()) != 1 { + usage() + } + filePathToRead := flag.Arg(0) + b, err := ioutil.ReadFile(filePathToRead) + if err != nil { + fmt.Println(err.Error()) + return + } + + output, errs := androidmk.ConvertFile(os.Args[1], bytes.NewBuffer(b)) + if len(errs) > 0 { + for _, err := range errs { + fmt.Fprintln(os.Stderr, "ERROR: ", err) + } + os.Exit(1) + } + + fmt.Print(output) +} diff --git a/partner/androidmk/androidmk_test.go b/partner/androidmk/androidmk_test.go new file mode 100644 index 000000000..6bae836d9 --- /dev/null +++ b/partner/androidmk/androidmk_test.go @@ -0,0 +1,73 @@ +// Copyright 2016 Google Inc. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package main + +import ( + "bytes" + "fmt" + "strings" + "testing" + + "android/soong/androidmk/androidmk" + "android/soong/bpfix/bpfix" + + _ "android/soong/partner/bpfix/extensions" +) + +var testCases = []struct { + desc string + in string + expected string +}{ + { + desc: "headers replacement", + in: ` +include $(CLEAR_VARS) +LOCAL_MODULE := test +LOCAL_SRC_FILES := a.c +LOCAL_C_INCLUDES := test1 $(TARGET_OUT_HEADERS)/my_headers test2 +include $(BUILD_SHARED_LIBRARY)`, + expected: ` +cc_library_shared { + name: "test", + srcs: ["a.c"], + include_dirs: [ + "test1", + + "test2", + ], + header_libs: ["my_header_lib"] +}`, + }, +} + +func TestEndToEnd(t *testing.T) { + for i, test := range testCases { + expected, err := bpfix.Reformat(test.expected) + if err != nil { + t.Error(err) + } + + got, errs := androidmk.ConvertFile(fmt.Sprintf("<testcase %d>", i), bytes.NewBufferString(test.in)) + if len(errs) > 0 { + t.Errorf("Unexpected errors: %q", errs) + continue + } + + if got != expected { + t.Errorf("failed testcase '%s'\ninput:\n%s\n\nexpected:\n%s\ngot:\n%s\n", test.desc, strings.TrimSpace(test.in), expected, got) + } + } +} diff --git a/partner/bpfix/bpfix.go b/partner/bpfix/bpfix.go new file mode 100644 index 000000000..687fe1c04 --- /dev/null +++ b/partner/bpfix/bpfix.go @@ -0,0 +1,27 @@ +// Copyright 2017 Google Inc. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +// This file provides a command-line interface to bpfix + +package main + +import ( + "android/soong/bpfix/cmd_lib" + + _ "android/soong/partner/bpfix/extensions" +) + +func main() { + cmd_lib.Run() +} diff --git a/partner/bpfix/extensions/headers.go b/partner/bpfix/extensions/headers.go new file mode 100644 index 000000000..169dab667 --- /dev/null +++ b/partner/bpfix/extensions/headers.go @@ -0,0 +1,120 @@ +package extensions + +import ( + "strings" + + "github.com/google/blueprint/parser" + + "android/soong/bpfix/bpfix" +) + +var fixSteps = bpfix.FixStepsExtension{ + Name: "partner-include-dirs", + Steps: []bpfix.FixStep{ + { + Name: "fixIncludeDirs", + Fix: fixIncludeDirs, + }, + }, +} + +func init() { + bpfix.RegisterFixStepExtension(&fixSteps) +} + +type includeDirFix struct { + libName string + libType string + variable string + subdir string +} + +var commonIncludeDirs = []includeDirFix{ + { + libName: "my_header_lib", + libType: "header_libs", + variable: "TARGET_OUT_HEADERS", + subdir: "/my_headers", + }, +} + +func findHeaderLib(e parser.Expression) (*includeDirFix, bool) { + if op, ok := e.(*parser.Operator); ok { + if op.Operator != '+' { + return nil, false + } + arg0, ok := op.Args[0].(*parser.Variable) + arg1, ok1 := op.Args[1].(*parser.String) + if !ok || !ok1 { + return nil, false + } + for _, lib := range commonIncludeDirs { + if arg0.Name == lib.variable && arg1.Value == lib.subdir { + return &lib, true + } + } + } + return nil, false +} +func searchThroughOperatorList(mod *parser.Module, e parser.Expression) { + if list, ok := e.(*parser.List); ok { + newList := make([]parser.Expression, 0, len(list.Values)) + for _, item := range list.Values { + if lib, found := findHeaderLib(item); found { + if lib.libName != "" { + addLibrary(mod, lib.libType, lib.libName) + } + } else { + newList = append(newList, item) + } + } + list.Values = newList + } + if op, ok := e.(*parser.Operator); ok { + searchThroughOperatorList(mod, op.Args[0]) + searchThroughOperatorList(mod, op.Args[1]) + } +} +func getLiteralListProperty(mod *parser.Module, name string) (list *parser.List, found bool) { + prop, ok := mod.GetProperty(name) + if !ok { + return nil, false + } + list, ok = prop.Value.(*parser.List) + return list, ok +} +func addLibrary(mod *parser.Module, libType string, libName string) { + var list, ok = getLiteralListProperty(mod, libType) + if !ok { + list = new(parser.List) + prop := new(parser.Property) + prop.Name = libType + prop.Value = list + mod.Properties = append(mod.Properties, prop) + } else { + for _, v := range list.Values { + if stringValue, ok := v.(*parser.String); ok && stringValue.Value == libName { + return + } + } + } + lib := new(parser.String) + lib.Value = libName + list.Values = append(list.Values, lib) +} +func fixIncludeDirs(f *bpfix.Fixer) error { + tree := f.Tree() + for _, def := range tree.Defs { + mod, ok := def.(*parser.Module) + if !ok { + continue + } + if !strings.HasPrefix(mod.Type, "cc_") { + continue + } + if prop, ok := mod.GetProperty("include_dirs"); ok { + searchThroughOperatorList(mod, prop.Value) + } + } + return nil +} |