Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 1 | // Copyright 2017 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 build |
| 16 | |
| 17 | import ( |
Cole Faust | 583dfb4 | 2023-09-28 13:56:30 -0700 | [diff] [blame] | 18 | "android/soong/ui/metrics" |
| 19 | "android/soong/ui/status" |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 20 | "crypto/md5" |
| 21 | "fmt" |
| 22 | "io/ioutil" |
Dan Willemsen | 71edc8b | 2019-01-02 12:21:18 -0800 | [diff] [blame] | 23 | "os" |
| 24 | "os/user" |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 25 | "path/filepath" |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 26 | "strings" |
| 27 | ) |
| 28 | |
| 29 | var spaceSlashReplacer = strings.NewReplacer("/", "_", " ", "_") |
| 30 | |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 31 | const katiBuildSuffix = "" |
| 32 | const katiCleanspecSuffix = "-cleanspec" |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 33 | const katiPackageSuffix = "-package" |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 34 | const katiSoongOnlyPackageSuffix = "-soong-only-package" |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 35 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 36 | // genKatiSuffix creates a filename suffix for kati-generated files so that we |
| 37 | // can cache them based on their inputs. Such files include the generated Ninja |
| 38 | // files and env.sh environment variable setup files. |
| 39 | // |
| 40 | // The filename suffix should encode all common changes to Kati inputs. |
| 41 | // Currently that includes the TARGET_PRODUCT and kati-processed command line |
| 42 | // arguments. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 43 | func genKatiSuffix(ctx Context, config Config) { |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 44 | targetProduct := "unknown" |
| 45 | if p, err := config.TargetProductOrErr(); err == nil { |
| 46 | targetProduct = p |
| 47 | } |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 48 | // Construct the base suffix. |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 49 | katiSuffix := "-" + targetProduct + config.CoverageSuffix() |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 50 | |
| 51 | // Append kati arguments to the suffix. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 52 | if args := config.KatiArgs(); len(args) > 0 { |
| 53 | katiSuffix += "-" + spaceSlashReplacer.Replace(strings.Join(args, "_")) |
| 54 | } |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 55 | |
| 56 | // If the suffix is too long, replace it with a md5 hash and write a |
| 57 | // file that contains the original suffix. |
| 58 | if len(katiSuffix) > 64 { |
| 59 | shortSuffix := "-" + fmt.Sprintf("%x", md5.Sum([]byte(katiSuffix))) |
| 60 | config.SetKatiSuffix(shortSuffix) |
| 61 | |
| 62 | ctx.Verbosef("Kati ninja suffix too long: %q", katiSuffix) |
| 63 | ctx.Verbosef("Replacing with: %q", shortSuffix) |
| 64 | |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 65 | if err := ioutil.WriteFile(strings.TrimSuffix(config.KatiBuildNinjaFile(), "ninja")+"suf", []byte(katiSuffix), 0777); err != nil { |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 66 | ctx.Println("Error writing suffix file:", err) |
| 67 | } |
| 68 | } else { |
| 69 | config.SetKatiSuffix(katiSuffix) |
| 70 | } |
| 71 | } |
| 72 | |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 73 | func writeValueIfChanged(ctx Context, config Config, dir string, filename string, value string) { |
| 74 | filePath := filepath.Join(dir, filename) |
| 75 | previousValue := "" |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 76 | rawPreviousValue, err := os.ReadFile(filePath) |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 77 | if err == nil { |
| 78 | previousValue = string(rawPreviousValue) |
| 79 | } |
| 80 | |
| 81 | if previousValue != value { |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 82 | if err = os.WriteFile(filePath, []byte(value), 0666); err != nil { |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 83 | ctx.Fatalf("Failed to write: %v", err) |
| 84 | } |
| 85 | } |
| 86 | } |
| 87 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 88 | // Base function to construct and run the Kati command line with additional |
| 89 | // arguments, and a custom function closure to mutate the environment Kati runs |
| 90 | // in. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 91 | func runKati(ctx Context, config Config, extraSuffix string, args []string, envFunc func(*Environment)) { |
Taylor Santiago | 3c16e61 | 2024-05-30 14:41:31 -0700 | [diff] [blame] | 92 | executable := config.KatiBin() |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 93 | // cKati arguments. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 94 | args = append([]string{ |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 95 | // Instead of executing commands directly, generate a Ninja file. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 96 | "--ninja", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 97 | // Generate Ninja files in the output directory. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 98 | "--ninja_dir=" + config.OutDir(), |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 99 | // Filename suffix of the generated Ninja file. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 100 | "--ninja_suffix=" + config.KatiSuffix() + extraSuffix, |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 101 | // Remove common parts at the beginning of a Ninja file, like build_dir, |
| 102 | // local_pool and _kati_always_build_. Allows Kati to be run multiple |
| 103 | // times, with generated Ninja files combined in a single invocation |
| 104 | // using 'include'. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 105 | "--no_ninja_prelude", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 106 | // Support declaring phony outputs in AOSP Ninja. |
Dan Willemsen | 6587bed | 2020-04-18 20:25:59 -0700 | [diff] [blame] | 107 | "--use_ninja_phony_output", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 108 | // Regenerate the Ninja file if environment inputs have changed. e.g. |
| 109 | // CLI flags, .mk file timestamps, env vars, $(wildcard ..) and some |
| 110 | // $(shell ..) results. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 111 | "--regen", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 112 | // Skip '-include' directives starting with the specified path. Used to |
| 113 | // ignore generated .mk files. |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 114 | "--ignore_optional_include=" + filepath.Join(config.OutDir(), "%.P"), |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 115 | // Detect the use of $(shell echo ...). |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 116 | "--detect_android_echo", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 117 | // Colorful ANSI-based warning and error messages. |
Dan Willemsen | c38d366 | 2017-02-24 10:53:23 -0800 | [diff] [blame] | 118 | "--color_warnings", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 119 | // Generate all targets, not just the top level requested ones. |
Dan Willemsen | c38d366 | 2017-02-24 10:53:23 -0800 | [diff] [blame] | 120 | "--gen_all_targets", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 121 | // Use the built-in emulator of GNU find for better file finding |
| 122 | // performance. Used with $(shell find ...). |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 123 | "--use_find_emulator", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 124 | // Fail when the find emulator encounters problems. |
Dan Willemsen | 418420e | 2017-05-30 14:07:45 -0700 | [diff] [blame] | 125 | "--werror_find_emulator", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 126 | // Do not provide any built-in rules. |
Dan Willemsen | d368d6f | 2018-06-15 21:53:18 -0700 | [diff] [blame] | 127 | "--no_builtin_rules", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 128 | // Fail when suffix rules are used. |
Dan Willemsen | d368d6f | 2018-06-15 21:53:18 -0700 | [diff] [blame] | 129 | "--werror_suffix_rules", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 130 | // Fail when a real target depends on a phony target. |
Dan Willemsen | 6097746 | 2019-04-18 09:40:15 -0700 | [diff] [blame] | 131 | "--werror_real_to_phony", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 132 | // Makes real_to_phony checks assume that any top-level or leaf |
| 133 | // dependencies that does *not* have a '/' in it is a phony target. |
Dan Willemsen | cc62890 | 2019-01-24 15:53:06 -0800 | [diff] [blame] | 134 | "--top_level_phony", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 135 | // Fail when a phony target contains slashes. |
| 136 | "--werror_phony_looks_real", |
| 137 | // Fail when writing to a read-only directory. |
| 138 | "--werror_writable", |
| 139 | // Print Kati's internal statistics, such as the number of variables, |
| 140 | // implicit/explicit/suffix rules, and so on. |
Dan Willemsen | 75d2c17 | 2017-10-12 20:46:34 -0700 | [diff] [blame] | 141 | "--kati_stats", |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 142 | }, args...) |
| 143 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 144 | // Generate a minimal Ninja file. |
| 145 | // |
| 146 | // Used for build_test and multiproduct_kati, which runs Kati several |
| 147 | // hundred times for different configurations to test file generation logic. |
| 148 | // These can result in generating Ninja files reaching ~1GB or more, |
| 149 | // resulting in ~hundreds of GBs of writes. |
| 150 | // |
| 151 | // Since we don't care about executing the Ninja files in these test cases, |
| 152 | // generating the Ninja file content wastes time, so skip writing any |
| 153 | // information out with --empty_ninja_file. |
| 154 | // |
| 155 | // From https://github.com/google/kati/commit/87b8da7af2c8bea28b1d8ab17679453d859f96e5 |
Colin Cross | f3bdbcb | 2021-06-01 11:43:55 -0700 | [diff] [blame] | 156 | if config.EmptyNinjaFile() { |
Dan Willemsen | f99915f | 2018-10-25 22:04:42 -0700 | [diff] [blame] | 157 | args = append(args, "--empty_ninja_file") |
| 158 | } |
| 159 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 160 | // Apply 'local_pool' to to all rules that don't specify a pool. |
Colin Cross | 9016b91 | 2019-11-11 14:57:42 -0800 | [diff] [blame] | 161 | if config.UseRemoteBuild() { |
| 162 | args = append(args, "--default_pool=local_pool") |
| 163 | } |
| 164 | |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 165 | cmd := Command(ctx, config, "ckati", executable, args...) |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 166 | |
| 167 | // Set up the nsjail sandbox. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 168 | cmd.Sandbox = katiSandbox |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 169 | |
| 170 | // Set up stdout and stderr. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 171 | pipe, err := cmd.StdoutPipe() |
| 172 | if err != nil { |
| 173 | ctx.Fatalln("Error getting output pipe for ckati:", err) |
| 174 | } |
| 175 | cmd.Stderr = cmd.Stdout |
| 176 | |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 177 | var username string |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 178 | // Pass on various build environment metadata to Kati. |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 179 | if usernameFromEnv, ok := cmd.Environment.Get("BUILD_USERNAME"); !ok { |
| 180 | username = "unknown" |
Dan Willemsen | a17ac96 | 2020-03-10 15:38:57 -0700 | [diff] [blame] | 181 | if u, err := user.Current(); err == nil { |
| 182 | username = u.Username |
| 183 | } else { |
| 184 | ctx.Println("Failed to get current user:", err) |
Dan Willemsen | 71edc8b | 2019-01-02 12:21:18 -0800 | [diff] [blame] | 185 | } |
Dan Willemsen | a17ac96 | 2020-03-10 15:38:57 -0700 | [diff] [blame] | 186 | cmd.Environment.Set("BUILD_USERNAME", username) |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 187 | } else { |
| 188 | username = usernameFromEnv |
Dan Willemsen | 71edc8b | 2019-01-02 12:21:18 -0800 | [diff] [blame] | 189 | } |
| 190 | |
LaMont Jones | 99f1896 | 2024-10-17 11:50:44 -0700 | [diff] [blame] | 191 | // SOONG_USE_PARTIAL_COMPILE may be used in makefiles, but both cases must be supported. |
| 192 | // |
| 193 | // In general, the partial compile features will be implemented in Soong-based rules. We |
| 194 | // also allow them to be used in makefiles. Clear the environment variable when calling |
| 195 | // kati so that we avoid reanalysis when the user changes it. We will pass it to Ninja. |
| 196 | // As a result, rules where we want to allow the developer to toggle the feature ("use |
| 197 | // the partial compile feature" vs "legacy, aka full compile behavior") need to use this |
| 198 | // in the rule, since changing it will not cause reanalysis. |
| 199 | // |
| 200 | // Shell syntax in the rule might look something like this: |
| 201 | // if [[ -n ${SOONG_USE_PARTIAL_COMPILE} ]]; then |
| 202 | // # partial compile behavior |
| 203 | // else |
| 204 | // # legacy behavior |
| 205 | // fi |
| 206 | cmd.Environment.Unset("SOONG_USE_PARTIAL_COMPILE") |
| 207 | |
Jeongik Cha | f2af38d | 2023-05-23 06:59:39 +0900 | [diff] [blame] | 208 | // Unset BUILD_HOSTNAME during kati run to avoid kati rerun, kati will use BUILD_HOSTNAME from a file. |
| 209 | cmd.Environment.Unset("BUILD_HOSTNAME") |
Cole Faust | e14082c | 2025-01-06 15:38:34 -0800 | [diff] [blame] | 210 | |
| 211 | _, ok := cmd.Environment.Get("BUILD_NUMBER") |
Jeongik Cha | f2af38d | 2023-05-23 06:59:39 +0900 | [diff] [blame] | 212 | // Unset BUILD_NUMBER during kati run to avoid kati rerun, kati will use BUILD_NUMBER from a file. |
| 213 | cmd.Environment.Unset("BUILD_NUMBER") |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 214 | if ok { |
| 215 | cmd.Environment.Set("HAS_BUILD_NUMBER", "true") |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 216 | } else { |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 217 | cmd.Environment.Set("HAS_BUILD_NUMBER", "false") |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 218 | } |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 219 | |
| 220 | // Apply the caller's function closure to mutate the environment variables. |
| 221 | envFunc(cmd.Environment) |
Dan Willemsen | 71edc8b | 2019-01-02 12:21:18 -0800 | [diff] [blame] | 222 | |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 223 | cmd.StartOrFatal() |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 224 | // Set up the ToolStatus command line reader for Kati for a consistent UI |
| 225 | // for the user. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 226 | status.KatiReader(ctx.Status.StartTool(), pipe) |
| 227 | cmd.WaitOrFatal() |
| 228 | } |
| 229 | |
| 230 | func runKatiBuild(ctx Context, config Config) { |
Nan Zhang | 17f2767 | 2018-12-12 16:01:49 -0800 | [diff] [blame] | 231 | ctx.BeginTrace(metrics.RunKati, "kati build") |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 232 | defer ctx.EndTrace() |
| 233 | |
| 234 | args := []string{ |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 235 | // Mark the output directory as writable. |
Dan Willemsen | 25a5618 | 2018-08-31 20:25:32 -0700 | [diff] [blame] | 236 | "--writable", config.OutDir() + "/", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 237 | // Fail when encountering implicit rules. e.g. |
| 238 | // %.foo: %.bar |
| 239 | // cp $< $@ |
Dan Willemsen | 9f43597 | 2020-05-28 15:28:00 -0700 | [diff] [blame] | 240 | "--werror_implicit_rules", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 241 | // Entry point for the Kati Ninja file generation. |
Dan Willemsen | b2e6c2e | 2017-07-13 17:24:44 -0700 | [diff] [blame] | 242 | "-f", "build/make/core/main.mk", |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 243 | } |
| 244 | |
Dan Willemsen | 3d60b11 | 2018-04-04 22:25:56 -0700 | [diff] [blame] | 245 | if !config.BuildBrokenDupRules() { |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 246 | // Fail when redefining / duplicating a target. |
Dan Willemsen | 3d60b11 | 2018-04-04 22:25:56 -0700 | [diff] [blame] | 247 | args = append(args, "--werror_overriding_commands") |
| 248 | } |
| 249 | |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 250 | args = append(args, config.KatiArgs()...) |
| 251 | |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 252 | args = append(args, |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 253 | // Location of the Make vars .mk file generated by Soong. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 254 | "SOONG_MAKEVARS_MK="+config.SoongMakeVarsMk(), |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 255 | // Location of the Android.mk file generated by Soong. This |
| 256 | // file contains Soong modules represented as Kati modules, |
| 257 | // allowing Kati modules to depend on Soong modules. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 258 | "SOONG_ANDROID_MK="+config.SoongAndroidMk(), |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 259 | // Directory containing outputs for the target device. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 260 | "TARGET_DEVICE_DIR="+config.TargetDeviceDir(), |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 261 | // Directory containing .mk files for packaging purposes, such as |
| 262 | // the dist.mk file, containing dist-for-goals data. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 263 | "KATI_PACKAGE_MK_DIR="+config.KatiPackageMkDir()) |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 264 | |
Dan Willemsen | 2d31a44 | 2018-10-20 21:33:41 -0700 | [diff] [blame] | 265 | runKati(ctx, config, katiBuildSuffix, args, func(env *Environment) {}) |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 266 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 267 | // compress and dist the main build ninja file. |
Colin Cross | 8ba7d47 | 2020-06-25 11:27:52 -0700 | [diff] [blame] | 268 | distGzipFile(ctx, config, config.KatiBuildNinjaFile()) |
| 269 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 270 | // Cleanup steps. |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 271 | cleanCopyHeaders(ctx, config) |
Colin Cross | 41ad6b6 | 2021-03-09 12:02:15 -0800 | [diff] [blame] | 272 | cleanOldInstalledFiles(ctx, config) |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 273 | } |
| 274 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 275 | // Clean out obsolete header files on the disk that were *not copied* during the |
| 276 | // build with BUILD_COPY_HEADERS and LOCAL_COPY_HEADERS. |
| 277 | // |
| 278 | // These should be increasingly uncommon, as it's a deprecated feature and there |
| 279 | // isn't an equivalent feature in Soong. |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 280 | func cleanCopyHeaders(ctx Context, config Config) { |
| 281 | ctx.BeginTrace("clean", "clean copy headers") |
| 282 | defer ctx.EndTrace() |
| 283 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 284 | // Read and parse the list of copied headers from a file in the product |
| 285 | // output directory. |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 286 | data, err := ioutil.ReadFile(filepath.Join(config.ProductOut(), ".copied_headers_list")) |
| 287 | if err != nil { |
| 288 | if os.IsNotExist(err) { |
| 289 | return |
| 290 | } |
| 291 | ctx.Fatalf("Failed to read copied headers list: %v", err) |
| 292 | } |
| 293 | |
| 294 | headers := strings.Fields(string(data)) |
| 295 | if len(headers) < 1 { |
| 296 | ctx.Fatal("Failed to parse copied headers list: %q", string(data)) |
| 297 | } |
| 298 | headerDir := headers[0] |
| 299 | headers = headers[1:] |
| 300 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 301 | // Walk the tree and remove any headers that are not in the list of copied |
| 302 | // headers in the current build. |
Dan Willemsen | 7842826 | 2019-12-13 18:50:24 -0800 | [diff] [blame] | 303 | filepath.Walk(headerDir, |
| 304 | func(path string, info os.FileInfo, err error) error { |
| 305 | if err != nil { |
| 306 | return nil |
| 307 | } |
| 308 | if info.IsDir() { |
| 309 | return nil |
| 310 | } |
| 311 | if !inList(path, headers) { |
| 312 | ctx.Printf("Removing obsolete header %q", path) |
| 313 | if err := os.Remove(path); err != nil { |
| 314 | ctx.Fatalf("Failed to remove obsolete header %q: %v", path, err) |
| 315 | } |
| 316 | } |
| 317 | return nil |
| 318 | }) |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 319 | } |
| 320 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 321 | // Clean out any previously installed files from the disk that are not installed |
| 322 | // in the current build. |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 323 | func cleanOldInstalledFiles(ctx Context, config Config) { |
| 324 | ctx.BeginTrace("clean", "clean old installed files") |
| 325 | defer ctx.EndTrace() |
| 326 | |
| 327 | // We shouldn't be removing files from one side of the two-step asan builds |
| 328 | var suffix string |
| 329 | if v, ok := config.Environment().Get("SANITIZE_TARGET"); ok { |
| 330 | if sanitize := strings.Fields(v); inList("address", sanitize) { |
| 331 | suffix = "_asan" |
| 332 | } |
| 333 | } |
| 334 | |
| 335 | cleanOldFiles(ctx, config.ProductOut(), ".installable_files"+suffix) |
| 336 | |
| 337 | cleanOldFiles(ctx, config.HostOut(), ".installable_test_files") |
| 338 | } |
| 339 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 340 | // Generate the Ninja file containing the packaging command lines for the dist |
| 341 | // dir. |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 342 | func runKatiPackage(ctx Context, config Config, soongOnly bool) { |
Nan Zhang | 17f2767 | 2018-12-12 16:01:49 -0800 | [diff] [blame] | 343 | ctx.BeginTrace(metrics.RunKati, "kati package") |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 344 | defer ctx.EndTrace() |
| 345 | |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 346 | entryPoint := "build/make/packaging/main.mk" |
| 347 | suffix := katiPackageSuffix |
| 348 | ninjaFile := config.KatiPackageNinjaFile() |
| 349 | if soongOnly { |
| 350 | entryPoint = "build/make/packaging/main_soong_only.mk" |
| 351 | suffix = katiSoongOnlyPackageSuffix |
| 352 | ninjaFile = config.KatiSoongOnlyPackageNinjaFile() |
| 353 | } |
| 354 | |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 355 | args := []string{ |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 356 | // Mark the dist dir as writable. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 357 | "--writable", config.DistDir() + "/", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 358 | // Fail when encountering implicit rules. e.g. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 359 | "--werror_implicit_rules", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 360 | // Fail when redefining / duplicating a target. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 361 | "--werror_overriding_commands", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 362 | // Entry point. |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 363 | "-f", entryPoint, |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 364 | // Directory containing .mk files for packaging purposes, such as |
| 365 | // the dist.mk file, containing dist-for-goals data. |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 366 | "KATI_PACKAGE_MK_DIR=" + config.KatiPackageMkDir(), |
| 367 | } |
| 368 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 369 | // Run Kati against a restricted set of environment variables. |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 370 | runKati(ctx, config, suffix, args, func(env *Environment) { |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 371 | env.Allow([]string{ |
| 372 | // Some generic basics |
| 373 | "LANG", |
| 374 | "LC_MESSAGES", |
| 375 | "PATH", |
| 376 | "PWD", |
| 377 | "TMPDIR", |
| 378 | |
| 379 | // Tool configs |
Dan Willemsen | 70c1ff8 | 2019-08-21 14:56:13 -0700 | [diff] [blame] | 380 | "ASAN_SYMBOLIZER_PATH", |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 381 | "JAVA_HOME", |
| 382 | "PYTHONDONTWRITEBYTECODE", |
| 383 | |
| 384 | // Build configuration |
| 385 | "ANDROID_BUILD_SHELL", |
| 386 | "DIST_DIR", |
| 387 | "OUT_DIR", |
Jeongik Cha | 4e49bbd | 2023-04-26 21:06:24 +0900 | [diff] [blame] | 388 | "FILE_NAME_TAG", |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 389 | }...) |
| 390 | |
| 391 | if config.Dist() { |
| 392 | env.Set("DIST", "true") |
Dan Willemsen | 2d31a44 | 2018-10-20 21:33:41 -0700 | [diff] [blame] | 393 | env.Set("DIST_DIR", config.DistDir()) |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 394 | } |
| 395 | }) |
Colin Cross | 8ba7d47 | 2020-06-25 11:27:52 -0700 | [diff] [blame] | 396 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 397 | // Compress and dist the packaging Ninja file. |
Cole Faust | c5bfbdd | 2025-01-08 13:05:40 -0800 | [diff] [blame] | 398 | distGzipFile(ctx, config, ninjaFile) |
Dan Willemsen | 1e70446 | 2016-08-21 15:17:17 -0700 | [diff] [blame] | 399 | } |
Dan Willemsen | 29f8827 | 2017-02-18 18:12:41 -0800 | [diff] [blame] | 400 | |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 401 | // Run Kati on the cleanspec files to clean the build. |
Dan Willemsen | 59fdf96 | 2017-07-24 22:26:54 -0700 | [diff] [blame] | 402 | func runKatiCleanSpec(ctx Context, config Config) { |
Nan Zhang | 17f2767 | 2018-12-12 16:01:49 -0800 | [diff] [blame] | 403 | ctx.BeginTrace(metrics.RunKati, "kati cleanspec") |
Dan Willemsen | 59fdf96 | 2017-07-24 22:26:54 -0700 | [diff] [blame] | 404 | defer ctx.EndTrace() |
| 405 | |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 406 | runKati(ctx, config, katiCleanspecSuffix, []string{ |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 407 | // Fail when encountering implicit rules. e.g. |
Dan Willemsen | 2997123 | 2018-09-26 14:58:30 -0700 | [diff] [blame] | 408 | "--werror_implicit_rules", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 409 | // Fail when redefining / duplicating a target. |
Dan Willemsen | 3d60b11 | 2018-04-04 22:25:56 -0700 | [diff] [blame] | 410 | "--werror_overriding_commands", |
Jingwen Chen | b1d30d6 | 2020-11-18 02:43:19 -0500 | [diff] [blame] | 411 | // Entry point. |
Dan Willemsen | 59fdf96 | 2017-07-24 22:26:54 -0700 | [diff] [blame] | 412 | "-f", "build/make/core/cleanbuild.mk", |
Dan Willemsen | fb1271a | 2018-09-26 15:00:42 -0700 | [diff] [blame] | 413 | "SOONG_MAKEVARS_MK=" + config.SoongMakeVarsMk(), |
| 414 | "TARGET_DEVICE_DIR=" + config.TargetDeviceDir(), |
Dan Willemsen | 2d31a44 | 2018-10-20 21:33:41 -0700 | [diff] [blame] | 415 | }, func(env *Environment) {}) |
Dan Willemsen | 59fdf96 | 2017-07-24 22:26:54 -0700 | [diff] [blame] | 416 | } |