Merge "use CommonAttributes for Tags in protos filegroups"
This commit is contained in:
@@ -122,16 +122,18 @@ func (fg *fileGroup) ConvertWithBp2build(ctx TopDownMutatorContext) {
|
|||||||
if fg.ShouldConvertToProtoLibrary(ctx) {
|
if fg.ShouldConvertToProtoLibrary(ctx) {
|
||||||
// TODO(b/246997908): we can remove this tag if we could figure out a
|
// TODO(b/246997908): we can remove this tag if we could figure out a
|
||||||
// solution for this bug.
|
// solution for this bug.
|
||||||
tags := []string{"manual"}
|
|
||||||
attrs := &ProtoAttrs{
|
attrs := &ProtoAttrs{
|
||||||
Srcs: srcs,
|
Srcs: srcs,
|
||||||
Strip_import_prefix: fg.properties.Path,
|
Strip_import_prefix: fg.properties.Path,
|
||||||
Tags: tags,
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
tags := []string{"manual"}
|
||||||
ctx.CreateBazelTargetModule(
|
ctx.CreateBazelTargetModule(
|
||||||
bazel.BazelTargetModuleProperties{Rule_class: "proto_library"},
|
bazel.BazelTargetModuleProperties{Rule_class: "proto_library"},
|
||||||
CommonAttributes{Name: fg.Name() + convertedProtoLibrarySuffix},
|
CommonAttributes{
|
||||||
|
Name: fg.Name() + convertedProtoLibrarySuffix,
|
||||||
|
Tags: bazel.MakeStringListAttribute(tags),
|
||||||
|
},
|
||||||
attrs)
|
attrs)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@@ -164,7 +164,6 @@ type ProtoAttrs struct {
|
|||||||
Srcs bazel.LabelListAttribute
|
Srcs bazel.LabelListAttribute
|
||||||
Strip_import_prefix *string
|
Strip_import_prefix *string
|
||||||
Deps bazel.LabelListAttribute
|
Deps bazel.LabelListAttribute
|
||||||
Tags []string
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// For each package in the include_dirs property a proto_library target should
|
// For each package in the include_dirs property a proto_library target should
|
||||||
|
Reference in New Issue
Block a user