2020-11-19 18:38:02 +08:00
|
|
|
// Copyright 2020 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 bazel
|
|
|
|
|
2021-03-12 19:04:21 +08:00
|
|
|
import (
|
|
|
|
"fmt"
|
2021-03-24 22:04:33 +08:00
|
|
|
"path/filepath"
|
2021-03-24 22:14:47 +08:00
|
|
|
"regexp"
|
2021-03-12 19:04:21 +08:00
|
|
|
"sort"
|
|
|
|
)
|
2021-02-24 20:20:12 +08:00
|
|
|
|
2020-12-14 21:25:34 +08:00
|
|
|
// BazelTargetModuleProperties contain properties and metadata used for
|
|
|
|
// Blueprint to BUILD file conversion.
|
|
|
|
type BazelTargetModuleProperties struct {
|
|
|
|
// The Bazel rule class for this target.
|
2021-02-20 00:06:17 +08:00
|
|
|
Rule_class string `blueprint:"mutated"`
|
2021-01-27 10:58:43 +08:00
|
|
|
|
|
|
|
// The target label for the bzl file containing the definition of the rule class.
|
2021-02-20 00:06:17 +08:00
|
|
|
Bzl_load_location string `blueprint:"mutated"`
|
2020-12-14 21:25:34 +08:00
|
|
|
}
|
2021-01-26 22:18:53 +08:00
|
|
|
|
2021-02-07 23:05:16 +08:00
|
|
|
const BazelTargetModuleNamePrefix = "__bp2build__"
|
|
|
|
|
2021-03-24 22:14:47 +08:00
|
|
|
var productVariableSubstitutionPattern = regexp.MustCompile("%(d|s)")
|
|
|
|
|
2021-04-19 13:00:15 +08:00
|
|
|
// Label is used to represent a Bazel compatible Label. Also stores the original
|
|
|
|
// bp text to support string replacement.
|
2021-01-26 22:18:53 +08:00
|
|
|
type Label struct {
|
2021-04-19 13:00:15 +08:00
|
|
|
// The string representation of a Bazel target label. This can be a relative
|
|
|
|
// or fully qualified label. These labels are used for generating BUILD
|
|
|
|
// files with bp2build.
|
|
|
|
Label string
|
|
|
|
|
|
|
|
// The original Soong/Blueprint module name that the label was derived from.
|
|
|
|
// This is used for replacing references to the original name with the new
|
|
|
|
// label, for example in genrule cmds.
|
|
|
|
//
|
|
|
|
// While there is a reversible 1:1 mapping from the module name to Bazel
|
|
|
|
// label with bp2build that could make computing the original module name
|
|
|
|
// from the label automatic, it is not the case for handcrafted targets,
|
|
|
|
// where modules can have a custom label mapping through the { bazel_module:
|
|
|
|
// { label: <label> } } property.
|
|
|
|
//
|
|
|
|
// With handcrafted labels, those modules don't go through bp2build
|
|
|
|
// conversion, but relies on handcrafted targets in the source tree.
|
|
|
|
OriginalModuleName string
|
2021-01-26 22:18:53 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// LabelList is used to represent a list of Bazel labels.
|
|
|
|
type LabelList struct {
|
|
|
|
Includes []Label
|
|
|
|
Excludes []Label
|
|
|
|
}
|
|
|
|
|
2021-03-24 22:04:33 +08:00
|
|
|
// GlobsInDir returns a list of glob expressions for a list of extensions
|
|
|
|
// (optionally recursive) within a directory.
|
|
|
|
func GlobsInDir(dir string, recursive bool, extensions []string) []string {
|
|
|
|
globs := []string{}
|
|
|
|
|
|
|
|
globInfix := ""
|
|
|
|
if dir == "." {
|
|
|
|
if recursive {
|
|
|
|
// e.g "**/*.h"
|
|
|
|
globInfix = "**/"
|
|
|
|
} // else e.g. "*.h"
|
|
|
|
for _, ext := range extensions {
|
|
|
|
globs = append(globs, globInfix+"*"+ext)
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
if recursive {
|
|
|
|
// e.g. "foo/bar/**/*.h"
|
|
|
|
dir += "/**"
|
|
|
|
} // else e.g. "foo/bar/*.h"
|
|
|
|
for _, ext := range extensions {
|
|
|
|
globs = append(globs, dir+"/*"+ext)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return globs
|
|
|
|
}
|
|
|
|
|
|
|
|
// LooseHdrsGlobs returns the list of non-recursive header globs for each parent directory of
|
|
|
|
// each source file in this LabelList's Includes.
|
|
|
|
func (ll *LabelList) LooseHdrsGlobs(exts []string) []string {
|
|
|
|
var globs []string
|
|
|
|
for _, parentDir := range ll.uniqueParentDirectories() {
|
|
|
|
globs = append(globs,
|
|
|
|
GlobsInDir(parentDir, false, exts)...)
|
|
|
|
}
|
|
|
|
return globs
|
|
|
|
}
|
|
|
|
|
|
|
|
// uniqueParentDirectories returns a list of the unique parent directories for
|
|
|
|
// all files in ll.Includes.
|
|
|
|
func (ll *LabelList) uniqueParentDirectories() []string {
|
|
|
|
dirMap := map[string]bool{}
|
|
|
|
for _, label := range ll.Includes {
|
|
|
|
dirMap[filepath.Dir(label.Label)] = true
|
|
|
|
}
|
|
|
|
dirs := []string{}
|
|
|
|
for dir := range dirMap {
|
|
|
|
dirs = append(dirs, dir)
|
|
|
|
}
|
|
|
|
return dirs
|
|
|
|
}
|
|
|
|
|
2021-01-26 22:18:53 +08:00
|
|
|
// Append appends the fields of other labelList to the corresponding fields of ll.
|
|
|
|
func (ll *LabelList) Append(other LabelList) {
|
|
|
|
if len(ll.Includes) > 0 || len(other.Includes) > 0 {
|
|
|
|
ll.Includes = append(ll.Includes, other.Includes...)
|
|
|
|
}
|
|
|
|
if len(ll.Excludes) > 0 || len(other.Excludes) > 0 {
|
|
|
|
ll.Excludes = append(other.Excludes, other.Excludes...)
|
|
|
|
}
|
|
|
|
}
|
2021-02-24 20:20:12 +08:00
|
|
|
|
2021-03-12 19:04:21 +08:00
|
|
|
func UniqueBazelLabels(originalLabels []Label) []Label {
|
|
|
|
uniqueLabelsSet := make(map[Label]bool)
|
|
|
|
for _, l := range originalLabels {
|
|
|
|
uniqueLabelsSet[l] = true
|
|
|
|
}
|
|
|
|
var uniqueLabels []Label
|
|
|
|
for l, _ := range uniqueLabelsSet {
|
|
|
|
uniqueLabels = append(uniqueLabels, l)
|
|
|
|
}
|
|
|
|
sort.SliceStable(uniqueLabels, func(i, j int) bool {
|
|
|
|
return uniqueLabels[i].Label < uniqueLabels[j].Label
|
|
|
|
})
|
|
|
|
return uniqueLabels
|
|
|
|
}
|
|
|
|
|
|
|
|
func UniqueBazelLabelList(originalLabelList LabelList) LabelList {
|
|
|
|
var uniqueLabelList LabelList
|
|
|
|
uniqueLabelList.Includes = UniqueBazelLabels(originalLabelList.Includes)
|
|
|
|
uniqueLabelList.Excludes = UniqueBazelLabels(originalLabelList.Excludes)
|
|
|
|
return uniqueLabelList
|
|
|
|
}
|
|
|
|
|
2021-04-07 04:06:21 +08:00
|
|
|
// Subtract needle from haystack
|
|
|
|
func SubtractStrings(haystack []string, needle []string) []string {
|
|
|
|
// This is really a set
|
|
|
|
remainder := make(map[string]bool)
|
|
|
|
|
|
|
|
for _, s := range haystack {
|
|
|
|
remainder[s] = true
|
|
|
|
}
|
|
|
|
for _, s := range needle {
|
|
|
|
delete(remainder, s)
|
|
|
|
}
|
|
|
|
|
|
|
|
var strings []string
|
|
|
|
for s, _ := range remainder {
|
|
|
|
strings = append(strings, s)
|
|
|
|
}
|
|
|
|
|
|
|
|
sort.SliceStable(strings, func(i, j int) bool {
|
|
|
|
return strings[i] < strings[j]
|
|
|
|
})
|
|
|
|
|
|
|
|
return strings
|
|
|
|
}
|
|
|
|
|
|
|
|
// Subtract needle from haystack
|
|
|
|
func SubtractBazelLabels(haystack []Label, needle []Label) []Label {
|
|
|
|
// This is really a set
|
|
|
|
remainder := make(map[Label]bool)
|
|
|
|
|
|
|
|
for _, label := range haystack {
|
|
|
|
remainder[label] = true
|
|
|
|
}
|
|
|
|
for _, label := range needle {
|
|
|
|
delete(remainder, label)
|
|
|
|
}
|
|
|
|
|
|
|
|
var labels []Label
|
|
|
|
for label, _ := range remainder {
|
|
|
|
labels = append(labels, label)
|
|
|
|
}
|
|
|
|
|
|
|
|
sort.SliceStable(labels, func(i, j int) bool {
|
|
|
|
return labels[i].Label < labels[j].Label
|
|
|
|
})
|
|
|
|
|
|
|
|
return labels
|
|
|
|
}
|
|
|
|
|
|
|
|
// Subtract needle from haystack
|
|
|
|
func SubtractBazelLabelList(haystack LabelList, needle LabelList) LabelList {
|
|
|
|
var result LabelList
|
|
|
|
result.Includes = SubtractBazelLabels(haystack.Includes, needle.Includes)
|
|
|
|
// NOTE: Excludes are intentionally not subtracted
|
|
|
|
result.Excludes = haystack.Excludes
|
|
|
|
return result
|
|
|
|
}
|
|
|
|
|
2021-03-15 18:02:43 +08:00
|
|
|
const (
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
// ArchType names in arch.go
|
2021-03-15 18:02:43 +08:00
|
|
|
ARCH_ARM = "arm"
|
|
|
|
ARCH_ARM64 = "arm64"
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
ARCH_X86 = "x86"
|
|
|
|
ARCH_X86_64 = "x86_64"
|
|
|
|
|
|
|
|
// OsType names in arch.go
|
|
|
|
OS_ANDROID = "android"
|
|
|
|
OS_DARWIN = "darwin"
|
|
|
|
OS_FUCHSIA = "fuchsia"
|
|
|
|
OS_LINUX = "linux_glibc"
|
|
|
|
OS_LINUX_BIONIC = "linux_bionic"
|
|
|
|
OS_WINDOWS = "windows"
|
2021-03-15 18:02:43 +08:00
|
|
|
)
|
|
|
|
|
|
|
|
var (
|
2021-04-05 18:35:13 +08:00
|
|
|
// These are the list of OSes and architectures with a Bazel config_setting
|
|
|
|
// and constraint value equivalent. These exist in arch.go, but the android
|
|
|
|
// package depends on the bazel package, so a cyclic dependency prevents
|
|
|
|
// using those variables here.
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
|
|
|
|
// A map of architectures to the Bazel label of the constraint_value
|
|
|
|
// for the @platforms//cpu:cpu constraint_setting
|
|
|
|
PlatformArchMap = map[string]string{
|
|
|
|
ARCH_ARM: "//build/bazel/platforms/arch:arm",
|
|
|
|
ARCH_ARM64: "//build/bazel/platforms/arch:arm64",
|
|
|
|
ARCH_X86: "//build/bazel/platforms/arch:x86",
|
|
|
|
ARCH_X86_64: "//build/bazel/platforms/arch:x86_64",
|
|
|
|
}
|
|
|
|
|
|
|
|
// A map of target operating systems to the Bazel label of the
|
|
|
|
// constraint_value for the @platforms//os:os constraint_setting
|
|
|
|
PlatformOsMap = map[string]string{
|
|
|
|
OS_ANDROID: "//build/bazel/platforms/os:android",
|
|
|
|
OS_DARWIN: "//build/bazel/platforms/os:darwin",
|
|
|
|
OS_FUCHSIA: "//build/bazel/platforms/os:fuchsia",
|
|
|
|
OS_LINUX: "//build/bazel/platforms/os:linux",
|
|
|
|
OS_LINUX_BIONIC: "//build/bazel/platforms/os:linux_bionic",
|
|
|
|
OS_WINDOWS: "//build/bazel/platforms/os:windows",
|
|
|
|
}
|
2021-03-15 18:02:43 +08:00
|
|
|
)
|
|
|
|
|
2021-04-05 18:35:13 +08:00
|
|
|
type Attribute interface {
|
|
|
|
HasConfigurableValues() bool
|
|
|
|
}
|
|
|
|
|
2021-03-15 18:02:43 +08:00
|
|
|
// Arch-specific label_list typed Bazel attribute values. This should correspond
|
|
|
|
// to the types of architectures supported for compilation in arch.go.
|
|
|
|
type labelListArchValues struct {
|
|
|
|
X86 LabelList
|
|
|
|
X86_64 LabelList
|
|
|
|
Arm LabelList
|
|
|
|
Arm64 LabelList
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
Common LabelList
|
|
|
|
}
|
|
|
|
|
|
|
|
type labelListOsValues struct {
|
|
|
|
Android LabelList
|
|
|
|
Darwin LabelList
|
|
|
|
Fuchsia LabelList
|
|
|
|
Linux LabelList
|
|
|
|
LinuxBionic LabelList
|
|
|
|
Windows LabelList
|
2021-03-15 18:02:43 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// LabelListAttribute is used to represent a list of Bazel labels as an
|
|
|
|
// attribute.
|
|
|
|
type LabelListAttribute struct {
|
|
|
|
// The non-arch specific attribute label list Value. Required.
|
|
|
|
Value LabelList
|
|
|
|
|
|
|
|
// The arch-specific attribute label list values. Optional. If used, these
|
|
|
|
// are generated in a select statement and appended to the non-arch specific
|
|
|
|
// label list Value.
|
|
|
|
ArchValues labelListArchValues
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
|
|
|
|
// The os-specific attribute label list values. Optional. If used, these
|
|
|
|
// are generated in a select statement and appended to the non-os specific
|
|
|
|
// label list Value.
|
|
|
|
OsValues labelListOsValues
|
2021-03-15 18:02:43 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// MakeLabelListAttribute initializes a LabelListAttribute with the non-arch specific value.
|
|
|
|
func MakeLabelListAttribute(value LabelList) LabelListAttribute {
|
|
|
|
return LabelListAttribute{Value: UniqueBazelLabelList(value)}
|
|
|
|
}
|
|
|
|
|
2021-03-24 22:04:33 +08:00
|
|
|
// Append appends all values, including os and arch specific ones, from another
|
|
|
|
// LabelListAttribute to this LabelListAttribute.
|
|
|
|
func (attrs *LabelListAttribute) Append(other LabelListAttribute) {
|
|
|
|
for arch := range PlatformArchMap {
|
|
|
|
this := attrs.GetValueForArch(arch)
|
|
|
|
that := other.GetValueForArch(arch)
|
|
|
|
this.Append(that)
|
|
|
|
attrs.SetValueForArch(arch, this)
|
|
|
|
}
|
|
|
|
|
|
|
|
for os := range PlatformOsMap {
|
|
|
|
this := attrs.GetValueForOS(os)
|
|
|
|
that := other.GetValueForOS(os)
|
|
|
|
this.Append(that)
|
|
|
|
attrs.SetValueForOS(os, this)
|
|
|
|
}
|
|
|
|
|
|
|
|
attrs.Value.Append(other.Value)
|
|
|
|
}
|
|
|
|
|
2021-03-15 18:02:43 +08:00
|
|
|
// HasArchSpecificValues returns true if the attribute contains
|
|
|
|
// architecture-specific label_list values.
|
2021-04-05 18:35:13 +08:00
|
|
|
func (attrs LabelListAttribute) HasConfigurableValues() bool {
|
|
|
|
for arch := range PlatformArchMap {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
if len(attrs.GetValueForArch(arch).Includes) > 0 {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-04-05 18:35:13 +08:00
|
|
|
for os := range PlatformOsMap {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
if len(attrs.GetValueForOS(os).Includes) > 0 {
|
2021-03-15 18:02:43 +08:00
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return false
|
|
|
|
}
|
|
|
|
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
func (attrs *LabelListAttribute) archValuePtrs() map[string]*LabelList {
|
|
|
|
return map[string]*LabelList{
|
|
|
|
ARCH_X86: &attrs.ArchValues.X86,
|
|
|
|
ARCH_X86_64: &attrs.ArchValues.X86_64,
|
|
|
|
ARCH_ARM: &attrs.ArchValues.Arm,
|
|
|
|
ARCH_ARM64: &attrs.ArchValues.Arm64,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-03-15 18:02:43 +08:00
|
|
|
// GetValueForArch returns the label_list attribute value for an architecture.
|
|
|
|
func (attrs *LabelListAttribute) GetValueForArch(arch string) LabelList {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
var v *LabelList
|
|
|
|
if v = attrs.archValuePtrs()[arch]; v == nil {
|
2021-03-15 18:02:43 +08:00
|
|
|
panic(fmt.Errorf("Unknown arch: %s", arch))
|
|
|
|
}
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
return *v
|
2021-03-15 18:02:43 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// SetValueForArch sets the label_list attribute value for an architecture.
|
|
|
|
func (attrs *LabelListAttribute) SetValueForArch(arch string, value LabelList) {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
var v *LabelList
|
|
|
|
if v = attrs.archValuePtrs()[arch]; v == nil {
|
2021-03-15 18:02:43 +08:00
|
|
|
panic(fmt.Errorf("Unknown arch: %s", arch))
|
|
|
|
}
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
*v = value
|
|
|
|
}
|
|
|
|
|
|
|
|
func (attrs *LabelListAttribute) osValuePtrs() map[string]*LabelList {
|
|
|
|
return map[string]*LabelList{
|
|
|
|
OS_ANDROID: &attrs.OsValues.Android,
|
|
|
|
OS_DARWIN: &attrs.OsValues.Darwin,
|
|
|
|
OS_FUCHSIA: &attrs.OsValues.Fuchsia,
|
|
|
|
OS_LINUX: &attrs.OsValues.Linux,
|
|
|
|
OS_LINUX_BIONIC: &attrs.OsValues.LinuxBionic,
|
|
|
|
OS_WINDOWS: &attrs.OsValues.Windows,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// GetValueForOS returns the label_list attribute value for an OS target.
|
|
|
|
func (attrs *LabelListAttribute) GetValueForOS(os string) LabelList {
|
|
|
|
var v *LabelList
|
|
|
|
if v = attrs.osValuePtrs()[os]; v == nil {
|
|
|
|
panic(fmt.Errorf("Unknown os: %s", os))
|
|
|
|
}
|
|
|
|
return *v
|
|
|
|
}
|
|
|
|
|
|
|
|
// SetValueForArch sets the label_list attribute value for an OS target.
|
|
|
|
func (attrs *LabelListAttribute) SetValueForOS(os string, value LabelList) {
|
|
|
|
var v *LabelList
|
|
|
|
if v = attrs.osValuePtrs()[os]; v == nil {
|
|
|
|
panic(fmt.Errorf("Unknown os: %s", os))
|
|
|
|
}
|
|
|
|
*v = value
|
2021-03-15 18:02:43 +08:00
|
|
|
}
|
|
|
|
|
2021-02-24 20:20:12 +08:00
|
|
|
// StringListAttribute corresponds to the string_list Bazel attribute type with
|
|
|
|
// support for additional metadata, like configurations.
|
|
|
|
type StringListAttribute struct {
|
|
|
|
// The base value of the string list attribute.
|
|
|
|
Value []string
|
|
|
|
|
2021-04-05 18:35:13 +08:00
|
|
|
// The arch-specific attribute string list values. Optional. If used, these
|
|
|
|
// are generated in a select statement and appended to the non-arch specific
|
|
|
|
// label list Value.
|
2021-02-24 20:20:12 +08:00
|
|
|
ArchValues stringListArchValues
|
2021-04-05 18:35:13 +08:00
|
|
|
|
|
|
|
// The os-specific attribute string list values. Optional. If used, these
|
|
|
|
// are generated in a select statement and appended to the non-os specific
|
|
|
|
// label list Value.
|
|
|
|
OsValues stringListOsValues
|
2021-02-24 20:20:12 +08:00
|
|
|
}
|
|
|
|
|
2021-04-07 04:06:21 +08:00
|
|
|
// MakeStringListAttribute initializes a StringListAttribute with the non-arch specific value.
|
|
|
|
func MakeStringListAttribute(value []string) StringListAttribute {
|
|
|
|
// NOTE: These strings are not necessarily unique or sorted.
|
|
|
|
return StringListAttribute{Value: value}
|
|
|
|
}
|
|
|
|
|
2021-02-24 20:20:12 +08:00
|
|
|
// Arch-specific string_list typed Bazel attribute values. This should correspond
|
|
|
|
// to the types of architectures supported for compilation in arch.go.
|
|
|
|
type stringListArchValues struct {
|
2021-03-15 18:02:43 +08:00
|
|
|
X86 []string
|
|
|
|
X86_64 []string
|
|
|
|
Arm []string
|
|
|
|
Arm64 []string
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
Common []string
|
2021-02-24 20:20:12 +08:00
|
|
|
}
|
|
|
|
|
2021-04-05 18:35:13 +08:00
|
|
|
type stringListOsValues struct {
|
|
|
|
Android []string
|
|
|
|
Darwin []string
|
|
|
|
Fuchsia []string
|
|
|
|
Linux []string
|
|
|
|
LinuxBionic []string
|
|
|
|
Windows []string
|
|
|
|
}
|
|
|
|
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
// HasConfigurableValues returns true if the attribute contains
|
2021-02-24 20:20:12 +08:00
|
|
|
// architecture-specific string_list values.
|
2021-04-05 18:35:13 +08:00
|
|
|
func (attrs StringListAttribute) HasConfigurableValues() bool {
|
|
|
|
for arch := range PlatformArchMap {
|
2021-02-24 20:20:12 +08:00
|
|
|
if len(attrs.GetValueForArch(arch)) > 0 {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
2021-04-05 18:35:13 +08:00
|
|
|
|
|
|
|
for os := range PlatformOsMap {
|
|
|
|
if len(attrs.GetValueForOS(os)) > 0 {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
2021-02-24 20:20:12 +08:00
|
|
|
return false
|
|
|
|
}
|
|
|
|
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
func (attrs *StringListAttribute) archValuePtrs() map[string]*[]string {
|
|
|
|
return map[string]*[]string{
|
|
|
|
ARCH_X86: &attrs.ArchValues.X86,
|
|
|
|
ARCH_X86_64: &attrs.ArchValues.X86_64,
|
|
|
|
ARCH_ARM: &attrs.ArchValues.Arm,
|
|
|
|
ARCH_ARM64: &attrs.ArchValues.Arm64,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-24 20:20:12 +08:00
|
|
|
// GetValueForArch returns the string_list attribute value for an architecture.
|
|
|
|
func (attrs *StringListAttribute) GetValueForArch(arch string) []string {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
var v *[]string
|
|
|
|
if v = attrs.archValuePtrs()[arch]; v == nil {
|
2021-02-24 20:20:12 +08:00
|
|
|
panic(fmt.Errorf("Unknown arch: %s", arch))
|
|
|
|
}
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
return *v
|
2021-02-24 20:20:12 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// SetValueForArch sets the string_list attribute value for an architecture.
|
|
|
|
func (attrs *StringListAttribute) SetValueForArch(arch string, value []string) {
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
var v *[]string
|
|
|
|
if v = attrs.archValuePtrs()[arch]; v == nil {
|
2021-02-24 20:20:12 +08:00
|
|
|
panic(fmt.Errorf("Unknown arch: %s", arch))
|
|
|
|
}
|
Add os/target configurable selects for label list attributes.
This CL is pretty large, so I recommend starting with reading the newly
added tests for the expected behavior.
This change works in conjunction with the linked CLs in the Gerrit topic.
Those CLs add support for new platform() definitions for OS targets
specified in Soong's arch.go, which are configurable through
Android.bp's `target {}` property. It works similary to previous CLs
adding support for the `arch {}` property.
These configurable props are keyed by the OS: android, linux_bionic,
windows, and so on. They map to `select` statements in label list
attributes, which this CL enables for cc_library_headers' header_libs
and export_header_lib_headers props.
This enables //bionic/libc:libc_headers to be generated correctly, from:
cc_library_headers {
name: "libc_headers",
target: {
android: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
linux_bionic: {
header_libs: ["libc_headers_arch"],
export_header_lib_headers: ["libc_headers_arch"],
},
},
// omitted props
}
to:
cc_library_headers(
name = "libc_headers",
deps = [] + select({
"//build/bazel/platforms/os:android": [
":libc_headers_arch",
],
"//build/bazel/platforms/os:linux_bionic": [
":libc_headers_arch",
],
"//conditions:default": [],
}),
)
Test: TH
Test: Verify generated //bionic/libc:libc_headers
Fixes: 183597786
Change-Id: I01016cc2cc9a71449f02300d747f01decebf3f6e
2021-03-24 14:18:33 +08:00
|
|
|
*v = value
|
2021-02-24 20:20:12 +08:00
|
|
|
}
|
2021-03-24 22:14:47 +08:00
|
|
|
|
2021-04-05 18:35:13 +08:00
|
|
|
func (attrs *StringListAttribute) osValuePtrs() map[string]*[]string {
|
|
|
|
return map[string]*[]string{
|
|
|
|
OS_ANDROID: &attrs.OsValues.Android,
|
|
|
|
OS_DARWIN: &attrs.OsValues.Darwin,
|
|
|
|
OS_FUCHSIA: &attrs.OsValues.Fuchsia,
|
|
|
|
OS_LINUX: &attrs.OsValues.Linux,
|
|
|
|
OS_LINUX_BIONIC: &attrs.OsValues.LinuxBionic,
|
|
|
|
OS_WINDOWS: &attrs.OsValues.Windows,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// GetValueForOS returns the string_list attribute value for an OS target.
|
|
|
|
func (attrs *StringListAttribute) GetValueForOS(os string) []string {
|
|
|
|
var v *[]string
|
|
|
|
if v = attrs.osValuePtrs()[os]; v == nil {
|
|
|
|
panic(fmt.Errorf("Unknown os: %s", os))
|
|
|
|
}
|
|
|
|
return *v
|
|
|
|
}
|
|
|
|
|
|
|
|
// SetValueForArch sets the string_list attribute value for an OS target.
|
|
|
|
func (attrs *StringListAttribute) SetValueForOS(os string, value []string) {
|
|
|
|
var v *[]string
|
|
|
|
if v = attrs.osValuePtrs()[os]; v == nil {
|
|
|
|
panic(fmt.Errorf("Unknown os: %s", os))
|
|
|
|
}
|
|
|
|
*v = value
|
|
|
|
}
|
|
|
|
|
2021-03-24 22:14:47 +08:00
|
|
|
// TryVariableSubstitution, replace string substitution formatting within each string in slice with
|
|
|
|
// Starlark string.format compatible tag for productVariable.
|
|
|
|
func TryVariableSubstitutions(slice []string, productVariable string) ([]string, bool) {
|
|
|
|
ret := make([]string, 0, len(slice))
|
|
|
|
changesMade := false
|
|
|
|
for _, s := range slice {
|
|
|
|
newS, changed := TryVariableSubstitution(s, productVariable)
|
|
|
|
ret = append(ret, newS)
|
|
|
|
changesMade = changesMade || changed
|
|
|
|
}
|
|
|
|
return ret, changesMade
|
|
|
|
}
|
|
|
|
|
|
|
|
// TryVariableSubstitution, replace string substitution formatting within s with Starlark
|
|
|
|
// string.format compatible tag for productVariable.
|
|
|
|
func TryVariableSubstitution(s string, productVariable string) (string, bool) {
|
|
|
|
sub := productVariableSubstitutionPattern.ReplaceAllString(s, "{"+productVariable+"}")
|
|
|
|
return sub, s != sub
|
|
|
|
}
|