blob: 347f5c05ad080220c8fb14deee5a861565618471 [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 Jobredeaux5cc9c9d2021-08-26 15:07:48 +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,
Jingwen Chen5e49b822021-08-24 05:14:22 +0000147 "build/bazel_common_rules":/* recursive = */ true,
Romain Jobredeaux9e09bba2021-09-08 18:09:02 +0000148 "build/make/tools":/* recursive = */ true,
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400149 "build/pesto":/* recursive = */ true,
150
151 // external/bazelbuild-rules_android/... is needed by mixed builds, otherwise mixed builds analysis fails
152 // e.g. ERROR: Analysis of target '@soong_injection//mixed_builds:buildroot' failed
153 "external/bazelbuild-rules_android":/* recursive = */ true,
Jingwen Chen91632252021-08-10 13:00:33 +0000154 "external/bazel-skylib":/* recursive = */ true,
Romain Jobredeaux9e09bba2021-09-08 18:09:02 +0000155 "external/guava":/* recursive = */ true,
156 "external/error_prone":/* recursive = */ true,
157 "external/jsr305":/* recursive = */ true,
158 "frameworks/ex/common":/* recursive = */ true,
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400159
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400160 "prebuilts/sdk":/* recursive = */ false,
161 "prebuilts/sdk/tools":/* recursive = */ false,
Romain Jobredeaux8c98c252021-08-02 17:27:06 +0000162 "prebuilts/r8":/* recursive = */ false,
Romain Jobredeauxf1b0ac82021-08-12 14:39:00 +0000163 "packages/apps/Music":/* recursive = */ true,
Romain Jobredeaux9e09bba2021-09-08 18:09:02 +0000164 "packages/apps/QuickSearchBox":/* recursive = */ true,
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400165 }
166
Jingwen Chen12b4c272021-03-10 02:05:59 -0500167 // Configure modules in these directories to enable bp2build_available: true or false by default.
168 bp2buildDefaultConfig = Bp2BuildConfig{
Jingwen Chenf59a8e12021-07-16 09:28:53 +0000169 "bionic": Bp2BuildDefaultTrueRecursively,
170 "build/bazel/examples/apex/minimal": Bp2BuildDefaultTrueRecursively,
171 "external/gwp_asan": Bp2BuildDefaultTrueRecursively,
172 "system/core/libcutils": Bp2BuildDefaultTrueRecursively,
Lukacs T. Berki497f17d2021-04-26 12:15:57 +0200173 "system/core/property_service/libpropertyinfoparser": Bp2BuildDefaultTrueRecursively,
Jingwen Chen28288162021-04-28 07:19:54 +0000174 "system/libbase": Bp2BuildDefaultTrueRecursively,
175 "system/logging/liblog": Bp2BuildDefaultTrueRecursively,
Jingwen Chen739d01e2021-07-28 15:26:20 +0000176 "system/timezone/apex": Bp2BuildDefaultTrueRecursively,
177 "system/timezone/output_data": Bp2BuildDefaultTrueRecursively,
Jingwen Chen28288162021-04-28 07:19:54 +0000178 "external/arm-optimized-routines": Bp2BuildDefaultTrueRecursively,
Chris Parsons2c788392021-08-10 11:58:07 -0400179 "external/fmtlib": Bp2BuildDefaultTrueRecursively,
180 "external/jemalloc_new": Bp2BuildDefaultTrueRecursively,
181 "external/libcxxabi": Bp2BuildDefaultTrueRecursively,
Jingwen Chen75be1ca2021-05-12 05:04:58 +0000182 "external/scudo": Bp2BuildDefaultTrueRecursively,
Jingwen Chen49109762021-05-25 05:16:48 +0000183 "prebuilts/clang/host/linux-x86": Bp2BuildDefaultTrueRecursively,
Jingwen Chen12b4c272021-03-10 02:05:59 -0500184 }
Jingwen Chen5d72cba2021-03-25 09:28:38 +0000185
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400186 // Per-module denylist to always opt modules out of both bp2build and mixed builds.
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000187 bp2buildModuleDoNotConvertList = []string{
Jingwen Chen28288162021-04-28 07:19:54 +0000188 // Things that transitively depend on unconverted libc_* modules.
Rupert Shuttleworth47aa5842021-04-30 04:04:15 -0400189 "libbionic_spawn_benchmark", // http://b/186824595, cc_library_static, depends on //external/google-benchmark (http://b/186822740)
190 // also depends on //system/logging/liblog:liblog (http://b/186822772)
191
192 "libc_malloc_debug", // http://b/186824339, cc_library_static, depends on //system/libbase:libbase (http://b/186823646)
193 "libc_malloc_debug_backtrace", // http://b/186824112, cc_library_static, depends on //external/libcxxabi:libc++demangle (http://b/186823773)
194
195 "libcutils", // http://b/186827426, cc_library, depends on //system/core/libprocessgroup:libprocessgroup_headers (http://b/186826841)
196 "libcutils_sockets", // http://b/186826853, cc_library, depends on //system/libbase:libbase (http://b/186826479)
197
198 "liblinker_debuggerd_stub", // http://b/186824327, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
199 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
200 // also depends on //system/logging/liblog:liblog (http://b/186822772)
201 "liblinker_main", // http://b/186825989, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
202 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
203 // also depends on//system/logging/liblog:liblog (http://b/186822772)
204 "liblinker_malloc", // http://b/186826466, cc_library_static, depends on //external/zlib:libz (http://b/186823782)
205 // also depends on //system/libziparchive:libziparchive (http://b/186823656)
206 // also depends on //system/logging/liblog:liblog (http://b/186822772)
Jingwen Chen53542922021-05-18 09:01:49 +0000207 "libc_ndk", // http://b/187013218, cc_library_static, depends on //bionic/libm:libm (http://b/183064661)
Jingwen Chen53542922021-05-18 09:01:49 +0000208 "libc_malloc_hooks", // http://b/187016307, cc_library, ld.lld: error: undefined symbol: __malloc_hook
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200209
Jingwen Chen790324e2021-04-30 08:20:01 +0000210 // http://b/186823769: Needs C++ STL support, includes from unconverted standard libraries in //external/libcxx
211 // c++_static
Rupert Shuttleworthfb955382021-05-03 04:32:36 -0400212 "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 +0000213 // libcxx
214 "libBionicBenchmarksUtils", // cc_library_static, fatal error: 'map' file not found, from libcxx
215 "fmtlib", // cc_library_static, fatal error: 'cassert' file not found, from libcxx
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400216 "fmtlib_ndk", // cc_library_static, fatal error: 'cassert' file not found
Chris Parsons51f8c392021-08-03 21:01:05 -0400217 "liblog", // http://b/186822772: cc_library, 'sys/cdefs.h' file not found
Chris Parsons69fa9f92021-07-13 11:47:44 -0400218 "libbase", // Requires liblog. http://b/186826479, cc_library, fatal error: 'memory' file not found, from libcxx.
Chris Parsons51f8c392021-08-03 21:01:05 -0400219 // Also depends on fmtlib.
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200220
Chris Parsons51f8c392021-08-03 21:01:05 -0400221 "libseccomp_policy", // depends on libbase
Lukacs T. Berkib5ac5af2021-04-27 11:02:11 +0200222
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400223 "gwp_asan_crash_handler", // cc_library, ld.lld: error: undefined symbol: memset
224
Jingwen Chen790324e2021-04-30 08:20:01 +0000225 // Tests. Handle later.
226 "libbionic_tests_headers_posix", // http://b/186024507, cc_library_static, sched.h, time.h not found
227 "libjemalloc5_integrationtest",
228 "libjemalloc5_stresstestlib",
229 "libjemalloc5_unittest",
Rupert Shuttleworth6e4950a2021-07-27 01:34:59 -0400230
231 // APEX support
232 "com.android.runtime", // http://b/194746715, apex, depends on 'libc_malloc_debug' and 'libc_malloc_hooks'
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400233 }
Rupert Shuttleworthc143cc52021-04-13 13:08:04 -0400234
Jingwen Chen179856a2021-05-03 09:15:48 +0000235 // Per-module denylist of cc_library modules to only generate the static
236 // variant if their shared variant isn't ready or buildable by Bazel.
237 bp2buildCcLibraryStaticOnlyList = []string{
Jingwen Chen53542922021-05-18 09:01:49 +0000238 "libstdc++", // http://b/186822597, cc_library, ld.lld: error: undefined symbol: __errno
239 "libjemalloc5", // http://b/188503688, cc_library, `target: { android: { enabled: false } }` for android targets.
Jingwen Chen179856a2021-05-03 09:15:48 +0000240 }
241
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400242 // Per-module denylist to opt modules out of mixed builds. Such modules will
243 // still be generated via bp2build.
Chris Parsons2c788392021-08-10 11:58:07 -0400244 mixedBuildsDisabledList = []string{
245 "libc++abi", // http://b/195970501, cc_library_static, duplicate symbols because it propagates libc objects.
246 "libc++demangle", // http://b/195970501, cc_library_static, duplicate symbols because it propagates libc objects.
247 }
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000248
249 // Used for quicker lookups
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400250 bp2buildModuleDoNotConvert = map[string]bool{}
Jingwen Chen179856a2021-05-03 09:15:48 +0000251 bp2buildCcLibraryStaticOnly = map[string]bool{}
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400252 mixedBuildsDisabled = map[string]bool{}
Jingwen Chen12b4c272021-03-10 02:05:59 -0500253)
254
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000255func init() {
256 for _, moduleName := range bp2buildModuleDoNotConvertList {
257 bp2buildModuleDoNotConvert[moduleName] = true
258 }
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400259
Jingwen Chen179856a2021-05-03 09:15:48 +0000260 for _, moduleName := range bp2buildCcLibraryStaticOnlyList {
261 bp2buildCcLibraryStaticOnly[moduleName] = true
262 }
263
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400264 for _, moduleName := range mixedBuildsDisabledList {
265 mixedBuildsDisabled[moduleName] = true
266 }
267}
268
Jingwen Chen179856a2021-05-03 09:15:48 +0000269func GenerateCcLibraryStaticOnly(ctx BazelConversionPathContext) bool {
270 return bp2buildCcLibraryStaticOnly[ctx.Module().Name()]
271}
272
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400273func ShouldKeepExistingBuildFileForDir(dir string) bool {
274 if _, ok := bp2buildKeepExistingBuildFile[dir]; ok {
275 // Exact dir match
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400276 return true
277 }
Rupert Shuttleworth00960792021-05-12 21:20:13 -0400278 // Check if subtree match
279 for prefix, recursive := range bp2buildKeepExistingBuildFile {
280 if recursive {
281 if strings.HasPrefix(dir, prefix+"/") {
282 return true
283 }
284 }
285 }
286 // Default
287 return false
Rupert Shuttleworth2a4fc3e2021-04-21 07:10:09 -0400288}
289
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400290// MixedBuildsEnabled checks that a module is ready to be replaced by a
291// converted or handcrafted Bazel target.
292func (b *BazelModuleBase) MixedBuildsEnabled(ctx BazelConversionPathContext) bool {
293 if !ctx.Config().BazelContext.BazelEnabled() {
294 return false
295 }
296 if len(b.GetBazelLabel(ctx, ctx.Module())) == 0 {
297 return false
298 }
Jingwen Chen179856a2021-05-03 09:15:48 +0000299 if GenerateCcLibraryStaticOnly(ctx) {
300 // Don't use partially-converted cc_library targets in mixed builds,
301 // since mixed builds would generally rely on both static and shared
302 // variants of a cc_library.
303 return false
304 }
Chris Parsonsbab4d7e2021-04-15 17:27:08 -0400305 return !mixedBuildsDisabled[ctx.Module().Name()]
Rupert Shuttleworth4f43fe92021-03-30 14:13:16 +0000306}
307
Liz Kammerea6666f2021-02-17 10:17:28 -0500308// ConvertWithBp2build returns whether the given BazelModuleBase should be converted with bp2build.
Jingwen Chen12b4c272021-03-10 02:05:59 -0500309func (b *BazelModuleBase) ConvertWithBp2build(ctx BazelConversionPathContext) bool {
Jingwen Chen5d72cba2021-03-25 09:28:38 +0000310 if bp2buildModuleDoNotConvert[ctx.Module().Name()] {
311 return false
312 }
313
Jingwen Chen12b4c272021-03-10 02:05:59 -0500314 // Ensure that the module type of this module has a bp2build converter. This
315 // prevents mixed builds from using auto-converted modules just by matching
316 // the package dir; it also has to have a bp2build mutator as well.
317 if ctx.Config().bp2buildModuleTypeConfig[ctx.ModuleType()] == false {
318 return false
319 }
320
321 packagePath := ctx.ModuleDir()
322 config := ctx.Config().bp2buildPackageConfig
323
324 // This is a tristate value: true, false, or unset.
325 propValue := b.bazelProperties.Bazel_module.Bp2build_available
326 if bp2buildDefaultTrueRecursively(packagePath, config) {
327 // Allow modules to explicitly opt-out.
328 return proptools.BoolDefault(propValue, true)
329 }
330
331 // Allow modules to explicitly opt-in.
332 return proptools.BoolDefault(propValue, false)
333}
334
335// bp2buildDefaultTrueRecursively checks that the package contains a prefix from the
336// set of package prefixes where all modules must be converted. That is, if the
337// package is x/y/z, and the list contains either x, x/y, or x/y/z, this function will
338// return true.
339//
340// However, if the package is x/y, and it matches a Bp2BuildDefaultFalse "x/y" entry
341// exactly, this module will return false early.
342//
343// This function will also return false if the package doesn't match anything in
344// the config.
345func bp2buildDefaultTrueRecursively(packagePath string, config Bp2BuildConfig) bool {
346 ret := false
347
Jingwen Chen91220d72021-03-24 02:18:33 -0400348 // Return exact matches in the config.
349 if config[packagePath] == Bp2BuildDefaultTrueRecursively {
350 return true
351 }
Jingwen Chen12b4c272021-03-10 02:05:59 -0500352 if config[packagePath] == Bp2BuildDefaultFalse {
353 return false
354 }
355
Jingwen Chen91220d72021-03-24 02:18:33 -0400356 // If not, check for the config recursively.
Jingwen Chen12b4c272021-03-10 02:05:59 -0500357 packagePrefix := ""
358 // e.g. for x/y/z, iterate over x, x/y, then x/y/z, taking the final value from the allowlist.
359 for _, part := range strings.Split(packagePath, "/") {
360 packagePrefix += part
361 if config[packagePrefix] == Bp2BuildDefaultTrueRecursively {
362 // package contains this prefix and this prefix should convert all modules
363 return true
364 }
365 // Continue to the next part of the package dir.
366 packagePrefix += "/"
367 }
368
369 return ret
Liz Kammerea6666f2021-02-17 10:17:28 -0500370}
Liz Kammerba3ea162021-02-17 13:22:03 -0500371
372// GetBazelBuildFileContents returns the file contents of a hand-crafted BUILD file if available or
373// an error if there are errors reading the file.
374// TODO(b/181575318): currently we append the whole BUILD file, let's change that to do
375// something more targeted based on the rule type and target.
376func (b *BazelModuleBase) GetBazelBuildFileContents(c Config, path, name string) (string, error) {
Liz Kammerbdc60992021-02-24 16:55:11 -0500377 if !strings.Contains(b.HandcraftedLabel(), path) {
378 return "", fmt.Errorf("%q not found in bazel_module.label %q", path, b.HandcraftedLabel())
Liz Kammerba3ea162021-02-17 13:22:03 -0500379 }
380 name = filepath.Join(path, name)
381 f, err := c.fs.Open(name)
382 if err != nil {
383 return "", err
384 }
385 defer f.Close()
386
387 data, err := ioutil.ReadAll(f)
388 if err != nil {
389 return "", err
390 }
391 return string(data[:]), nil
392}
Liz Kammerbdc60992021-02-24 16:55:11 -0500393
394// ConvertedToBazel returns whether this module has been converted to Bazel, whether automatically
395// or manually
Jingwen Chen12b4c272021-03-10 02:05:59 -0500396func (b *BazelModuleBase) ConvertedToBazel(ctx BazelConversionPathContext) bool {
397 return b.ConvertWithBp2build(ctx) || b.HasHandcraftedLabel()
Liz Kammerbdc60992021-02-24 16:55:11 -0500398}