2019-11-23 01:03:51 +01:00
|
|
|
// 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.
|
|
|
|
|
|
|
|
package android
|
|
|
|
|
|
|
|
// This file provides module types that implement wrapper module types that add conditionals on
|
|
|
|
// Soong config variables.
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"path/filepath"
|
|
|
|
"strings"
|
|
|
|
"text/scanner"
|
|
|
|
|
|
|
|
"github.com/google/blueprint"
|
|
|
|
"github.com/google/blueprint/parser"
|
|
|
|
"github.com/google/blueprint/proptools"
|
|
|
|
|
|
|
|
"android/soong/android/soongconfig"
|
|
|
|
)
|
|
|
|
|
|
|
|
func init() {
|
|
|
|
RegisterModuleType("soong_config_module_type_import", soongConfigModuleTypeImportFactory)
|
|
|
|
RegisterModuleType("soong_config_module_type", soongConfigModuleTypeFactory)
|
|
|
|
RegisterModuleType("soong_config_string_variable", soongConfigStringVariableDummyFactory)
|
|
|
|
RegisterModuleType("soong_config_bool_variable", soongConfigBoolVariableDummyFactory)
|
|
|
|
}
|
|
|
|
|
|
|
|
type soongConfigModuleTypeImport struct {
|
|
|
|
ModuleBase
|
|
|
|
properties soongConfigModuleTypeImportProperties
|
|
|
|
}
|
|
|
|
|
|
|
|
type soongConfigModuleTypeImportProperties struct {
|
|
|
|
From string
|
|
|
|
Module_types []string
|
|
|
|
}
|
|
|
|
|
|
|
|
// soong_config_module_type_import imports module types with conditionals on Soong config
|
|
|
|
// variables from another Android.bp file. The imported module type will exist for all
|
|
|
|
// modules after the import in the Android.bp file.
|
|
|
|
//
|
|
|
|
// For example, an Android.bp file could have:
|
|
|
|
//
|
|
|
|
// soong_config_module_type_import {
|
2020-02-04 22:17:24 +01:00
|
|
|
// from: "device/acme/Android.bp",
|
2019-11-23 01:03:51 +01:00
|
|
|
// module_types: ["acme_cc_defaults"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// acme_cc_defaults {
|
|
|
|
// name: "acme_defaults",
|
|
|
|
// cflags: ["-DGENERIC"],
|
|
|
|
// soong_config_variables: {
|
|
|
|
// board: {
|
|
|
|
// soc_a: {
|
|
|
|
// cflags: ["-DSOC_A"],
|
|
|
|
// },
|
|
|
|
// soc_b: {
|
|
|
|
// cflags: ["-DSOC_B"],
|
|
|
|
// },
|
|
|
|
// },
|
|
|
|
// feature: {
|
|
|
|
// cflags: ["-DFEATURE"],
|
|
|
|
// },
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// width: {
|
|
|
|
// cflags: ["-DWIDTH=%s"],
|
|
|
|
// },
|
2019-11-23 01:03:51 +01:00
|
|
|
// },
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// cc_library {
|
|
|
|
// name: "libacme_foo",
|
|
|
|
// defaults: ["acme_defaults"],
|
|
|
|
// srcs: ["*.cpp"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// And device/acme/Android.bp could have:
|
|
|
|
//
|
|
|
|
// soong_config_module_type {
|
|
|
|
// name: "acme_cc_defaults",
|
|
|
|
// module_type: "cc_defaults",
|
|
|
|
// config_namespace: "acme",
|
2020-03-24 03:39:34 +01:00
|
|
|
// variables: ["board"],
|
|
|
|
// bool_variables: ["feature"],
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// value_variables: ["width"],
|
2019-11-23 01:03:51 +01:00
|
|
|
// properties: ["cflags", "srcs"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// soong_config_string_variable {
|
|
|
|
// name: "board",
|
|
|
|
// values: ["soc_a", "soc_b"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// If an acme BoardConfig.mk file contained:
|
|
|
|
//
|
|
|
|
// SOONG_CONFIG_NAMESPACES += acme
|
|
|
|
// SOONG_CONFIG_acme += \
|
|
|
|
// board \
|
|
|
|
// feature \
|
|
|
|
//
|
|
|
|
// SOONG_CONFIG_acme_board := soc_a
|
|
|
|
// SOONG_CONFIG_acme_feature := true
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// SOONG_CONFIG_acme_width := 200
|
2019-11-23 01:03:51 +01:00
|
|
|
//
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// Then libacme_foo would build with cflags "-DGENERIC -DSOC_A -DFEATURE -DWIDTH=200".
|
2019-11-23 01:03:51 +01:00
|
|
|
func soongConfigModuleTypeImportFactory() Module {
|
|
|
|
module := &soongConfigModuleTypeImport{}
|
|
|
|
|
|
|
|
module.AddProperties(&module.properties)
|
|
|
|
AddLoadHook(module, func(ctx LoadHookContext) {
|
|
|
|
importModuleTypes(ctx, module.properties.From, module.properties.Module_types...)
|
|
|
|
})
|
|
|
|
|
|
|
|
initAndroidModuleBase(module)
|
|
|
|
return module
|
|
|
|
}
|
|
|
|
|
|
|
|
func (m *soongConfigModuleTypeImport) Name() string {
|
2020-03-11 00:10:06 +01:00
|
|
|
// The generated name is non-deterministic, but it does not
|
|
|
|
// matter because this module does not emit any rules.
|
|
|
|
return soongconfig.CanonicalizeToProperty(m.properties.From) +
|
|
|
|
"soong_config_module_type_import_" + fmt.Sprintf("%p", m)
|
2019-11-23 01:03:51 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
func (*soongConfigModuleTypeImport) Nameless() {}
|
|
|
|
func (*soongConfigModuleTypeImport) GenerateAndroidBuildActions(ModuleContext) {}
|
|
|
|
|
|
|
|
// Create dummy modules for soong_config_module_type and soong_config_*_variable
|
|
|
|
|
|
|
|
type soongConfigModuleTypeModule struct {
|
|
|
|
ModuleBase
|
|
|
|
properties soongconfig.ModuleTypeProperties
|
|
|
|
}
|
|
|
|
|
|
|
|
// soong_config_module_type defines module types with conditionals on Soong config
|
2020-02-04 22:17:24 +01:00
|
|
|
// variables. The new module type will exist for all modules after the definition
|
|
|
|
// in an Android.bp file, and can be imported into other Android.bp files using
|
|
|
|
// soong_config_module_type_import.
|
2019-11-23 01:03:51 +01:00
|
|
|
//
|
|
|
|
// For example, an Android.bp file could have:
|
|
|
|
//
|
|
|
|
// soong_config_module_type {
|
|
|
|
// name: "acme_cc_defaults",
|
|
|
|
// module_type: "cc_defaults",
|
|
|
|
// config_namespace: "acme",
|
2020-03-24 03:39:34 +01:00
|
|
|
// variables: ["board"],
|
|
|
|
// bool_variables: ["feature"],
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// value_variables: ["width"],
|
2019-11-23 01:03:51 +01:00
|
|
|
// properties: ["cflags", "srcs"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// soong_config_string_variable {
|
|
|
|
// name: "board",
|
|
|
|
// values: ["soc_a", "soc_b"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// acme_cc_defaults {
|
|
|
|
// name: "acme_defaults",
|
|
|
|
// cflags: ["-DGENERIC"],
|
|
|
|
// soong_config_variables: {
|
|
|
|
// board: {
|
|
|
|
// soc_a: {
|
|
|
|
// cflags: ["-DSOC_A"],
|
|
|
|
// },
|
|
|
|
// soc_b: {
|
|
|
|
// cflags: ["-DSOC_B"],
|
|
|
|
// },
|
|
|
|
// },
|
|
|
|
// feature: {
|
|
|
|
// cflags: ["-DFEATURE"],
|
|
|
|
// },
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// width: {
|
|
|
|
// cflags: ["-DWIDTH=%s"],
|
|
|
|
// },
|
2019-11-23 01:03:51 +01:00
|
|
|
// },
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// cc_library {
|
|
|
|
// name: "libacme_foo",
|
|
|
|
// defaults: ["acme_defaults"],
|
|
|
|
// srcs: ["*.cpp"],
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// If an acme BoardConfig.mk file contained:
|
|
|
|
//
|
|
|
|
// SOONG_CONFIG_NAMESPACES += acme
|
|
|
|
// SOONG_CONFIG_acme += \
|
|
|
|
// board \
|
|
|
|
// feature \
|
|
|
|
//
|
|
|
|
// SOONG_CONFIG_acme_board := soc_a
|
|
|
|
// SOONG_CONFIG_acme_feature := true
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
// SOONG_CONFIG_acme_width := 200
|
2019-11-23 01:03:51 +01:00
|
|
|
//
|
|
|
|
// Then libacme_foo would build with cflags "-DGENERIC -DSOC_A -DFEATURE".
|
|
|
|
func soongConfigModuleTypeFactory() Module {
|
|
|
|
module := &soongConfigModuleTypeModule{}
|
|
|
|
|
|
|
|
module.AddProperties(&module.properties)
|
|
|
|
|
|
|
|
AddLoadHook(module, func(ctx LoadHookContext) {
|
|
|
|
// A soong_config_module_type module should implicitly import itself.
|
|
|
|
importModuleTypes(ctx, ctx.BlueprintsFile(), module.properties.Name)
|
|
|
|
})
|
|
|
|
|
|
|
|
initAndroidModuleBase(module)
|
|
|
|
|
|
|
|
return module
|
|
|
|
}
|
|
|
|
|
|
|
|
func (m *soongConfigModuleTypeModule) Name() string {
|
|
|
|
return m.properties.Name
|
|
|
|
}
|
|
|
|
func (*soongConfigModuleTypeModule) Nameless() {}
|
|
|
|
func (*soongConfigModuleTypeModule) GenerateAndroidBuildActions(ctx ModuleContext) {}
|
|
|
|
|
|
|
|
type soongConfigStringVariableDummyModule struct {
|
|
|
|
ModuleBase
|
|
|
|
properties soongconfig.VariableProperties
|
|
|
|
stringProperties soongconfig.StringVariableProperties
|
|
|
|
}
|
|
|
|
|
|
|
|
type soongConfigBoolVariableDummyModule struct {
|
|
|
|
ModuleBase
|
|
|
|
properties soongconfig.VariableProperties
|
|
|
|
}
|
|
|
|
|
|
|
|
// soong_config_string_variable defines a variable and a set of possible string values for use
|
|
|
|
// in a soong_config_module_type definition.
|
|
|
|
func soongConfigStringVariableDummyFactory() Module {
|
|
|
|
module := &soongConfigStringVariableDummyModule{}
|
|
|
|
module.AddProperties(&module.properties, &module.stringProperties)
|
|
|
|
initAndroidModuleBase(module)
|
|
|
|
return module
|
|
|
|
}
|
|
|
|
|
|
|
|
// soong_config_string_variable defines a variable with true or false values for use
|
|
|
|
// in a soong_config_module_type definition.
|
|
|
|
func soongConfigBoolVariableDummyFactory() Module {
|
|
|
|
module := &soongConfigBoolVariableDummyModule{}
|
|
|
|
module.AddProperties(&module.properties)
|
|
|
|
initAndroidModuleBase(module)
|
|
|
|
return module
|
|
|
|
}
|
|
|
|
|
|
|
|
func (m *soongConfigStringVariableDummyModule) Name() string {
|
|
|
|
return m.properties.Name
|
|
|
|
}
|
|
|
|
func (*soongConfigStringVariableDummyModule) Nameless() {}
|
|
|
|
func (*soongConfigStringVariableDummyModule) GenerateAndroidBuildActions(ctx ModuleContext) {}
|
|
|
|
|
|
|
|
func (m *soongConfigBoolVariableDummyModule) Name() string {
|
|
|
|
return m.properties.Name
|
|
|
|
}
|
|
|
|
func (*soongConfigBoolVariableDummyModule) Nameless() {}
|
|
|
|
func (*soongConfigBoolVariableDummyModule) GenerateAndroidBuildActions(ctx ModuleContext) {}
|
|
|
|
|
|
|
|
func importModuleTypes(ctx LoadHookContext, from string, moduleTypes ...string) {
|
|
|
|
from = filepath.Clean(from)
|
|
|
|
if filepath.Ext(from) != ".bp" {
|
|
|
|
ctx.PropertyErrorf("from", "%q must be a file with extension .bp", from)
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
|
|
|
if strings.HasPrefix(from, "../") {
|
|
|
|
ctx.PropertyErrorf("from", "%q must not use ../ to escape the source tree",
|
|
|
|
from)
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
|
|
|
moduleTypeDefinitions := loadSoongConfigModuleTypeDefinition(ctx, from)
|
|
|
|
if moduleTypeDefinitions == nil {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
for _, moduleType := range moduleTypes {
|
|
|
|
if factory, ok := moduleTypeDefinitions[moduleType]; ok {
|
|
|
|
ctx.registerScopedModuleType(moduleType, factory)
|
|
|
|
} else {
|
|
|
|
ctx.PropertyErrorf("module_types", "module type %q not defined in %q",
|
|
|
|
moduleType, from)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// loadSoongConfigModuleTypeDefinition loads module types from an Android.bp file. It caches the
|
|
|
|
// result so each file is only parsed once.
|
|
|
|
func loadSoongConfigModuleTypeDefinition(ctx LoadHookContext, from string) map[string]blueprint.ModuleFactory {
|
|
|
|
type onceKeyType string
|
|
|
|
key := NewCustomOnceKey(onceKeyType(filepath.Clean(from)))
|
|
|
|
|
|
|
|
reportErrors := func(ctx LoadHookContext, filename string, errs ...error) {
|
|
|
|
for _, err := range errs {
|
|
|
|
if parseErr, ok := err.(*parser.ParseError); ok {
|
|
|
|
ctx.Errorf(parseErr.Pos, "%s", parseErr.Err)
|
|
|
|
} else {
|
|
|
|
ctx.Errorf(scanner.Position{Filename: filename}, "%s", err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return ctx.Config().Once(key, func() interface{} {
|
2020-02-06 01:26:19 +01:00
|
|
|
ctx.AddNinjaFileDeps(from)
|
2019-11-23 01:03:51 +01:00
|
|
|
r, err := ctx.Config().fs.Open(from)
|
|
|
|
if err != nil {
|
|
|
|
ctx.PropertyErrorf("from", "failed to open %q: %s", from, err)
|
|
|
|
return (map[string]blueprint.ModuleFactory)(nil)
|
|
|
|
}
|
|
|
|
|
|
|
|
mtDef, errs := soongconfig.Parse(r, from)
|
|
|
|
|
|
|
|
if len(errs) > 0 {
|
|
|
|
reportErrors(ctx, from, errs...)
|
|
|
|
return (map[string]blueprint.ModuleFactory)(nil)
|
|
|
|
}
|
|
|
|
|
|
|
|
globalModuleTypes := ctx.moduleFactories()
|
|
|
|
|
|
|
|
factories := make(map[string]blueprint.ModuleFactory)
|
|
|
|
|
|
|
|
for name, moduleType := range mtDef.ModuleTypes {
|
|
|
|
factory := globalModuleTypes[moduleType.BaseModuleType]
|
|
|
|
if factory != nil {
|
|
|
|
factories[name] = soongConfigModuleFactory(factory, moduleType)
|
|
|
|
} else {
|
|
|
|
reportErrors(ctx, from,
|
|
|
|
fmt.Errorf("missing global module type factory for %q", moduleType.BaseModuleType))
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if ctx.Failed() {
|
|
|
|
return (map[string]blueprint.ModuleFactory)(nil)
|
|
|
|
}
|
|
|
|
|
|
|
|
return factories
|
|
|
|
}).(map[string]blueprint.ModuleFactory)
|
|
|
|
}
|
|
|
|
|
|
|
|
// soongConfigModuleFactory takes an existing soongConfigModuleFactory and a ModuleType and returns
|
|
|
|
// a new soongConfigModuleFactory that wraps the existing soongConfigModuleFactory and adds conditional on Soong config
|
|
|
|
// variables.
|
|
|
|
func soongConfigModuleFactory(factory blueprint.ModuleFactory,
|
|
|
|
moduleType *soongconfig.ModuleType) blueprint.ModuleFactory {
|
|
|
|
|
|
|
|
conditionalFactoryProps := soongconfig.CreateProperties(factory, moduleType)
|
|
|
|
if conditionalFactoryProps.IsValid() {
|
|
|
|
return func() (blueprint.Module, []interface{}) {
|
|
|
|
module, props := factory()
|
|
|
|
|
2020-01-28 18:46:50 +01:00
|
|
|
conditionalProps := proptools.CloneEmptyProperties(conditionalFactoryProps)
|
2019-11-23 01:03:51 +01:00
|
|
|
props = append(props, conditionalProps.Interface())
|
|
|
|
|
|
|
|
AddLoadHook(module, func(ctx LoadHookContext) {
|
|
|
|
config := ctx.Config().VendorConfig(moduleType.ConfigNamespace)
|
soong config: add value_variable substitution
There are some cases that aren't handled with the existing variable
types for booleans or known lists of strings. Similarly to our
product_variables that uses %s / %d for things like
PLATFORM_SDK_VERSION, allow vendors to define their own config variables
to be substituted into properties.
For example, some of the makefiles that I've attempted to convert had
the option to pass in version numbers from the board, or the default
display size:
-DDISPLAY_VERSION=550
-DDISP_H=1080
These examples happen to be integers, but since our configuration
language (make) doesn't support numbers, %s works just as well.
This change will allow the above to be represented using:
soong_config_module_type {
name: "acme_cc_defaults",
module_type: "cc_defaults",
config_namespace: "acme",
value_variables: [
"DISPLAY_VERSION",
"DISP_H",
],
properties: ["cflags"],
}
acme_cc_defaults {
name: "my_defaults",
soong_config_variables: {
DISPLAY_VERSION: {
cflags: ["-DDISPLAY_VERSION=%s"],
},
DISP_H: {
cflags: ["-DDISP_H=%s"],
}
},
}
Bug: 153161144
Test: built-in tests
Change-Id: I18f35746b5cc39c304a136980249e886d38c6df6
Merged-In: I18f35746b5cc39c304a136980249e886d38c6df6
(cherry picked from commit b0935db8c3838e90582fdece217bc956ba951f07)
2020-03-24 03:42:18 +01:00
|
|
|
newProps, err := soongconfig.PropertiesToApply(moduleType, conditionalProps, config)
|
|
|
|
if err != nil {
|
|
|
|
ctx.ModuleErrorf("%s", err)
|
|
|
|
return
|
|
|
|
}
|
|
|
|
for _, ps := range newProps {
|
2019-11-23 01:03:51 +01:00
|
|
|
ctx.AppendProperties(ps)
|
|
|
|
}
|
|
|
|
})
|
|
|
|
|
|
|
|
return module, props
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
return factory
|
|
|
|
}
|
|
|
|
}
|