| commit | e712879d5dbdb06da48cf054f0a9c5d582a8387e | [log] [tgz] |
|---|---|---|
| author | Vishwath Mohan <vishwath@google.com> | Fri Nov 17 11:08:10 2017 -0800 |
| committer | Vishwath Mohan <vishwath@google.com> | Tue Nov 21 14:09:09 2017 -0800 |
| tree | 4b568b550a6361fb378d1df48cd4b6e6895e6735 | |
| parent | 6693613f1176b71b273be4328c5b245883bc9f8e [diff] |
Reduce how often both mutated variants are needed. This CL rolls back how often we bubble up both sanitized and un-sanitized variants of a component. With this change only CFI-enabled target static libraries will do this, all other cases suppress one of the two variants (both from being installed and from being exposed to Make for make-embedded builds). This means we shouldn't need a separate sanitizer suffix for ASAN at all (.asan), and similarly for non static-lib CFI components (.cfi), so this CL changes that as well. Lastly, because the version of ar meant for the host is not built with plugin support (which CFI requires), this CL disables CFI for host targets. This CL should fix the following 2 issues: (1) Removing warnings about multiple rules existing for the same installable target. (2) Fixing VTS packaging, which had been broken by the generation of the .asan suffix. Bug: 69172424, 69059192, 67507323 Test: m -j40 # Soong generated .mk file does not have duplicate rules. Test: SANITIZE_TARGET="address" m -j40 libstagefright # installed correctly. Change-Id: Ib90fdbc8a6ad3924fc2a691b7277a8a1bc67cda8
Soong is the replacement for the old Android make-based build system. It replaces Android.mk files with Android.bp files, which are JSON-like simple declarative descriptions of modules to build.
By design, Android.bp files are very simple. There are no conditionals or control flow statements - any complexity is handled in build logic written in Go. The syntax and semantics of Android.bp files are intentionally similar to Bazel BUILD files when possible.
A module in an Android.bp file starts with a module type, followed by a set of properties in name: value, format:
cc_binary {
name: "gzip",
srcs: ["src/test/minigzip.c"],
shared_libs: ["libz"],
stl: "none",
}
Every module must have a name property, and the value must be unique across all Android.bp files.
For a list of valid module types and their properties see $OUT_DIR/soong/.bootstrap/docs/soong_build.html.
An Android.bp file may contain top-level variable assignments:
gzip_srcs = ["src/test/minigzip.c"],
cc_binary {
name: "gzip",
srcs: gzip_srcs,
shared_libs: ["libz"],
stl: "none",
}
Variables are scoped to the remainder of the file they are declared in, as well as any child blueprint files. Variables are immutable with one exception - they can be appended to with a += assignment, but only before they have been referenced.
Android.bp files can contain C-style multiline /* */ and C++ style single-line // comments.
Variables and properties are strongly typed, variables dynamically based on the first assignment, and properties statically by the module type. The supported types are:
true or false)int)"string")["string1", "string2"]){key1: "value1", key2: ["value2"]})Maps may values of any type, including nested maps. Lists and maps may have trailing commas after the last value.
Strings, lists of strings, and maps can be appended using the + operator. Integers can be summed up using the + operator. Appending a map produces the union of keys in both maps, appending the values of any keys that are present in both maps.
A defaults module can be used to repeat the same properties in multiple modules. For example:
cc_defaults {
name: "gzip_defaults",
shared_libs: ["libz"],
stl: "none",
}
cc_binary {
name: "gzip",
defaults: ["gzip_defaults"],
srcs: ["src/test/minigzip.c"],
}
Soong includes a canonical formatter for blueprint files, similar to gofmt. To recursively reformat all Android.bp files in the current directory:
bpfmt -w .
The canonical format includes 4 space indents, newlines after every element of a multi-element list, and always includes a trailing comma in lists and maps.
Soong includes a tool perform a first pass at converting Android.mk files to Android.bp files:
androidmk Android.mk > Android.bp
The tool converts variables, modules, comments, and some conditionals, but any custom Makefile rules, complex conditionals or extra includes must be converted by hand.
host_supported: true. The androidmk converter will produce multiple conflicting modules, which must be resolved by hand to a single module with any differences inside target: { android: { }, host: { } } blocks.The build logic is written in Go using the blueprint framework. Build logic receives module definitions parsed into Go structures using reflection and produces build rules. The build rules are collected by blueprint and written to a ninja build file.
Soong deliberately does not support conditionals in Android.bp files. Instead, complexity in build rules that would require conditionals are handled in Go, where high level language features can be used and implicit dependencies introduced by conditionals can be tracked. Most conditionals are converted to a map property, where one of the values in the map will be selected and appended to the top level properties.
For example, to support architecture specific files:
cc_library {
...
srcs: ["generic.cpp"],
arch: {
arm: {
srcs: ["arm.cpp"],
},
x86: {
srcs: ["x86.cpp"],
},
},
}
See art/build/art.go or external/llvm/soong/llvm.go for examples of more complex conditionals on product variables or environment variables.
Email android-building@googlegroups.com (external) for any questions, or see go/soong (internal).