blob: 6942d57cae0de685e12275d0ad99fb0175e9a4cd [file] [log] [blame]
Liz Kammerea6666f2021-02-17 10:17:28 -05001// Copyright 2021 Google Inc. All rights reserved.
2//
3// Licensed under the Apache License, Version 2.0 (the "License");
4// you may not use this file except in compliance with the License.
5// You may obtain a copy of the License at
6//
7// http://www.apache.org/licenses/LICENSE-2.0
8//
9// Unless required by applicable law or agreed to in writing, software
10// distributed under the License is distributed on an "AS IS" BASIS,
11// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12// See the License for the specific language governing permissions and
13// limitations under the License.
14
15package android
16
Liz Kammerba3ea162021-02-17 13:22:03 -050017import (
18 "fmt"
19 "io/ioutil"
20 "path/filepath"
21 "strings"
22
Liz Kammerbdc60992021-02-24 16:55:11 -050023 "github.com/google/blueprint"
Liz Kammerba3ea162021-02-17 13:22:03 -050024 "github.com/google/blueprint/proptools"
25)
26
27type bazelModuleProperties struct {
28 // The label of the Bazel target replacing this Soong module. When run in conversion mode, this
29 // will import the handcrafted build target into the autogenerated file. Note: this may result in
30 // a conflict due to duplicate targets if bp2build_available is also set.
31 Label *string
32
33 // If true, bp2build will generate the converted Bazel target for this module. Note: this may
34 // cause a conflict due to the duplicate targets if label is also set.
Jingwen Chen12b4c272021-03-10 02:05:59 -050035 //
36 // This is a bool pointer to support tristates: true, false, not set.
37 //
38 // To opt-in a module, set bazel_module: { bp2build_available: true }
39 // To opt-out a module, set bazel_module: { bp2build_available: false }
40 // To defer the default setting for the directory, do not set the value.
41 Bp2build_available *bool
Liz Kammerba3ea162021-02-17 13:22:03 -050042}
43
44// Properties contains common module properties for Bazel migration purposes.
45type properties struct {
46 // In USE_BAZEL_ANALYSIS=1 mode, this represents the Bazel target replacing
47 // this Soong module.
48 Bazel_module bazelModuleProperties
49}
Liz Kammerea6666f2021-02-17 10:17:28 -050050
51// BazelModuleBase contains the property structs with metadata for modules which can be converted to
52// Bazel.
53type BazelModuleBase struct {
Liz Kammerba3ea162021-02-17 13:22:03 -050054 bazelProperties properties
Liz Kammerea6666f2021-02-17 10:17:28 -050055}
56
57// Bazelable is specifies the interface for modules that can be converted to Bazel.
58type Bazelable interface {
Liz Kammerba3ea162021-02-17 13:22:03 -050059 bazelProps() *properties
60 HasHandcraftedLabel() bool
Liz Kammerbdc60992021-02-24 16:55:11 -050061 HandcraftedLabel() string
62 GetBazelLabel(ctx BazelConversionPathContext, module blueprint.Module) string
Jingwen Chen12b4c272021-03-10 02:05:59 -050063 ConvertWithBp2build(ctx BazelConversionPathContext) bool
Liz Kammerba3ea162021-02-17 13:22:03 -050064 GetBazelBuildFileContents(c Config, path, name string) (string, error)
Jingwen Chen12b4c272021-03-10 02:05:59 -050065 ConvertedToBazel(ctx BazelConversionPathContext) bool
Liz Kammerea6666f2021-02-17 10:17:28 -050066}
67
68// BazelModule is a lightweight wrapper interface around Module for Bazel-convertible modules.
69type BazelModule interface {
70 Module
71 Bazelable
72}
73
74// InitBazelModule is a wrapper function that decorates a BazelModule with Bazel-conversion
75// properties.
76func InitBazelModule(module BazelModule) {
77 module.AddProperties(module.bazelProps())
78}
79
80// bazelProps returns the Bazel properties for the given BazelModuleBase.
Liz Kammerba3ea162021-02-17 13:22:03 -050081func (b *BazelModuleBase) bazelProps() *properties {
Liz Kammerea6666f2021-02-17 10:17:28 -050082 return &b.bazelProperties
83}
84
Liz Kammerba3ea162021-02-17 13:22:03 -050085// HasHandcraftedLabel returns whether this module has a handcrafted Bazel label.
86func (b *BazelModuleBase) HasHandcraftedLabel() bool {
87 return b.bazelProperties.Bazel_module.Label != nil
88}
89
90// HandcraftedLabel returns the handcrafted label for this module, or empty string if there is none
91func (b *BazelModuleBase) HandcraftedLabel() string {
92 return proptools.String(b.bazelProperties.Bazel_module.Label)
93}
94
Liz Kammerea6666f2021-02-17 10:17:28 -050095// GetBazelLabel returns the Bazel label for the given BazelModuleBase.
Liz Kammerbdc60992021-02-24 16:55:11 -050096func (b *BazelModuleBase) GetBazelLabel(ctx BazelConversionPathContext, module blueprint.Module) string {
97 if b.HasHandcraftedLabel() {
98 return b.HandcraftedLabel()
99 }
Jingwen Chen12b4c272021-03-10 02:05:59 -0500100 if b.ConvertWithBp2build(ctx) {
Liz Kammerbdc60992021-02-24 16:55:11 -0500101 return bp2buildModuleLabel(ctx, module)
102 }
103 return "" // no label for unconverted module
Liz Kammerea6666f2021-02-17 10:17:28 -0500104}
105
Jingwen Chen12b4c272021-03-10 02:05:59 -0500106// Configuration to decide if modules in a directory should default to true/false for bp2build_available
107type Bp2BuildConfig map[string]BazelConversionConfigEntry
108type BazelConversionConfigEntry int
109
110const (
Jingwen Chen91220d72021-03-24 02:18:33 -0400111 // A sentinel value to be used as a key in Bp2BuildConfig for modules with
112 // no package path. This is also the module dir for top level Android.bp
113 // modules.
114 BP2BUILD_TOPLEVEL = "."
115
Jingwen Chen12b4c272021-03-10 02:05:59 -0500116 // iota + 1 ensures that the int value is not 0 when used in the Bp2buildAllowlist map,
117 // which can also mean that the key doesn't exist in a lookup.
118
119 // all modules in this package and subpackages default to bp2build_available: true.
120 // allows modules to opt-out.
121 Bp2BuildDefaultTrueRecursively BazelConversionConfigEntry = iota + 1
122
123 // all modules in this package (not recursively) default to bp2build_available: false.
124 // allows modules to opt-in.
125 Bp2BuildDefaultFalse
126)
127
128var (
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400129 // Keep any existing BUILD files (and do not generate new BUILD files) for these directories
Jingwen Chenb643c7a2021-07-26 04:45:48 +0000130 // in the synthetic Bazel workspace.
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400131 bp2buildKeepExistingBuildFile = map[string]bool{
132 // This is actually build/bazel/build.BAZEL symlinked to ./BUILD
Jingwen Chenf59a8e12021-07-16 09:28:53 +0000133 ".":/*recursive = */ false,
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400134
Jingwen Chenb643c7a2021-07-26 04:45:48 +0000135 // build/bazel/examples/apex/... BUILD files should be generated, so
136 // build/bazel is not recursive. Instead list each subdirectory under
137 // build/bazel explicitly.
138 "build/bazel":/* recursive = */ false,
139 "build/bazel/examples/android_app":/* recursive = */ true,
Romain Jobredeaux748652e2021-08-06 21:51:33 +0000140 "build/bazel/examples/java":/* recursive = */ true,
Jingwen Chenb643c7a2021-07-26 04:45:48 +0000141 "build/bazel/bazel_skylib":/* recursive = */ true,
142 "build/bazel/rules":/* recursive = */ true,
143 "build/bazel/rules_cc":/* recursive = */ true,
144 "build/bazel/tests":/* recursive = */ true,
145 "build/bazel/platforms":/* recursive = */ true,
146 "build/bazel/product_variables":/* recursive = */ true,
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400147 "build/pesto":/* recursive = */ true,
148
149 // external/bazelbuild-rules_android/... is needed by mixed builds, otherwise mixed builds analysis fails
150 // e.g. ERROR: Analysis of target '@soong_injection//mixed_builds:buildroot' failed
151 "external/bazelbuild-rules_android":/* recursive = */ true,
152
Romain Jobredeaux748652e2021-08-06 21:51:33 +0000153 "prebuilts/jdk":/* recursive = */ true,
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400154 "prebuilts/sdk":/* recursive = */ false,
155 "prebuilts/sdk/tools":/* recursive = */ false,
Romain Jobredeaux8c98c252021-08-02 17:27:06 +0000156 "prebuilts/r8":/* recursive = */ false,
Damian Patel8b222192021-07-20 20:41:14 +0000157 "packages/apps/Music":/* recursive = */ false,
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400158 }
159
Jingwen Chen12b4c272021-03-10 02:05:59 -0500160 // Configure modules in these directories to enable bp2build_available: true or false by default.
161 bp2buildDefaultConfig = Bp2BuildConfig{
Jingwen Chenf59a8e12021-07-16 09:28:53 +0000162 "bionic": Bp2BuildDefaultTrueRecursively,
163 "build/bazel/examples/apex/minimal": Bp2BuildDefaultTrueRecursively,
164 "external/gwp_asan": Bp2BuildDefaultTrueRecursively,
165 "system/core/libcutils": Bp2BuildDefaultTrueRecursively,
Lukacs T. Berki497f17d2021-04-26 12:15:57 +0200166 "system/core/property_service/libpropertyinfoparser": Bp2BuildDefaultTrueRecursively,
Jingwen Chen28288162021-04-28 07:19:54 +0000167 "system/libbase": Bp2BuildDefaultTrueRecursively,
168 "system/logging/liblog": Bp2BuildDefaultTrueRecursively,
Jingwen Chen739d01e2021-07-28 15:26:20 +0000169 "system/timezone/apex": Bp2BuildDefaultTrueRecursively,
170 "system/timezone/output_data": Bp2BuildDefaultTrueRecursively,
Jingwen Chen28288162021-04-28 07:19:54 +0000171 "external/arm-optimized-routines": Bp2BuildDefaultTrueRecursively,
Chris Parsons2c788392021-08-10 11:58:07 -0400172 "external/fmtlib": Bp2BuildDefaultTrueRecursively,
173 "external/jemalloc_new": Bp2BuildDefaultTrueRecursively,
174 "external/libcxxabi": Bp2BuildDefaultTrueRecursively,
Jingwen Chen75be1ca2021-05-12 05:04:58 +0000175 "external/scudo": Bp2BuildDefaultTrueRecursively,
Jingwen Chen49109762021-05-25 05:16:48 +0000176 "prebuilts/clang/host/linux-x86": Bp2BuildDefaultTrueRecursively,
Jingwen Chen12b4c272021-03-10 02:05:59 -0500177 }
Jingwen Chen5d72cba2021-03-25 09:28:38 +0000178
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400179 // Per-module denylist to always opt modules out of both bp2build and mixed builds.
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000180 bp2buildModuleDoNotConvertList = []string{
Jingwen Chen28288162021-04-28 07:19:54 +0000181 // Things that transitively depend on unconverted libc_* modules.
Rupert Shuttleworth47aa5842021-04-30 04:04:15 -0400182 "libbionic_spawn_benchmark", // http://b/186824595, cc_library_static, depends on //external/google-benchmark (http://b/186822740)
183 // also depends on //system/logging/liblog:liblog (http://b/186822772)
184
185 "libc_malloc_debug", // http://b/186824339, cc_library_static, depends on //system/libbase:libbase (http://b/186823646)
186 "libc_malloc_debug_backtrace", // http://b/186824112, cc_library_static, depends on //external/libcxxabi:libc++demangle (http://b/186823773)
187
188 "libcutils", // http://b/186827426, cc_library, depends on //system/core/libprocessgroup:libprocessgroup_headers (http://b/186826841)
189 "libcutils_sockets", // http://b/186826853, cc_library, depends on //system/libbase:libbase (http://b/186826479)
190
191 "liblinker_debuggerd_stub", // http://b/186824327, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
192 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
193 // also depends on //system/logging/liblog:liblog (http://b/186822772)
194 "liblinker_main", // http://b/186825989, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
195 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
196 // also depends on//system/logging/liblog:liblog (http://b/186822772)
197 "liblinker_malloc", // http://b/186826466, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
198 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
199 // also depends on //system/logging/liblog:liblog (http://b/186822772)
Jingwen Chen53542922021-05-18 09:01:49 +0000200 "libc_ndk", // http://b/187013218, cc_library_static, depends on //bionic/libm:libm (http://b/183064661)
Jingwen Chen53542922021-05-18 09:01:49 +0000201 "libc_malloc_hooks", // http://b/187016307, cc_library, ld.lld: error: undefined symbol: __malloc_hook
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200202
Jingwen Chen790324e2021-04-30 08:20:01 +0000203 // http://b/186823769: Needs C++ STL support, includes from unconverted standard libraries in //external/libcxx
204 // c++_static
Rupert Shuttleworthfb955382021-05-03 04:32:36 -0400205 "libbase_ndk", // http://b/186826477, cc_library, no such target '//build/bazel/platforms/os:darwin' when --platforms //build/bazel/platforms:android_x86 is added
Jingwen Chen790324e2021-04-30 08:20:01 +0000206 // libcxx
207 "libBionicBenchmarksUtils", // cc_library_static, fatal error: 'map' file not found, from libcxx
208 "fmtlib", // cc_library_static, fatal error: 'cassert' file not found, from libcxx
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400209 "fmtlib_ndk", // cc_library_static, fatal error: 'cassert' file not found
Chris Parsons51f8c392021-08-03 21:01:05 -0400210 "liblog", // http://b/186822772: cc_library, 'sys/cdefs.h' file not found
Chris Parsons69fa9f92021-07-13 11:47:44 -0400211 "libbase", // Requires liblog. http://b/186826479, cc_library, fatal error: 'memory' file not found, from libcxx.
Chris Parsons51f8c392021-08-03 21:01:05 -0400212 // Also depends on fmtlib.
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200213
Chris Parsons51f8c392021-08-03 21:01:05 -0400214 "libseccomp_policy", // depends on libbase
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200215
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400216 "gwp_asan_crash_handler", // cc_library, ld.lld: error: undefined symbol: memset
217
Jingwen Chen790324e2021-04-30 08:20:01 +0000218 // Tests. Handle later.
219 "libbionic_tests_headers_posix", // http://b/186024507, cc_library_static, sched.h, time.h not found
220 "libjemalloc5_integrationtest",
221 "libjemalloc5_stresstestlib",
222 "libjemalloc5_unittest",
Rupert Shuttleworth6e4950a2021-07-27 01:34:59 -0400223
224 // APEX support
225 "com.android.runtime", // http://b/194746715, apex, depends on 'libc_malloc_debug' and 'libc_malloc_hooks'
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400226 }
Rupert Shuttleworthc143cc52021-04-13 13:08:04 -0400227
Jingwen Chen179856a2021-05-03 09:15:48 +0000228 // Per-module denylist of cc_library modules to only generate the static
229 // variant if their shared variant isn't ready or buildable by Bazel.
230 bp2buildCcLibraryStaticOnlyList = []string{
Jingwen Chen53542922021-05-18 09:01:49 +0000231 "libstdc++", // http://b/186822597, cc_library, ld.lld: error: undefined symbol: __errno
232 "libjemalloc5", // http://b/188503688, cc_library, `target: { android: { enabled: false } }` for android targets.
Jingwen Chen179856a2021-05-03 09:15:48 +0000233 }
234
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400235 // Per-module denylist to opt modules out of mixed builds. Such modules will
236 // still be generated via bp2build.
Chris Parsons2c788392021-08-10 11:58:07 -0400237 mixedBuildsDisabledList = []string{
238 "libc++abi", // http://b/195970501, cc_library_static, duplicate symbols because it propagates libc objects.
239 "libc++demangle", // http://b/195970501, cc_library_static, duplicate symbols because it propagates libc objects.
240 }
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000241
242 // Used for quicker lookups
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400243 bp2buildModuleDoNotConvert = map[string]bool{}
Jingwen Chen179856a2021-05-03 09:15:48 +0000244 bp2buildCcLibraryStaticOnly = map[string]bool{}
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400245 mixedBuildsDisabled = map[string]bool{}
Jingwen Chen12b4c272021-03-10 02:05:59 -0500246)
247
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000248func init() {
249 for _, moduleName := range bp2buildModuleDoNotConvertList {
250 bp2buildModuleDoNotConvert[moduleName] = true
251 }
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400252
Jingwen Chen179856a2021-05-03 09:15:48 +0000253 for _, moduleName := range bp2buildCcLibraryStaticOnlyList {
254 bp2buildCcLibraryStaticOnly[moduleName] = true
255 }
256
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400257 for _, moduleName := range mixedBuildsDisabledList {
258 mixedBuildsDisabled[moduleName] = true
259 }
260}
261
Jingwen Chen179856a2021-05-03 09:15:48 +0000262func GenerateCcLibraryStaticOnly(ctx BazelConversionPathContext) bool {
263 return bp2buildCcLibraryStaticOnly[ctx.Module().Name()]
264}
265
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400266func ShouldKeepExistingBuildFileForDir(dir string) bool {
267 if _, ok := bp2buildKeepExistingBuildFile[dir]; ok {
268 // Exact dir match
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400269 return true
270 }
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400271 // Check if subtree match
272 for prefix, recursive := range bp2buildKeepExistingBuildFile {
273 if recursive {
274 if strings.HasPrefix(dir, prefix+"/") {
275 return true
276 }
277 }
278 }
279 // Default
280 return false
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400281}
282
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400283// MixedBuildsEnabled checks that a module is ready to be replaced by a
284// converted or handcrafted Bazel target.
285func (b *BazelModuleBase) MixedBuildsEnabled(ctx BazelConversionPathContext) bool {
286 if !ctx.Config().BazelContext.BazelEnabled() {
287 return false
288 }
289 if len(b.GetBazelLabel(ctx, ctx.Module())) == 0 {
290 return false
291 }
Jingwen Chen179856a2021-05-03 09:15:48 +0000292 if GenerateCcLibraryStaticOnly(ctx) {
293 // Don't use partially-converted cc_library targets in mixed builds,
294 // since mixed builds would generally rely on both static and shared
295 // variants of a cc_library.
296 return false
297 }
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400298 return !mixedBuildsDisabled[ctx.Module().Name()]
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000299}
300
Liz Kammerea6666f2021-02-17 10:17:28 -0500301// ConvertWithBp2build returns whether the given BazelModuleBase should be converted with bp2build.
Jingwen Chen12b4c272021-03-10 02:05:59 -0500302func (b *BazelModuleBase) ConvertWithBp2build(ctx BazelConversionPathContext) bool {
Jingwen Chen5d72cba2021-03-25 09:28:38 +0000303 if bp2buildModuleDoNotConvert[ctx.Module().Name()] {
304 return false
305 }
306
Jingwen Chen12b4c272021-03-10 02:05:59 -0500307 // Ensure that the module type of this module has a bp2build converter. This
308 // prevents mixed builds from using auto-converted modules just by matching
309 // the package dir; it also has to have a bp2build mutator as well.
310 if ctx.Config().bp2buildModuleTypeConfig[ctx.ModuleType()] == false {
311 return false
312 }
313
314 packagePath := ctx.ModuleDir()
315 config := ctx.Config().bp2buildPackageConfig
316
317 // This is a tristate value: true, false, or unset.
318 propValue := b.bazelProperties.Bazel_module.Bp2build_available
319 if bp2buildDefaultTrueRecursively(packagePath, config) {
320 // Allow modules to explicitly opt-out.
321 return proptools.BoolDefault(propValue, true)
322 }
323
324 // Allow modules to explicitly opt-in.
325 return proptools.BoolDefault(propValue, false)
326}
327
328// bp2buildDefaultTrueRecursively checks that the package contains a prefix from the
329// set of package prefixes where all modules must be converted. That is, if the
330// package is x/y/z, and the list contains either x, x/y, or x/y/z, this function will
331// return true.
332//
333// However, if the package is x/y, and it matches a Bp2BuildDefaultFalse "x/y" entry
334// exactly, this module will return false early.
335//
336// This function will also return false if the package doesn't match anything in
337// the config.
338func bp2buildDefaultTrueRecursively(packagePath string, config Bp2BuildConfig) bool {
339 ret := false
340
Jingwen Chen91220d72021-03-24 02:18:33 -0400341 // Return exact matches in the config.
342 if config[packagePath] == Bp2BuildDefaultTrueRecursively {
343 return true
344 }
Jingwen Chen12b4c272021-03-10 02:05:59 -0500345 if config[packagePath] == Bp2BuildDefaultFalse {
346 return false
347 }
348
Jingwen Chen91220d72021-03-24 02:18:33 -0400349 // If not, check for the config recursively.
Jingwen Chen12b4c272021-03-10 02:05:59 -0500350 packagePrefix := ""
351 // e.g. for x/y/z, iterate over x, x/y, then x/y/z, taking the final value from the allowlist.
352 for _, part := range strings.Split(packagePath, "/") {
353 packagePrefix += part
354 if config[packagePrefix] == Bp2BuildDefaultTrueRecursively {
355 // package contains this prefix and this prefix should convert all modules
356 return true
357 }
358 // Continue to the next part of the package dir.
359 packagePrefix += "/"
360 }
361
362 return ret
Liz Kammerea6666f2021-02-17 10:17:28 -0500363}
Liz Kammerba3ea162021-02-17 13:22:03 -0500364
365// GetBazelBuildFileContents returns the file contents of a hand-crafted BUILD file if available or
366// an error if there are errors reading the file.
367// TODO(b/181575318): currently we append the whole BUILD file, let's change that to do
368// something more targeted based on the rule type and target.
369func (b *BazelModuleBase) GetBazelBuildFileContents(c Config, path, name string) (string, error) {
Liz Kammerbdc60992021-02-24 16:55:11 -0500370 if !strings.Contains(b.HandcraftedLabel(), path) {
371 return "", fmt.Errorf("%q not found in bazel_module.label %q", path, b.HandcraftedLabel())
Liz Kammerba3ea162021-02-17 13:22:03 -0500372 }
373 name = filepath.Join(path, name)
374 f, err := c.fs.Open(name)
375 if err != nil {
376 return "", err
377 }
378 defer f.Close()
379
380 data, err := ioutil.ReadAll(f)
381 if err != nil {
382 return "", err
383 }
384 return string(data[:]), nil
385}
Liz Kammerbdc60992021-02-24 16:55:11 -0500386
387// ConvertedToBazel returns whether this module has been converted to Bazel, whether automatically
388// or manually
Jingwen Chen12b4c272021-03-10 02:05:59 -0500389func (b *BazelModuleBase) ConvertedToBazel(ctx BazelConversionPathContext) bool {
390 return b.ConvertWithBp2build(ctx) || b.HasHandcraftedLabel()
Liz Kammerbdc60992021-02-24 16:55:11 -0500391}