diff options
author | Jingwen Chen <jingwen@google.com> | 2021-01-25 22:18:32 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2021-01-25 22:18:32 +0000 |
commit | 467f8b6c469d75a83e166fbbcf9c0b43c3b04fbc (patch) | |
tree | 247acc274d41567ca9d122077af37a838a6dbc9b /bp2build | |
parent | 1c1c8d1cd84ade622166f92db0c9d4aa434d1895 (diff) | |
parent | 288e2babde2525566b554e57b40ab9540887a5a6 (diff) | |
download | soong-467f8b6c469d75a83e166fbbcf9c0b43c3b04fbc.tar.gz |
Merge "queryview: prefix internal attribute names."
Diffstat (limited to 'bp2build')
-rw-r--r-- | bp2build/androidbp_to_build_templates.go | 38 | ||||
-rw-r--r-- | bp2build/build_conversion_test.go | 56 | ||||
-rw-r--r-- | bp2build/bzl_conversion.go | 6 | ||||
-rw-r--r-- | bp2build/bzl_conversion_test.go | 18 |
4 files changed, 59 insertions, 59 deletions
diff --git a/bp2build/androidbp_to_build_templates.go b/bp2build/androidbp_to_build_templates.go index 9bac86ba8..5fed4fab5 100644 --- a/bp2build/androidbp_to_build_templates.go +++ b/bp2build/androidbp_to_build_templates.go @@ -25,10 +25,10 @@ load("//build/bazel/queryview_rules:soong_module.bzl", "soong_module") // for expanding more attributes. soongModuleTarget = `soong_module( name = "%s", - module_name = "%s", - module_type = "%s", - module_variant = "%s", - module_deps = %s, + soong_module_name = "%s", + soong_module_type = "%s", + soong_module_variant = "%s", + soong_module_deps = %s, %s)` bazelTarget = `%s( @@ -38,7 +38,7 @@ load("//build/bazel/queryview_rules:soong_module.bzl", "soong_module") // A simple provider to mark and differentiate Soong module rule shims from // regular Bazel rules. Every Soong module rule shim returns a // SoongModuleInfo provider, and can only depend on rules returning - // SoongModuleInfo in the `module_deps` attribute. + // SoongModuleInfo in the `soong_module_deps` attribute. providersBzl = `SoongModuleInfo = provider( fields = { "name": "Name of module", @@ -57,19 +57,19 @@ load("//build/bazel/queryview_rules:providers.bzl", "SoongModuleInfo") def _generic_soong_module_impl(ctx): return [ SoongModuleInfo( - name = ctx.attr.module_name, - type = ctx.attr.module_type, - variant = ctx.attr.module_variant, + name = ctx.attr.soong_module_name, + type = ctx.attr.soong_module_type, + variant = ctx.attr.soong_module_variant, ), ] generic_soong_module = rule( implementation = _generic_soong_module_impl, attrs = { - "module_name": attr.string(mandatory = True), - "module_type": attr.string(mandatory = True), - "module_variant": attr.string(), - "module_deps": attr.label_list(providers = [SoongModuleInfo]), + "soong_module_name": attr.string(mandatory = True), + "soong_module_type": attr.string(mandatory = True), + "soong_module_variant": attr.string(), + "soong_module_deps": attr.label_list(providers = [SoongModuleInfo]), }, ) @@ -89,20 +89,20 @@ def _is_supported_type(value): else: return False -# soong_module is a macro that supports arbitrary kwargs, and uses module_type to +# soong_module is a macro that supports arbitrary kwargs, and uses soong_module_type to # expand to the right underlying shim. -def soong_module(name, module_type, **kwargs): - soong_module_rule = soong_module_rule_map.get(module_type) +def soong_module(name, soong_module_type, **kwargs): + soong_module_rule = soong_module_rule_map.get(soong_module_type) if soong_module_rule == None: # This module type does not have an existing rule to map to, so use the # generic_soong_module rule instead. generic_soong_module( name = name, - module_type = module_type, - module_name = kwargs.pop("module_name", ""), - module_variant = kwargs.pop("module_variant", ""), - module_deps = kwargs.pop("module_deps", []), + soong_module_type = soong_module_type, + soong_module_name = kwargs.pop("soong_module_name", ""), + soong_module_variant = kwargs.pop("soong_module_variant", ""), + soong_module_deps = kwargs.pop("soong_module_deps", []), ) else: supported_kwargs = dict() diff --git a/bp2build/build_conversion_test.go b/bp2build/build_conversion_test.go index 5fd3999f4..4e5161b62 100644 --- a/bp2build/build_conversion_test.go +++ b/bp2build/build_conversion_test.go @@ -31,10 +31,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], )`, }, @@ -46,10 +46,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], ramdisk = True, )`, @@ -62,10 +62,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], owner = "a_string_with\"quotes\"_and_\\backslashes\\\\", )`, @@ -78,10 +78,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], required = [ "bar", @@ -96,10 +96,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], target_required = [ "qux", @@ -124,10 +124,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], dist = { "tag": ".foo", @@ -162,10 +162,10 @@ func TestGenerateSoongModuleTargets(t *testing.T) { `, expectedBazelTarget: `soong_module( name = "foo", - module_name = "foo", - module_type = "custom", - module_variant = "", - module_deps = [ + soong_module_name = "foo", + soong_module_type = "custom", + soong_module_variant = "", + soong_module_deps = [ ], dists = [ { diff --git a/bp2build/bzl_conversion.go b/bp2build/bzl_conversion.go index 04c45420b..f2f6b0180 100644 --- a/bp2build/bzl_conversion.go +++ b/bp2build/bzl_conversion.go @@ -109,9 +109,9 @@ func generateRules(moduleTypeFactories map[string]android.ModuleFactory) map[str factoryName := runtime.FuncForPC(reflect.ValueOf(factory).Pointer()).Name() pkg := strings.Split(factoryName, ".")[0] attrs := `{ - "module_name": attr.string(mandatory = True), - "module_variant": attr.string(), - "module_deps": attr.label_list(providers = [SoongModuleInfo]), + "soong_module_name": attr.string(mandatory = True), + "soong_module_variant": attr.string(), + "soong_module_deps": attr.label_list(providers = [SoongModuleInfo]), ` attrs += getAttributes(factory) attrs += " }," diff --git a/bp2build/bzl_conversion_test.go b/bp2build/bzl_conversion_test.go index 01c727123..c1e660eb6 100644 --- a/bp2build/bzl_conversion_test.go +++ b/bp2build/bzl_conversion_test.go @@ -83,9 +83,9 @@ def _custom_impl(ctx): custom = rule( implementation = _custom_impl, attrs = { - "module_name": attr.string(mandatory = True), - "module_variant": attr.string(), - "module_deps": attr.label_list(providers = [SoongModuleInfo]), + "soong_module_name": attr.string(mandatory = True), + "soong_module_variant": attr.string(), + "soong_module_deps": attr.label_list(providers = [SoongModuleInfo]), "bool_prop": attr.bool(), "bool_ptr_prop": attr.bool(), "int64_ptr_prop": attr.int(), @@ -107,9 +107,9 @@ def _custom_defaults_impl(ctx): custom_defaults = rule( implementation = _custom_defaults_impl, attrs = { - "module_name": attr.string(mandatory = True), - "module_variant": attr.string(), - "module_deps": attr.label_list(providers = [SoongModuleInfo]), + "soong_module_name": attr.string(mandatory = True), + "soong_module_variant": attr.string(), + "soong_module_deps": attr.label_list(providers = [SoongModuleInfo]), "bool_prop": attr.bool(), "bool_ptr_prop": attr.bool(), "int64_ptr_prop": attr.int(), @@ -131,9 +131,9 @@ def _custom_test__impl(ctx): custom_test_ = rule( implementation = _custom_test__impl, attrs = { - "module_name": attr.string(mandatory = True), - "module_variant": attr.string(), - "module_deps": attr.label_list(providers = [SoongModuleInfo]), + "soong_module_name": attr.string(mandatory = True), + "soong_module_variant": attr.string(), + "soong_module_deps": attr.label_list(providers = [SoongModuleInfo]), "bool_prop": attr.bool(), "bool_ptr_prop": attr.bool(), "int64_ptr_prop": attr.int(), |