Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 1 | // Copyright 2015 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 | |
| 15 | package main |
| 16 | |
| 17 | import ( |
Usta Shrestha | 2ba28a3 | 2022-10-24 11:33:09 -0400 | [diff] [blame] | 18 | "bytes" |
Jeongik Cha | a87506f | 2023-06-01 23:16:41 +0900 | [diff] [blame] | 19 | "errors" |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 20 | "flag" |
| 21 | "fmt" |
| 22 | "os" |
| 23 | "path/filepath" |
Jingwen Chen | eb76c43 | 2021-01-28 08:22:12 -0500 | [diff] [blame] | 24 | "strings" |
Lukacs T. Berki | c99c947 | 2021-03-24 10:50:06 +0100 | [diff] [blame] | 25 | "time" |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 26 | |
Dan Willemsen | 66213a6 | 2021-09-21 17:50:30 -0700 | [diff] [blame] | 27 | "android/soong/android" |
Jeongik Cha | e114e60 | 2023-03-19 00:12:39 +0900 | [diff] [blame] | 28 | "android/soong/android/allowlists" |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 29 | "android/soong/bp2build" |
Lukacs T. Berki | 7690c09 | 2021-02-26 14:27:36 +0100 | [diff] [blame] | 30 | "android/soong/shared" |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 31 | "android/soong/ui/metrics/bp2build_metrics_proto" |
Lukacs T. Berki | e571dc3 | 2021-08-25 14:14:13 +0200 | [diff] [blame] | 32 | |
Jeongik Cha | b745e2e | 2023-04-11 14:28:43 +0900 | [diff] [blame] | 33 | "github.com/google/blueprint" |
Colin Cross | 70b4059 | 2015-03-23 12:57:34 -0700 | [diff] [blame] | 34 | "github.com/google/blueprint/bootstrap" |
Lukacs T. Berki | d518e1a | 2021-04-14 13:49:50 +0200 | [diff] [blame] | 35 | "github.com/google/blueprint/deptools" |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 36 | "github.com/google/blueprint/metrics" |
Dan Willemsen | 66213a6 | 2021-09-21 17:50:30 -0700 | [diff] [blame] | 37 | androidProtobuf "google.golang.org/protobuf/android" |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 38 | ) |
| 39 | |
Colin Cross | e87040b | 2017-12-11 15:52:26 -0800 | [diff] [blame] | 40 | var ( |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 41 | topDir string |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 42 | availableEnvFile string |
| 43 | usedEnvFile string |
| 44 | |
Lukacs T. Berki | 809d2ed | 2021-08-18 10:55:32 +0200 | [diff] [blame] | 45 | globFile string |
| 46 | globListDir string |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 47 | delveListen string |
| 48 | delvePath string |
| 49 | |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 50 | cmdlineArgs android.CmdArgs |
Colin Cross | e87040b | 2017-12-11 15:52:26 -0800 | [diff] [blame] | 51 | ) |
| 52 | |
| 53 | func init() { |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 54 | // Flags that make sense in every mode |
Lukacs T. Berki | 7690c09 | 2021-02-26 14:27:36 +0100 | [diff] [blame] | 55 | flag.StringVar(&topDir, "top", "", "Top directory of the Android source tree") |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 56 | flag.StringVar(&cmdlineArgs.SoongOutDir, "soong_out", "", "Soong output directory (usually $TOP/out/soong)") |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 57 | flag.StringVar(&availableEnvFile, "available_env", "", "File containing available environment variables") |
| 58 | flag.StringVar(&usedEnvFile, "used_env", "", "File containing used environment variables") |
Lukacs T. Berki | b078ade | 2021-08-31 10:42:08 +0200 | [diff] [blame] | 59 | flag.StringVar(&globFile, "globFile", "build-globs.ninja", "the Ninja file of globs to output") |
| 60 | flag.StringVar(&globListDir, "globListDir", "", "the directory containing the glob list files") |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 61 | flag.StringVar(&cmdlineArgs.OutDir, "out", "", "the ninja builddir directory") |
Lukacs T. Berki | b078ade | 2021-08-31 10:42:08 +0200 | [diff] [blame] | 62 | flag.StringVar(&cmdlineArgs.ModuleListFile, "l", "", "file that lists filepaths to parse") |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 63 | |
| 64 | // Debug flags |
Lukacs T. Berki | 7d613bf | 2021-03-02 10:09:41 +0100 | [diff] [blame] | 65 | flag.StringVar(&delveListen, "delve_listen", "", "Delve port to listen on for debugging") |
| 66 | flag.StringVar(&delvePath, "delve_path", "", "Path to Delve. Only used if --delve_listen is set") |
Lukacs T. Berki | b078ade | 2021-08-31 10:42:08 +0200 | [diff] [blame] | 67 | flag.StringVar(&cmdlineArgs.Cpuprofile, "cpuprofile", "", "write cpu profile to file") |
| 68 | flag.StringVar(&cmdlineArgs.TraceFile, "trace", "", "write trace to file") |
| 69 | flag.StringVar(&cmdlineArgs.Memprofile, "memprofile", "", "write memory profile to file") |
| 70 | flag.BoolVar(&cmdlineArgs.NoGC, "nogc", false, "turn off GC for debugging") |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 71 | |
| 72 | // Flags representing various modes soong_build can run in |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 73 | flag.StringVar(&cmdlineArgs.ModuleGraphFile, "module_graph_file", "", "JSON module graph file to output") |
| 74 | flag.StringVar(&cmdlineArgs.ModuleActionsFile, "module_actions_file", "", "JSON file to output inputs/outputs of actions of modules") |
| 75 | flag.StringVar(&cmdlineArgs.DocFile, "soong_docs", "", "build documentation file to output") |
| 76 | flag.StringVar(&cmdlineArgs.BazelQueryViewDir, "bazel_queryview_dir", "", "path to the bazel queryview directory relative to --top") |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 77 | flag.StringVar(&cmdlineArgs.Bp2buildMarker, "bp2build_marker", "", "If set, run bp2build, touch the specified marker file then exit") |
| 78 | flag.StringVar(&cmdlineArgs.SymlinkForestMarker, "symlink_forest_marker", "", "If set, create the bp2build symlink forest, touch the specified marker file, then exit") |
Lukacs T. Berki | f900807 | 2021-08-16 15:24:48 +0200 | [diff] [blame] | 79 | flag.StringVar(&cmdlineArgs.OutFile, "o", "build.ninja", "the Ninja file to output") |
Kiyoung Kim | a37d9ba | 2023-04-19 13:13:45 +0900 | [diff] [blame] | 80 | flag.StringVar(&cmdlineArgs.SoongVariables, "soong_variables", "soong.variables", "the file contains all build variables") |
MarkDacek | d06db5d | 2022-11-29 00:47:59 +0000 | [diff] [blame] | 81 | flag.StringVar(&cmdlineArgs.BazelForceEnabledModules, "bazel-force-enabled-modules", "", "additional modules to build with Bazel. Comma-delimited") |
Lukacs T. Berki | b078ade | 2021-08-31 10:42:08 +0200 | [diff] [blame] | 82 | flag.BoolVar(&cmdlineArgs.EmptyNinjaFile, "empty-ninja-file", false, "write out a 0-byte ninja file") |
LaMont Jones | 52a7243 | 2023-03-09 18:19:35 +0000 | [diff] [blame] | 83 | flag.BoolVar(&cmdlineArgs.MultitreeBuild, "multitree-build", false, "this is a multitree build") |
Chris Parsons | ef615e5 | 2022-08-18 22:04:11 -0400 | [diff] [blame] | 84 | flag.BoolVar(&cmdlineArgs.BazelMode, "bazel-mode", false, "use bazel for analysis of certain modules") |
Jingwen Chen | e647db8 | 2022-11-01 11:28:29 +0000 | [diff] [blame] | 85 | flag.BoolVar(&cmdlineArgs.BazelModeStaging, "bazel-mode-staging", false, "use bazel for analysis of certain near-ready modules") |
Chris Parsons | 9402ca8 | 2023-02-23 17:28:06 -0500 | [diff] [blame] | 86 | flag.BoolVar(&cmdlineArgs.UseBazelProxy, "use-bazel-proxy", false, "communicate with bazel using unix socket proxy instead of spawning subprocesses") |
Sebastian Pickl | 1c4188c | 2023-10-24 11:18:34 +0000 | [diff] [blame] | 87 | flag.BoolVar(&cmdlineArgs.BuildFromTextStub, "build-from-text-stub", false, "build Java stubs from API text files instead of source files") |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 88 | flag.BoolVar(&cmdlineArgs.EnsureAllowlistIntegrity, "ensure-allowlist-integrity", false, "verify that allowlisted modules are mixed-built") |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 89 | // Flags that probably shouldn't be flags of soong_build, but we haven't found |
Lukacs T. Berki | b078ade | 2021-08-31 10:42:08 +0200 | [diff] [blame] | 90 | // the time to remove them yet |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 91 | flag.BoolVar(&cmdlineArgs.RunGoTests, "t", false, "build and run go tests during bootstrap") |
Dan Willemsen | 66213a6 | 2021-09-21 17:50:30 -0700 | [diff] [blame] | 92 | |
| 93 | // Disable deterministic randomization in the protobuf package, so incremental |
| 94 | // builds with unrelated Soong changes don't trigger large rebuilds (since we |
| 95 | // write out text protos in command lines, and command line changes trigger |
| 96 | // rebuilds). |
| 97 | androidProtobuf.DisableRand() |
Colin Cross | e87040b | 2017-12-11 15:52:26 -0800 | [diff] [blame] | 98 | } |
| 99 | |
Jeff Gaston | 088e29e | 2017-11-29 16:47:17 -0800 | [diff] [blame] | 100 | func newNameResolver(config android.Config) *android.NameResolver { |
Paul Duffin | 3f7bf9f | 2022-11-08 12:21:15 +0000 | [diff] [blame] | 101 | return android.NewNameResolver(config) |
Jeff Gaston | 088e29e | 2017-11-29 16:47:17 -0800 | [diff] [blame] | 102 | } |
| 103 | |
Lukacs T. Berki | ffc9e8d | 2021-09-07 17:54:38 +0200 | [diff] [blame] | 104 | func newContext(configuration android.Config) *android.Context { |
Colin Cross | ae8600b | 2020-10-29 17:09:13 -0700 | [diff] [blame] | 105 | ctx := android.NewContext(configuration) |
Chris Parsons | f3c96ef | 2020-09-29 02:23:17 -0400 | [diff] [blame] | 106 | ctx.SetNameInterface(newNameResolver(configuration)) |
| 107 | ctx.SetAllowMissingDependencies(configuration.AllowMissingDependencies()) |
Spandan Das | c576383 | 2022-11-08 18:42:16 +0000 | [diff] [blame] | 108 | ctx.AddIncludeTags(configuration.IncludeTags()...) |
Sam Delmerico | 98a7329 | 2023-02-21 11:50:29 -0500 | [diff] [blame] | 109 | ctx.AddSourceRootDirs(configuration.SourceRootDirs()...) |
Chris Parsons | f3c96ef | 2020-09-29 02:23:17 -0400 | [diff] [blame] | 110 | return ctx |
| 111 | } |
| 112 | |
Chris Parsons | f874e46 | 2022-05-10 13:50:12 -0400 | [diff] [blame] | 113 | // Bazel-enabled mode. Attaches a mutator to queue Bazel requests, adds a |
| 114 | // BeforePrepareBuildActionsHook to invoke Bazel, and then uses Bazel metadata |
| 115 | // for modules that should be handled by Bazel. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 116 | func runMixedModeBuild(ctx *android.Context, extraNinjaDeps []string) string { |
Chris Parsons | f874e46 | 2022-05-10 13:50:12 -0400 | [diff] [blame] | 117 | ctx.EventHandler.Begin("mixed_build") |
| 118 | defer ctx.EventHandler.End("mixed_build") |
Lukacs T. Berki | d518e1a | 2021-04-14 13:49:50 +0200 | [diff] [blame] | 119 | |
Chris Parsons | f874e46 | 2022-05-10 13:50:12 -0400 | [diff] [blame] | 120 | bazelHook := func() error { |
Cole Faust | bc65a3f | 2023-08-01 16:38:55 +0000 | [diff] [blame] | 121 | err := ctx.Config().BazelContext.QueueBazelSandwichCqueryRequests(ctx.Config()) |
| 122 | if err != nil { |
| 123 | return err |
| 124 | } |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 125 | return ctx.Config().BazelContext.InvokeBazel(ctx.Config(), ctx) |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 126 | } |
Chris Parsons | f874e46 | 2022-05-10 13:50:12 -0400 | [diff] [blame] | 127 | ctx.SetBeforePrepareBuildActionsHook(bazelHook) |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 128 | ninjaDeps, err := bootstrap.RunBlueprint(cmdlineArgs.Args, bootstrap.DoEverything, ctx.Context, ctx.Config()) |
| 129 | maybeQuit(err, "") |
Chris Parsons | 027881c | 2022-05-24 15:38:38 -0400 | [diff] [blame] | 130 | ninjaDeps = append(ninjaDeps, extraNinjaDeps...) |
Lukacs T. Berki | 809d2ed | 2021-08-18 10:55:32 +0200 | [diff] [blame] | 131 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 132 | bazelPaths, err := readFileLines(ctx.Config().Getenv("BAZEL_DEPS_FILE")) |
MarkDacek | 0d5bca5 | 2022-10-10 20:07:48 +0000 | [diff] [blame] | 133 | if err != nil { |
| 134 | panic("Bazel deps file not found: " + err.Error()) |
| 135 | } |
| 136 | ninjaDeps = append(ninjaDeps, bazelPaths...) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 137 | ninjaDeps = append(ninjaDeps, writeBuildGlobsNinjaFile(ctx)...) |
Lukacs T. Berki | 809d2ed | 2021-08-18 10:55:32 +0200 | [diff] [blame] | 138 | |
Paul Duffin | 780a185 | 2022-11-05 10:17:12 +0000 | [diff] [blame] | 139 | writeDepFile(cmdlineArgs.OutFile, ctx.EventHandler, ninjaDeps) |
Jeongik Cha | 591366d | 2023-05-08 11:32:52 +0900 | [diff] [blame] | 140 | |
Jeongik Cha | a87506f | 2023-06-01 23:16:41 +0900 | [diff] [blame] | 141 | if needToWriteNinjaHint(ctx) { |
Jeongik Cha | 591366d | 2023-05-08 11:32:52 +0900 | [diff] [blame] | 142 | writeNinjaHint(ctx) |
| 143 | } |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 144 | return cmdlineArgs.OutFile |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 145 | } |
| 146 | |
Jeongik Cha | a87506f | 2023-06-01 23:16:41 +0900 | [diff] [blame] | 147 | func needToWriteNinjaHint(ctx *android.Context) bool { |
| 148 | switch ctx.Config().GetenvWithDefault("SOONG_GENERATES_NINJA_HINT", "") { |
| 149 | case "always": |
| 150 | return true |
| 151 | case "depend": |
| 152 | if _, err := os.Stat(filepath.Join(ctx.Config().OutDir(), ".ninja_log")); errors.Is(err, os.ErrNotExist) { |
| 153 | return true |
| 154 | } |
| 155 | } |
| 156 | return false |
| 157 | } |
| 158 | |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 159 | // Run the code-generation phase to convert BazelTargetModules to BUILD files. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 160 | func runQueryView(queryviewDir, queryviewMarker string, ctx *android.Context) { |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 161 | ctx.EventHandler.Begin("queryview") |
| 162 | defer ctx.EventHandler.End("queryview") |
Cole Faust | b85d1a1 | 2022-11-08 18:14:01 -0800 | [diff] [blame] | 163 | codegenContext := bp2build.NewCodegenContext(ctx.Config(), ctx, bp2build.QueryView, topDir) |
Spandan Das | 98cb856 | 2023-03-09 23:05:47 +0000 | [diff] [blame] | 164 | err := createBazelWorkspace(codegenContext, shared.JoinPath(topDir, queryviewDir), false) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 165 | maybeQuit(err, "") |
Lukacs T. Berki | 3a82169 | 2021-09-06 17:08:02 +0200 | [diff] [blame] | 166 | touch(shared.JoinPath(topDir, queryviewMarker)) |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 167 | } |
| 168 | |
Jeongik Cha | e114e60 | 2023-03-19 00:12:39 +0900 | [diff] [blame] | 169 | func writeNinjaHint(ctx *android.Context) error { |
Jeongik Cha | 73d4911 | 2023-05-04 18:16:11 +0900 | [diff] [blame] | 170 | ctx.BeginEvent("ninja_hint") |
| 171 | defer ctx.EndEvent("ninja_hint") |
Jeongik Cha | b745e2e | 2023-04-11 14:28:43 +0900 | [diff] [blame] | 172 | // The current predictor focuses on reducing false negatives. |
| 173 | // If there are too many false positives (e.g., most modules are marked as positive), |
| 174 | // real long-running jobs cannot run early. |
| 175 | // Therefore, the model should be adjusted in this case. |
| 176 | // The model should also be adjusted if there are critical false negatives. |
| 177 | predicate := func(j *blueprint.JsonModule) (prioritized bool, weight int) { |
| 178 | prioritized = false |
| 179 | weight = 0 |
| 180 | for prefix, w := range allowlists.HugeModuleTypePrefixMap { |
| 181 | if strings.HasPrefix(j.Type, prefix) { |
| 182 | prioritized = true |
| 183 | weight = w |
| 184 | return |
| 185 | } |
Jeongik Cha | e114e60 | 2023-03-19 00:12:39 +0900 | [diff] [blame] | 186 | } |
Jeongik Cha | b745e2e | 2023-04-11 14:28:43 +0900 | [diff] [blame] | 187 | dep_count := len(j.Deps) |
| 188 | src_count := 0 |
| 189 | for _, a := range j.Module["Actions"].([]blueprint.JSONAction) { |
| 190 | src_count += len(a.Inputs) |
| 191 | } |
| 192 | input_size := dep_count + src_count |
| 193 | |
| 194 | // Current threshold is an arbitrary value which only consider recall rather than accuracy. |
| 195 | if input_size > allowlists.INPUT_SIZE_THRESHOLD { |
| 196 | prioritized = true |
| 197 | weight += ((input_size) / allowlists.INPUT_SIZE_THRESHOLD) * allowlists.DEFAULT_PRIORITIZED_WEIGHT |
| 198 | |
| 199 | // To prevent some modules from having too large a priority value. |
| 200 | if weight > allowlists.HIGH_PRIORITIZED_WEIGHT { |
| 201 | weight = allowlists.HIGH_PRIORITIZED_WEIGHT |
| 202 | } |
| 203 | } |
| 204 | return |
| 205 | } |
| 206 | |
| 207 | outputsMap := ctx.Context.GetWeightedOutputsFromPredicate(predicate) |
| 208 | var outputBuilder strings.Builder |
| 209 | for output, weight := range outputsMap { |
| 210 | outputBuilder.WriteString(fmt.Sprintf("%s,%d\n", output, weight)) |
Jeongik Cha | e114e60 | 2023-03-19 00:12:39 +0900 | [diff] [blame] | 211 | } |
| 212 | weightListFile := filepath.Join(topDir, ctx.Config().OutDir(), ".ninja_weight_list") |
| 213 | |
| 214 | err := os.WriteFile(weightListFile, []byte(outputBuilder.String()), 0644) |
| 215 | if err != nil { |
| 216 | return fmt.Errorf("could not write ninja weight list file %s", err) |
| 217 | } |
| 218 | return nil |
| 219 | } |
| 220 | |
Paul Duffin | 780a185 | 2022-11-05 10:17:12 +0000 | [diff] [blame] | 221 | func writeMetrics(configuration android.Config, eventHandler *metrics.EventHandler, metricsDir string) { |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 222 | if len(metricsDir) < 1 { |
| 223 | fmt.Fprintf(os.Stderr, "\nMissing required env var for generating soong metrics: LOG_DIR\n") |
| 224 | os.Exit(1) |
| 225 | } |
| 226 | metricsFile := filepath.Join(metricsDir, "soong_build_metrics.pb") |
| 227 | err := android.WriteMetrics(configuration, eventHandler, metricsFile) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 228 | maybeQuit(err, "error writing soong_build metrics %s", metricsFile) |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 229 | } |
| 230 | |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 231 | // Errors out if any modules expected to be mixed_built were not, unless |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 232 | // the modules did not exist. |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 233 | func checkForAllowlistIntegrityError(configuration android.Config, isStagingMode bool) error { |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 234 | modules := findMisconfiguredModules(configuration, isStagingMode) |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 235 | if len(modules) == 0 { |
| 236 | return nil |
| 237 | } |
| 238 | |
| 239 | return fmt.Errorf("Error: expected the following modules to be mixed_built: %s", modules) |
| 240 | } |
| 241 | |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 242 | // Returns true if the given module has all of the following true: |
| 243 | // 1. Is allowlisted to be built with Bazel. |
| 244 | // 2. Has a variant which is *not* built with Bazel. |
| 245 | // 3. Has no variant which is built with Bazel. |
| 246 | // |
| 247 | // This indicates the allowlisting of this variant had no effect. |
| 248 | // TODO(b/280457637): Return true for nonexistent modules. |
| 249 | func isAllowlistMisconfiguredForModule(module string, mixedBuildsEnabled map[string]struct{}, mixedBuildsDisabled map[string]struct{}) bool { |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 250 | _, enabled := mixedBuildsEnabled[module] |
| 251 | |
| 252 | if enabled { |
| 253 | return false |
| 254 | } |
| 255 | |
| 256 | _, disabled := mixedBuildsDisabled[module] |
| 257 | return disabled |
| 258 | |
| 259 | } |
| 260 | |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 261 | // Returns the list of modules that should have been mixed_built (per the |
| 262 | // allowlists and cmdline flags) but were not. |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 263 | // Note: nonexistent modules are excluded from the list. See b/280457637 |
| 264 | func findMisconfiguredModules(configuration android.Config, isStagingMode bool) []string { |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 265 | retval := []string{} |
| 266 | forceEnabledModules := configuration.BazelModulesForceEnabledByFlag() |
| 267 | |
| 268 | mixedBuildsEnabled := configuration.GetMixedBuildsEnabledModules() |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 269 | mixedBuildsDisabled := configuration.GetMixedBuildsDisabledModules() |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 270 | for _, module := range allowlists.ProdMixedBuildsEnabledList { |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 271 | if isAllowlistMisconfiguredForModule(module, mixedBuildsEnabled, mixedBuildsDisabled) { |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 272 | retval = append(retval, module) |
| 273 | } |
| 274 | } |
| 275 | |
| 276 | if isStagingMode { |
| 277 | for _, module := range allowlists.StagingMixedBuildsEnabledList { |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 278 | if isAllowlistMisconfiguredForModule(module, mixedBuildsEnabled, mixedBuildsDisabled) { |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 279 | retval = append(retval, module) |
| 280 | } |
| 281 | } |
| 282 | } |
| 283 | |
| 284 | for module, _ := range forceEnabledModules { |
MarkDacek | 6f6b962 | 2023-05-02 16:28:55 +0000 | [diff] [blame] | 285 | if isAllowlistMisconfiguredForModule(module, mixedBuildsEnabled, mixedBuildsDisabled) { |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 286 | retval = append(retval, module) |
| 287 | } |
| 288 | } |
| 289 | return retval |
| 290 | } |
| 291 | |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 292 | func writeJsonModuleGraphAndActions(ctx *android.Context, cmdArgs android.CmdArgs) { |
| 293 | graphFile, graphErr := os.Create(shared.JoinPath(topDir, cmdArgs.ModuleGraphFile)) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 294 | maybeQuit(graphErr, "graph err") |
kgui | 6700724 | 2022-01-25 13:50:25 +0800 | [diff] [blame] | 295 | defer graphFile.Close() |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 296 | actionsFile, actionsErr := os.Create(shared.JoinPath(topDir, cmdArgs.ModuleActionsFile)) |
| 297 | maybeQuit(actionsErr, "actions err") |
kgui | 6700724 | 2022-01-25 13:50:25 +0800 | [diff] [blame] | 298 | defer actionsFile.Close() |
| 299 | ctx.Context.PrintJSONGraphAndActions(graphFile, actionsFile) |
Lukacs T. Berki | 97bb9f1 | 2021-04-01 18:28:45 +0200 | [diff] [blame] | 300 | } |
| 301 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 302 | func writeBuildGlobsNinjaFile(ctx *android.Context) []string { |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 303 | ctx.EventHandler.Begin("globs_ninja_file") |
| 304 | defer ctx.EventHandler.End("globs_ninja_file") |
| 305 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 306 | globDir := bootstrap.GlobDirectory(ctx.Config().SoongOutDir(), globListDir) |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 307 | err := bootstrap.WriteBuildGlobsNinjaFile(&bootstrap.GlobSingleton{ |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 308 | GlobLister: ctx.Globs, |
Lukacs T. Berki | 809d2ed | 2021-08-18 10:55:32 +0200 | [diff] [blame] | 309 | GlobFile: globFile, |
| 310 | GlobDir: globDir, |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 311 | SrcDir: ctx.SrcDir(), |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 312 | }, ctx.Config()) |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 313 | maybeQuit(err, "") |
Lukacs T. Berki | 809d2ed | 2021-08-18 10:55:32 +0200 | [diff] [blame] | 314 | return bootstrap.GlobFileListFiles(globDir) |
| 315 | } |
| 316 | |
Paul Duffin | 780a185 | 2022-11-05 10:17:12 +0000 | [diff] [blame] | 317 | func writeDepFile(outputFile string, eventHandler *metrics.EventHandler, ninjaDeps []string) { |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 318 | eventHandler.Begin("ninja_deps") |
| 319 | defer eventHandler.End("ninja_deps") |
Lukacs T. Berki | e571dc3 | 2021-08-25 14:14:13 +0200 | [diff] [blame] | 320 | depFile := shared.JoinPath(topDir, outputFile+".d") |
| 321 | err := deptools.WriteDepFile(depFile, outputFile, ninjaDeps) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 322 | maybeQuit(err, "error writing depfile '%s'", depFile) |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 323 | } |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 324 | |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 325 | // runSoongOnlyBuild runs the standard Soong build in a number of different modes. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 326 | func runSoongOnlyBuild(ctx *android.Context, extraNinjaDeps []string) string { |
Paul Duffin | 39eae8f | 2022-11-05 14:59:52 +0000 | [diff] [blame] | 327 | ctx.EventHandler.Begin("soong_build") |
| 328 | defer ctx.EventHandler.End("soong_build") |
| 329 | |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 330 | var stopBefore bootstrap.StopBefore |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 331 | switch ctx.Config().BuildMode { |
| 332 | case android.GenerateModuleGraph: |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 333 | stopBefore = bootstrap.StopBeforeWriteNinja |
Usta Shrestha | 7fae695 | 2022-12-21 11:46:28 -0500 | [diff] [blame] | 334 | case android.GenerateQueryView, android.GenerateDocFile: |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 335 | stopBefore = bootstrap.StopBeforePrepareBuildActions |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 336 | default: |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 337 | stopBefore = bootstrap.DoEverything |
| 338 | } |
| 339 | |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 340 | ninjaDeps, err := bootstrap.RunBlueprint(cmdlineArgs.Args, stopBefore, ctx.Context, ctx.Config()) |
| 341 | maybeQuit(err, "") |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 342 | ninjaDeps = append(ninjaDeps, extraNinjaDeps...) |
| 343 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 344 | globListFiles := writeBuildGlobsNinjaFile(ctx) |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 345 | ninjaDeps = append(ninjaDeps, globListFiles...) |
| 346 | |
| 347 | // Convert the Soong module graph into Bazel BUILD files. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 348 | switch ctx.Config().BuildMode { |
| 349 | case android.GenerateQueryView: |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 350 | queryviewMarkerFile := cmdlineArgs.BazelQueryViewDir + ".marker" |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 351 | runQueryView(cmdlineArgs.BazelQueryViewDir, queryviewMarkerFile, ctx) |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 352 | writeDepFile(queryviewMarkerFile, ctx.EventHandler, ninjaDeps) |
| 353 | return queryviewMarkerFile |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 354 | case android.GenerateModuleGraph: |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 355 | writeJsonModuleGraphAndActions(ctx, cmdlineArgs) |
| 356 | writeDepFile(cmdlineArgs.ModuleGraphFile, ctx.EventHandler, ninjaDeps) |
| 357 | return cmdlineArgs.ModuleGraphFile |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 358 | case android.GenerateDocFile: |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 359 | // TODO: we could make writeDocs() return the list of documentation files |
| 360 | // written and add them to the .d file. Then soong_docs would be re-run |
| 361 | // whenever one is deleted. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 362 | err := writeDocs(ctx, shared.JoinPath(topDir, cmdlineArgs.DocFile)) |
| 363 | maybeQuit(err, "error building Soong documentation") |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 364 | writeDepFile(cmdlineArgs.DocFile, ctx.EventHandler, ninjaDeps) |
| 365 | return cmdlineArgs.DocFile |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 366 | default: |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 367 | // The actual output (build.ninja) was written in the RunBlueprint() call |
| 368 | // above |
| 369 | writeDepFile(cmdlineArgs.OutFile, ctx.EventHandler, ninjaDeps) |
Jeongik Cha | a87506f | 2023-06-01 23:16:41 +0900 | [diff] [blame] | 370 | if needToWriteNinjaHint(ctx) { |
Jeongik Cha | 591366d | 2023-05-08 11:32:52 +0900 | [diff] [blame] | 371 | writeNinjaHint(ctx) |
| 372 | } |
Paul Duffin | b713ddf | 2022-11-05 16:14:30 +0000 | [diff] [blame] | 373 | return cmdlineArgs.OutFile |
Paul Duffin | 0c09a43 | 2022-11-05 15:28:04 +0000 | [diff] [blame] | 374 | } |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 375 | } |
| 376 | |
| 377 | // soong_ui dumps the available environment variables to |
| 378 | // soong.environment.available . Then soong_build itself is run with an empty |
| 379 | // environment so that the only way environment variables can be accessed is |
| 380 | // using Config, which tracks access to them. |
| 381 | |
| 382 | // At the end of the build, a file called soong.environment.used is written |
| 383 | // containing the current value of all used environment variables. The next |
| 384 | // time soong_ui is run, it checks whether any environment variables that was |
| 385 | // used had changed and if so, it deletes soong.environment.used to cause a |
| 386 | // rebuild. |
| 387 | // |
| 388 | // The dependency of build.ninja on soong.environment.used is declared in |
| 389 | // build.ninja.d |
| 390 | func parseAvailableEnv() map[string]string { |
| 391 | if availableEnvFile == "" { |
| 392 | fmt.Fprintf(os.Stderr, "--available_env not set\n") |
| 393 | os.Exit(1) |
| 394 | } |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 395 | result, err := shared.EnvFromFile(shared.JoinPath(topDir, availableEnvFile)) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 396 | maybeQuit(err, "error reading available environment file '%s'", availableEnvFile) |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 397 | return result |
Lukacs T. Berki | 6790ebc | 2021-04-01 17:55:58 +0200 | [diff] [blame] | 398 | } |
| 399 | |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 400 | func main() { |
| 401 | flag.Parse() |
| 402 | |
Lukacs T. Berki | 7d613bf | 2021-03-02 10:09:41 +0100 | [diff] [blame] | 403 | shared.ReexecWithDelveMaybe(delveListen, delvePath) |
Lukacs T. Berki | 7690c09 | 2021-02-26 14:27:36 +0100 | [diff] [blame] | 404 | android.InitSandbox(topDir) |
Lukacs T. Berki | 7690c09 | 2021-02-26 14:27:36 +0100 | [diff] [blame] | 405 | |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 406 | availableEnv := parseAvailableEnv() |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 407 | configuration, err := android.NewConfig(cmdlineArgs, availableEnv) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 408 | maybeQuit(err, "") |
Lukacs T. Berki | d1e3f1f | 2021-03-16 08:55:23 +0100 | [diff] [blame] | 409 | if configuration.Getenv("ALLOW_MISSING_DEPENDENCIES") == "true" { |
| 410 | configuration.SetAllowMissingDependencies() |
| 411 | } |
| 412 | |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 413 | extraNinjaDeps := []string{configuration.ProductVariablesFileName, usedEnvFile} |
Lukacs T. Berki | 7d613bf | 2021-03-02 10:09:41 +0100 | [diff] [blame] | 414 | if shared.IsDebugging() { |
Colin Cross | aa812d1 | 2019-06-19 13:33:24 -0700 | [diff] [blame] | 415 | // Add a non-existent file to the dependencies so that soong_build will rerun when the debugger is |
| 416 | // enabled even if it completed successfully. |
Lukacs T. Berki | 9f6c24a | 2021-08-26 15:07:24 +0200 | [diff] [blame] | 417 | extraNinjaDeps = append(extraNinjaDeps, filepath.Join(configuration.SoongOutDir(), "always_rerun_for_delve")) |
Colin Cross | aa812d1 | 2019-06-19 13:33:24 -0700 | [diff] [blame] | 418 | } |
Jingwen Chen | daa54bc | 2020-12-14 02:58:54 -0500 | [diff] [blame] | 419 | |
Dan Willemsen | ccf36aa | 2022-04-20 23:11:43 -0700 | [diff] [blame] | 420 | // Bypass configuration.Getenv, as LOG_DIR does not need to be dependency tracked. By definition, it will |
| 421 | // change between every CI build, so tracking it would require re-running Soong for every build. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 422 | metricsDir := availableEnv["LOG_DIR"] |
Dan Willemsen | ccf36aa | 2022-04-20 23:11:43 -0700 | [diff] [blame] | 423 | |
Joe Onorato | 2e5e401 | 2022-06-07 17:16:08 -0700 | [diff] [blame] | 424 | ctx := newContext(configuration) |
Colin Cross | 46b0c75 | 2023-10-27 14:56:12 -0700 | [diff] [blame^] | 425 | android.StartBackgroundMetrics(configuration) |
Joe Onorato | 2e5e401 | 2022-06-07 17:16:08 -0700 | [diff] [blame] | 426 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 427 | var finalOutputFile string |
Joe Onorato | 2e5e401 | 2022-06-07 17:16:08 -0700 | [diff] [blame] | 428 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 429 | // Run Soong for a specific activity, like bp2build, queryview |
| 430 | // or the actual Soong build for the build.ninja file. |
| 431 | switch configuration.BuildMode { |
| 432 | case android.SymlinkForest: |
| 433 | finalOutputFile = runSymlinkForestCreation(ctx, extraNinjaDeps, metricsDir) |
| 434 | case android.Bp2build: |
| 435 | // Run the alternate pipeline of bp2build mutators and singleton to convert |
| 436 | // Blueprint to BUILD files before everything else. |
| 437 | finalOutputFile = runBp2Build(ctx, extraNinjaDeps, metricsDir) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 438 | default: |
| 439 | ctx.Register() |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 440 | isMixedBuildsEnabled := configuration.IsMixedBuildsEnabled() |
| 441 | if isMixedBuildsEnabled { |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 442 | finalOutputFile = runMixedModeBuild(ctx, extraNinjaDeps) |
MarkDacek | f47e142 | 2023-04-19 16:47:36 +0000 | [diff] [blame] | 443 | if cmdlineArgs.EnsureAllowlistIntegrity { |
| 444 | if err := checkForAllowlistIntegrityError(configuration, cmdlineArgs.BazelModeStaging); err != nil { |
| 445 | maybeQuit(err, "") |
| 446 | } |
| 447 | } |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 448 | } else { |
| 449 | finalOutputFile = runSoongOnlyBuild(ctx, extraNinjaDeps) |
| 450 | } |
| 451 | writeMetrics(configuration, ctx.EventHandler, metricsDir) |
| 452 | } |
Chris Parsons | c83398f | 2023-05-31 18:41:41 +0000 | [diff] [blame] | 453 | |
| 454 | // Register this environment variablesas being an implicit dependencies of |
| 455 | // soong_build. Changes to this environment variable will result in |
| 456 | // retriggering soong_build. |
| 457 | configuration.Getenv("USE_BAZEL_VERSION") |
| 458 | |
Chris Parsons | a3ae007 | 2023-05-10 21:10:08 +0000 | [diff] [blame] | 459 | writeUsedEnvironmentFile(configuration) |
| 460 | |
| 461 | // Touch the output file so that it's the newest file created by soong_build. |
| 462 | // This is necessary because, if soong_build generated any files which |
| 463 | // are ninja inputs to the main output file, then ninja would superfluously |
| 464 | // rebuild this output file on the next build invocation. |
| 465 | touch(shared.JoinPath(topDir, finalOutputFile)) |
Lukacs T. Berki | f0b3b94 | 2021-03-23 11:46:47 +0100 | [diff] [blame] | 466 | } |
| 467 | |
Chris Parsons | a3ae007 | 2023-05-10 21:10:08 +0000 | [diff] [blame] | 468 | func writeUsedEnvironmentFile(configuration android.Config) { |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 469 | if usedEnvFile == "" { |
| 470 | return |
| 471 | } |
| 472 | |
| 473 | path := shared.JoinPath(topDir, usedEnvFile) |
Lukacs T. Berki | f0b3b94 | 2021-03-23 11:46:47 +0100 | [diff] [blame] | 474 | data, err := shared.EnvFileContents(configuration.EnvDeps()) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 475 | maybeQuit(err, "error writing used environment file '%s'\n", usedEnvFile) |
Lukacs T. Berki | f0b3b94 | 2021-03-23 11:46:47 +0100 | [diff] [blame] | 476 | |
Usta Shrestha | 2ba28a3 | 2022-10-24 11:33:09 -0400 | [diff] [blame] | 477 | if preexistingData, err := os.ReadFile(path); err != nil { |
| 478 | if !os.IsNotExist(err) { |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 479 | maybeQuit(err, "error reading used environment file '%s'", usedEnvFile) |
Usta Shrestha | 2ba28a3 | 2022-10-24 11:33:09 -0400 | [diff] [blame] | 480 | } |
| 481 | } else if bytes.Equal(preexistingData, data) { |
| 482 | // used environment file is unchanged |
| 483 | return |
| 484 | } |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 485 | err = os.WriteFile(path, data, 0666) |
| 486 | maybeQuit(err, "error writing used environment file '%s'", usedEnvFile) |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 487 | } |
Jingwen Chen | 5ba7e47 | 2020-07-15 10:06:41 +0000 | [diff] [blame] | 488 | |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 489 | func touch(path string) { |
| 490 | f, err := os.OpenFile(path, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0666) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 491 | maybeQuit(err, "Error touching '%s'", path) |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 492 | err = f.Close() |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 493 | maybeQuit(err, "Error touching '%s'", path) |
Lukacs T. Berki | f8e2428 | 2021-04-14 10:31:00 +0200 | [diff] [blame] | 494 | |
| 495 | currentTime := time.Now().Local() |
| 496 | err = os.Chtimes(path, currentTime, currentTime) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 497 | maybeQuit(err, "error touching '%s'", path) |
Rupert Shuttleworth | 0096079 | 2021-05-12 21:20:13 -0400 | [diff] [blame] | 498 | } |
| 499 | |
Rupert Shuttleworth | e03bb61 | 2021-05-20 19:34:50 -0400 | [diff] [blame] | 500 | // Read the bazel.list file that the Soong Finder already dumped earlier (hopefully) |
| 501 | // It contains the locations of BUILD files, BUILD.bazel files, etc. in the source dir |
| 502 | func getExistingBazelRelatedFiles(topDir string) ([]string, error) { |
Lukacs T. Berki | f900807 | 2021-08-16 15:24:48 +0200 | [diff] [blame] | 503 | bazelFinderFile := filepath.Join(filepath.Dir(cmdlineArgs.ModuleListFile), "bazel.list") |
Rupert Shuttleworth | e03bb61 | 2021-05-20 19:34:50 -0400 | [diff] [blame] | 504 | if !filepath.IsAbs(bazelFinderFile) { |
| 505 | // Assume this was a relative path under topDir |
| 506 | bazelFinderFile = filepath.Join(topDir, bazelFinderFile) |
| 507 | } |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 508 | return readFileLines(bazelFinderFile) |
Rupert Shuttleworth | e03bb61 | 2021-05-20 19:34:50 -0400 | [diff] [blame] | 509 | } |
| 510 | |
Spandan Das | 5af0bd3 | 2022-09-28 20:43:08 +0000 | [diff] [blame] | 511 | func bazelArtifacts() []string { |
| 512 | return []string{ |
| 513 | "bazel-bin", |
| 514 | "bazel-genfiles", |
| 515 | "bazel-out", |
| 516 | "bazel-testlogs", |
Usta (Tsering) Shrestha | c4c07b1 | 2022-11-08 18:31:14 -0500 | [diff] [blame] | 517 | "bazel-workspace", |
Spandan Das | 5af0bd3 | 2022-09-28 20:43:08 +0000 | [diff] [blame] | 518 | "bazel-" + filepath.Base(topDir), |
| 519 | } |
| 520 | } |
| 521 | |
Lukacs T. Berki | c541cd2 | 2022-10-26 07:26:50 +0000 | [diff] [blame] | 522 | // This could in theory easily be separated into a binary that generically |
| 523 | // merges two directories into a symlink tree. The main obstacle is that this |
| 524 | // function currently depends on both Bazel-specific knowledge (the existence |
| 525 | // of bazel-* symlinks) and configuration (the set of BUILD.bazel files that |
| 526 | // should and should not be kept) |
| 527 | // |
| 528 | // Ideally, bp2build would write a file that contains instructions to the |
| 529 | // symlink tree creation binary. Then the latter would not need to depend on |
| 530 | // the very heavy-weight machinery of soong_build . |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 531 | func runSymlinkForestCreation(ctx *android.Context, extraNinjaDeps []string, metricsDir string) string { |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 532 | var ninjaDeps []string |
| 533 | var mkdirCount, symlinkCount uint64 |
| 534 | |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 535 | ctx.EventHandler.Do("symlink_forest", func() { |
Usta (Tsering) Shrestha | 93b2a9b | 2022-12-01 05:55:35 +0000 | [diff] [blame] | 536 | ninjaDeps = append(ninjaDeps, extraNinjaDeps...) |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 537 | verbose := ctx.Config().IsEnvTrue("BP2BUILD_VERBOSE") |
Usta (Tsering) Shrestha | 93b2a9b | 2022-12-01 05:55:35 +0000 | [diff] [blame] | 538 | |
| 539 | // PlantSymlinkForest() returns all the directories that were readdir()'ed. |
| 540 | // Such a directory SHOULD be added to `ninjaDeps` so that a child directory |
| 541 | // or file created/deleted under it would trigger an update of the symlink forest. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 542 | generatedRoot := shared.JoinPath(ctx.Config().SoongOutDir(), "bp2build") |
| 543 | workspaceRoot := shared.JoinPath(ctx.Config().SoongOutDir(), "workspace") |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 544 | var symlinkForestDeps []string |
Usta (Tsering) Shrestha | 93b2a9b | 2022-12-01 05:55:35 +0000 | [diff] [blame] | 545 | ctx.EventHandler.Do("plant", func() { |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 546 | symlinkForestDeps, mkdirCount, symlinkCount = bp2build.PlantSymlinkForest( |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 547 | verbose, topDir, workspaceRoot, generatedRoot, excludedFromSymlinkForest(ctx, verbose)) |
Usta (Tsering) Shrestha | 93b2a9b | 2022-12-01 05:55:35 +0000 | [diff] [blame] | 548 | }) |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 549 | ninjaDeps = append(ninjaDeps, symlinkForestDeps...) |
Usta (Tsering) Shrestha | 93b2a9b | 2022-12-01 05:55:35 +0000 | [diff] [blame] | 550 | }) |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 551 | |
| 552 | writeDepFile(cmdlineArgs.SymlinkForestMarker, ctx.EventHandler, ninjaDeps) |
| 553 | touch(shared.JoinPath(topDir, cmdlineArgs.SymlinkForestMarker)) |
usta | 4f5d2c1 | 2022-10-28 23:32:01 -0400 | [diff] [blame] | 554 | codegenMetrics := bp2build.ReadCodegenMetrics(metricsDir) |
| 555 | if codegenMetrics == nil { |
| 556 | m := bp2build.CreateCodegenMetrics() |
| 557 | codegenMetrics = &m |
| 558 | } else { |
| 559 | //TODO (usta) we cannot determine if we loaded a stale file, i.e. from an unrelated prior |
| 560 | //invocation of codegen. We should simply use a separate .pb file |
| 561 | } |
Usta Shrestha | da15c61 | 2022-11-08 14:12:36 -0500 | [diff] [blame] | 562 | codegenMetrics.SetSymlinkCount(symlinkCount) |
| 563 | codegenMetrics.SetMkDirCount(mkdirCount) |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 564 | writeBp2BuildMetrics(codegenMetrics, ctx.EventHandler, metricsDir) |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 565 | return cmdlineArgs.SymlinkForestMarker |
Lukacs T. Berki | c541cd2 | 2022-10-26 07:26:50 +0000 | [diff] [blame] | 566 | } |
| 567 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 568 | func excludedFromSymlinkForest(ctx *android.Context, verbose bool) []string { |
| 569 | excluded := bazelArtifacts() |
| 570 | if cmdlineArgs.OutDir[0] != '/' { |
| 571 | excluded = append(excluded, cmdlineArgs.OutDir) |
| 572 | } |
| 573 | |
| 574 | // Find BUILD files in the srcDir which are not in the allowlist |
| 575 | // (android.Bp2BuildConversionAllowlist#ShouldKeepExistingBuildFileForDir) |
| 576 | // and return their paths so they can be left out of the Bazel workspace dir (i.e. ignored) |
| 577 | existingBazelFiles, err := getExistingBazelRelatedFiles(topDir) |
| 578 | maybeQuit(err, "Error determining existing Bazel-related files") |
| 579 | |
| 580 | for _, path := range existingBazelFiles { |
| 581 | fullPath := shared.JoinPath(topDir, path) |
| 582 | fileInfo, err2 := os.Stat(fullPath) |
| 583 | if err2 != nil { |
| 584 | // Warn about error, but continue trying to check files |
| 585 | fmt.Fprintf(os.Stderr, "WARNING: Error accessing path '%s', err: %s\n", fullPath, err2) |
| 586 | continue |
| 587 | } |
| 588 | // Exclude only files named 'BUILD' or 'BUILD.bazel' and unless forcibly kept |
| 589 | if fileInfo.IsDir() || |
| 590 | (fileInfo.Name() != "BUILD" && fileInfo.Name() != "BUILD.bazel") || |
| 591 | ctx.Config().Bp2buildPackageConfig.ShouldKeepExistingBuildFileForDir(filepath.Dir(path)) { |
| 592 | // Don't ignore this existing build file |
| 593 | continue |
| 594 | } |
| 595 | if verbose { |
| 596 | fmt.Fprintf(os.Stderr, "Ignoring existing BUILD file: %s\n", path) |
| 597 | } |
| 598 | excluded = append(excluded, path) |
| 599 | } |
| 600 | |
| 601 | // Temporarily exclude stuff to make `bazel build //external/...` (and `bazel build //frameworks/...`) work |
| 602 | excluded = append(excluded, |
| 603 | // FIXME: 'autotest_lib' is a symlink back to external/autotest, and this causes an infinite |
| 604 | // symlink expansion error for Bazel |
| 605 | "external/autotest/venv/autotest_lib", |
| 606 | "external/autotest/autotest_lib", |
| 607 | "external/autotest/client/autotest_lib/client", |
| 608 | |
| 609 | // FIXME: The external/google-fruit/extras/bazel_root/third_party/fruit dir is poison |
| 610 | // It contains several symlinks back to real source dirs, and those source dirs contain |
| 611 | // BUILD files we want to ignore |
| 612 | "external/google-fruit/extras/bazel_root/third_party/fruit", |
| 613 | |
| 614 | // FIXME: 'frameworks/compile/slang' has a filegroup error due to an escaping issue |
| 615 | "frameworks/compile/slang", |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 616 | ) |
| 617 | return excluded |
| 618 | } |
| 619 | |
Jingwen Chen | daa54bc | 2020-12-14 02:58:54 -0500 | [diff] [blame] | 620 | // Run Soong in the bp2build mode. This creates a standalone context that registers |
| 621 | // an alternate pipeline of mutators and singletons specifically for generating |
| 622 | // Bazel BUILD files instead of Ninja files. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 623 | func runBp2Build(ctx *android.Context, extraNinjaDeps []string, metricsDir string) string { |
usta | 4f5d2c1 | 2022-10-28 23:32:01 -0400 | [diff] [blame] | 624 | var codegenMetrics *bp2build.CodegenMetrics |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 625 | ctx.EventHandler.Do("bp2build", func() { |
Jingwen Chen | eb76c43 | 2021-01-28 08:22:12 -0500 | [diff] [blame] | 626 | |
Chris Parsons | 8152a94 | 2023-06-06 16:17:50 +0000 | [diff] [blame] | 627 | ctx.EventHandler.Do("read_build", func() { |
Chris Parsons | 5011e61 | 2023-09-13 23:33:20 +0000 | [diff] [blame] | 628 | existingBazelFiles, err := getExistingBazelRelatedFiles(topDir) |
| 629 | maybeQuit(err, "Error determining existing Bazel-related files") |
| 630 | |
| 631 | err = ctx.RegisterExistingBazelTargets(topDir, existingBazelFiles) |
| 632 | maybeQuit(err, "Error parsing existing Bazel-related files") |
Chris Parsons | 8152a94 | 2023-06-06 16:17:50 +0000 | [diff] [blame] | 633 | }) |
| 634 | |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 635 | // Propagate "allow misssing dependencies" bit. This is normally set in |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 636 | // newContext(), but we create ctx without calling that method. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 637 | ctx.SetAllowMissingDependencies(ctx.Config().AllowMissingDependencies()) |
| 638 | ctx.SetNameInterface(newNameResolver(ctx.Config())) |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 639 | ctx.RegisterForBazelConversion() |
| 640 | ctx.SetModuleListFile(cmdlineArgs.ModuleListFile) |
Chris Parsons | 3a5c170 | 2023-06-13 01:10:20 +0000 | [diff] [blame] | 641 | // Skip cloning modules during bp2build's blueprint run. Some mutators set |
| 642 | // bp2build-related module values which should be preserved during codegen. |
| 643 | ctx.SkipCloneModulesAfterMutators = true |
Jingwen Chen | eb76c43 | 2021-01-28 08:22:12 -0500 | [diff] [blame] | 644 | |
Usta Shrestha | a117c58 | 2022-10-04 12:13:25 -0400 | [diff] [blame] | 645 | var ninjaDeps []string |
| 646 | ninjaDeps = append(ninjaDeps, extraNinjaDeps...) |
| 647 | |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 648 | // Run the loading and analysis pipeline to prepare the graph of regular |
| 649 | // Modules parsed from Android.bp files, and the BazelTargetModules mapped |
| 650 | // from the regular Modules. |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 651 | ctx.EventHandler.Do("bootstrap", func() { |
Usta Shrestha | a117c58 | 2022-10-04 12:13:25 -0400 | [diff] [blame] | 652 | blueprintArgs := cmdlineArgs |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 653 | bootstrapDeps, err := bootstrap.RunBlueprint(blueprintArgs.Args, |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 654 | bootstrap.StopBeforePrepareBuildActions, ctx.Context, ctx.Config()) |
Lukacs T. Berki | c357c81 | 2023-06-20 09:30:06 +0000 | [diff] [blame] | 655 | maybeQuit(err, "") |
Usta Shrestha | a117c58 | 2022-10-04 12:13:25 -0400 | [diff] [blame] | 656 | ninjaDeps = append(ninjaDeps, bootstrapDeps...) |
| 657 | }) |
Lukacs T. Berki | d518e1a | 2021-04-14 13:49:50 +0200 | [diff] [blame] | 658 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 659 | globListFiles := writeBuildGlobsNinjaFile(ctx) |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 660 | ninjaDeps = append(ninjaDeps, globListFiles...) |
Lukacs T. Berki | d518e1a | 2021-04-14 13:49:50 +0200 | [diff] [blame] | 661 | |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 662 | // Run the code-generation phase to convert BazelTargetModules to BUILD files |
Usta Shrestha | a117c58 | 2022-10-04 12:13:25 -0400 | [diff] [blame] | 663 | // and print conversion codegenMetrics to the user. |
Cole Faust | b85d1a1 | 2022-11-08 18:14:01 -0800 | [diff] [blame] | 664 | codegenContext := bp2build.NewCodegenContext(ctx.Config(), ctx, bp2build.Bp2Build, topDir) |
usta | aaf2fd1 | 2023-07-01 11:40:36 -0400 | [diff] [blame] | 665 | codegenMetrics = bp2build.Codegen(codegenContext) |
Lukacs T. Berki | b353cca | 2021-04-16 13:47:36 +0200 | [diff] [blame] | 666 | |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 667 | ninjaDeps = append(ninjaDeps, codegenContext.AdditionalNinjaDeps()...) |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 668 | |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 669 | writeDepFile(cmdlineArgs.Bp2buildMarker, ctx.EventHandler, ninjaDeps) |
| 670 | touch(shared.JoinPath(topDir, cmdlineArgs.Bp2buildMarker)) |
Usta Shrestha | 5c6b948 | 2022-05-26 12:22:17 -0400 | [diff] [blame] | 671 | }) |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 672 | |
Lukacs T. Berki | b353cca | 2021-04-16 13:47:36 +0200 | [diff] [blame] | 673 | // Only report metrics when in bp2build mode. The metrics aren't relevant |
| 674 | // for queryview, since that's a total repo-wide conversion and there's a |
| 675 | // 1:1 mapping for each module. |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 676 | if ctx.Config().IsEnvTrue("BP2BUILD_VERBOSE") { |
Usta Shrestha | a117c58 | 2022-10-04 12:13:25 -0400 | [diff] [blame] | 677 | codegenMetrics.Print() |
Sasha Smundak | 0fd93e0 | 2022-05-19 19:34:31 -0700 | [diff] [blame] | 678 | } |
Paul Duffin | b4e8d91 | 2022-11-04 13:35:50 +0000 | [diff] [blame] | 679 | writeBp2BuildMetrics(codegenMetrics, ctx.EventHandler, metricsDir) |
Sasha Smundak | af5ca92 | 2022-12-12 21:23:34 -0800 | [diff] [blame] | 680 | return cmdlineArgs.Bp2buildMarker |
Jingwen Chen | daa54bc | 2020-12-14 02:58:54 -0500 | [diff] [blame] | 681 | } |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 947fdbf | 2021-11-10 09:55:20 -0500 | [diff] [blame] | 682 | |
| 683 | // Write Bp2Build metrics into $LOG_DIR |
Paul Duffin | f490656 | 2022-11-05 14:21:16 +0000 | [diff] [blame] | 684 | func writeBp2BuildMetrics(codegenMetrics *bp2build.CodegenMetrics, eventHandler *metrics.EventHandler, metricsDir string) { |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 685 | for _, event := range eventHandler.CompletedEvents() { |
usta | 4f5d2c1 | 2022-10-28 23:32:01 -0400 | [diff] [blame] | 686 | codegenMetrics.AddEvent(&bp2build_metrics_proto.Event{ |
| 687 | Name: event.Id, |
| 688 | StartTime: uint64(event.Start.UnixNano()), |
| 689 | RealTime: event.RuntimeNanoseconds(), |
| 690 | }) |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 691 | } |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 947fdbf | 2021-11-10 09:55:20 -0500 | [diff] [blame] | 692 | if len(metricsDir) < 1 { |
| 693 | fmt.Fprintf(os.Stderr, "\nMissing required env var for generating bp2build metrics: LOG_DIR\n") |
| 694 | os.Exit(1) |
| 695 | } |
Chris Parsons | 715b08f | 2022-03-22 19:23:40 -0400 | [diff] [blame] | 696 | codegenMetrics.Write(metricsDir) |
Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | 947fdbf | 2021-11-10 09:55:20 -0500 | [diff] [blame] | 697 | } |
MarkDacek | 0d5bca5 | 2022-10-10 20:07:48 +0000 | [diff] [blame] | 698 | |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 699 | func readFileLines(path string) ([]string, error) { |
| 700 | data, err := os.ReadFile(path) |
| 701 | if err == nil { |
| 702 | return strings.Split(strings.TrimSpace(string(data)), "\n"), nil |
MarkDacek | 0d5bca5 | 2022-10-10 20:07:48 +0000 | [diff] [blame] | 703 | } |
Sasha Smundak | 1845f42 | 2022-12-13 14:18:58 -0800 | [diff] [blame] | 704 | return nil, err |
| 705 | |
| 706 | } |
| 707 | func maybeQuit(err error, format string, args ...interface{}) { |
| 708 | if err == nil { |
| 709 | return |
| 710 | } |
| 711 | if format != "" { |
| 712 | fmt.Fprintln(os.Stderr, fmt.Sprintf(format, args...)+": "+err.Error()) |
| 713 | } else { |
| 714 | fmt.Fprintln(os.Stderr, err) |
| 715 | } |
| 716 | os.Exit(1) |
MarkDacek | 0d5bca5 | 2022-10-10 20:07:48 +0000 | [diff] [blame] | 717 | } |