Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 1 | // Copyright 2016 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 | |
Colin Cross | 635c3b0 | 2016-05-18 15:37:25 -0700 | [diff] [blame] | 15 | package android |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 16 | |
| 17 | import ( |
| 18 | "bytes" |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 19 | "cmp" |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 20 | "fmt" |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 21 | "path/filepath" |
| 22 | "runtime" |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 23 | "slices" |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 24 | "sort" |
Colin Cross | 3165695 | 2018-05-24 16:11:20 -0700 | [diff] [blame] | 25 | "strings" |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 26 | |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 27 | "github.com/google/blueprint" |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 28 | "github.com/google/blueprint/pathtools" |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 29 | "github.com/google/blueprint/proptools" |
| 30 | ) |
| 31 | |
Dan Albert | f5415d7 | 2017-08-17 16:19:59 -0700 | [diff] [blame] | 32 | func init() { |
| 33 | RegisterMakeVarsProvider(pctx, androidMakeVarsProvider) |
| 34 | } |
| 35 | |
| 36 | func androidMakeVarsProvider(ctx MakeVarsContext) { |
Dan Albert | 1a24627 | 2020-07-06 14:49:35 -0700 | [diff] [blame] | 37 | ctx.Strict("MIN_SUPPORTED_SDK_VERSION", ctx.Config().MinSupportedSdkVersion().String()) |
Dan Albert | f5415d7 | 2017-08-17 16:19:59 -0700 | [diff] [blame] | 38 | } |
| 39 | |
Paul Duffin | 8eb4573 | 2022-10-04 19:03:31 +0100 | [diff] [blame] | 40 | // ///////////////////////////////////////////////////////////////////////////// |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 41 | |
| 42 | // BaseMakeVarsContext contains the common functions for other packages to use |
| 43 | // to declare make variables |
| 44 | type BaseMakeVarsContext interface { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 45 | Config() Config |
Dan Willemsen | 3fb1fae | 2018-03-12 15:30:26 -0700 | [diff] [blame] | 46 | DeviceConfig() DeviceConfig |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 47 | AddNinjaFileDeps(deps ...string) |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 48 | |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 49 | Failed() bool |
| 50 | |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 51 | // These are equivalent to Strict and Check, but do not attempt to |
| 52 | // evaluate the values before writing them to the Makefile. They can |
| 53 | // be used when all ninja variables have already been evaluated through |
| 54 | // Eval(). |
| 55 | StrictRaw(name, value string) |
| 56 | CheckRaw(name, value string) |
Colin Cross | 8177ad2 | 2019-11-04 10:27:48 -0800 | [diff] [blame] | 57 | |
| 58 | // GlobWithDeps returns a list of files that match the specified pattern but do not match any |
| 59 | // of the patterns in excludes. It also adds efficient dependencies to rerun the primary |
| 60 | // builder whenever a file matching the pattern as added or removed, without rerunning if a |
| 61 | // file that does not match the pattern is added to a searched directory. |
| 62 | GlobWithDeps(pattern string, excludes []string) ([]string, error) |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 63 | |
| 64 | // Phony creates a phony rule in Make, which will allow additional DistForGoal |
| 65 | // dependencies to be added to it. Phony can be called on the same name multiple |
| 66 | // times to add additional dependencies. |
| 67 | Phony(names string, deps ...Path) |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 68 | |
| 69 | // DistForGoal creates a rule to copy one or more Paths to the artifacts |
| 70 | // directory on the build server when the specified goal is built. |
| 71 | DistForGoal(goal string, paths ...Path) |
| 72 | |
| 73 | // DistForGoalWithFilename creates a rule to copy a Path to the artifacts |
| 74 | // directory on the build server with the given filename when the specified |
| 75 | // goal is built. |
| 76 | DistForGoalWithFilename(goal string, path Path, filename string) |
| 77 | |
| 78 | // DistForGoals creates a rule to copy one or more Paths to the artifacts |
| 79 | // directory on the build server when any of the specified goals are built. |
| 80 | DistForGoals(goals []string, paths ...Path) |
| 81 | |
| 82 | // DistForGoalsWithFilename creates a rule to copy a Path to the artifacts |
| 83 | // directory on the build server with the given filename when any of the |
| 84 | // specified goals are built. |
| 85 | DistForGoalsWithFilename(goals []string, path Path, filename string) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 86 | } |
| 87 | |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 88 | // MakeVarsContext contains the set of functions available for MakeVarsProvider |
| 89 | // and SingletonMakeVarsProvider implementations. |
| 90 | type MakeVarsContext interface { |
| 91 | BaseMakeVarsContext |
| 92 | |
| 93 | ModuleName(module blueprint.Module) string |
| 94 | ModuleDir(module blueprint.Module) string |
| 95 | ModuleSubDir(module blueprint.Module) string |
| 96 | ModuleType(module blueprint.Module) string |
Colin Cross | 3c0a83d | 2023-12-12 14:13:26 -0800 | [diff] [blame] | 97 | moduleProvider(module blueprint.Module, key blueprint.AnyProviderKey) (any, bool) |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 98 | BlueprintFile(module blueprint.Module) string |
| 99 | |
| 100 | ModuleErrorf(module blueprint.Module, format string, args ...interface{}) |
| 101 | Errorf(format string, args ...interface{}) |
| 102 | |
| 103 | VisitAllModules(visit func(Module)) |
| 104 | VisitAllModulesIf(pred func(Module) bool, visit func(Module)) |
| 105 | |
| 106 | // Verify the make variable matches the Soong version, fail the build |
| 107 | // if it does not. If the make variable is empty, just set it. |
| 108 | Strict(name, ninjaStr string) |
| 109 | // Check to see if the make variable matches the Soong version, warn if |
| 110 | // it does not. If the make variable is empty, just set it. |
| 111 | Check(name, ninjaStr string) |
| 112 | |
| 113 | // These are equivalent to the above, but sort the make and soong |
| 114 | // variables before comparing them. They also show the unique entries |
| 115 | // in each list when displaying the difference, instead of the entire |
| 116 | // string. |
| 117 | StrictSorted(name, ninjaStr string) |
| 118 | CheckSorted(name, ninjaStr string) |
| 119 | |
| 120 | // Evaluates a ninja string and returns the result. Used if more |
| 121 | // complicated modification needs to happen before giving it to Make. |
| 122 | Eval(ninjaStr string) (string, error) |
| 123 | } |
| 124 | |
| 125 | // MakeVarsModuleContext contains the set of functions available for modules |
| 126 | // implementing the ModuleMakeVarsProvider interface. |
| 127 | type MakeVarsModuleContext interface { |
| 128 | BaseMakeVarsContext |
| 129 | } |
| 130 | |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 131 | var _ PathContext = MakeVarsContext(nil) |
| 132 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 133 | type MakeVarsProvider func(ctx MakeVarsContext) |
| 134 | |
Colin Cross | 0875c52 | 2017-11-28 17:34:01 -0800 | [diff] [blame] | 135 | func RegisterMakeVarsProvider(pctx PackageContext, provider MakeVarsProvider) { |
Chris Parsons | f3c96ef | 2020-09-29 02:23:17 -0400 | [diff] [blame] | 136 | makeVarsInitProviders = append(makeVarsInitProviders, makeVarsProvider{pctx, provider}) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 137 | } |
| 138 | |
Colin Cross | ed023ec | 2019-02-19 12:38:45 -0800 | [diff] [blame] | 139 | // SingletonMakeVarsProvider is a Singleton with an extra method to provide extra values to be exported to Make. |
| 140 | type SingletonMakeVarsProvider interface { |
Colin Cross | ed023ec | 2019-02-19 12:38:45 -0800 | [diff] [blame] | 141 | // MakeVars uses a MakeVarsContext to provide extra values to be exported to Make. |
| 142 | MakeVars(ctx MakeVarsContext) |
| 143 | } |
| 144 | |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 145 | var singletonMakeVarsProvidersKey = NewOnceKey("singletonMakeVarsProvidersKey") |
| 146 | |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 147 | func getSingletonMakevarsProviders(config Config) *[]makeVarsProvider { |
| 148 | return config.Once(singletonMakeVarsProvidersKey, func() interface{} { |
| 149 | return &[]makeVarsProvider{} |
| 150 | }).(*[]makeVarsProvider) |
| 151 | } |
| 152 | |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 153 | // registerSingletonMakeVarsProvider adds a singleton that implements SingletonMakeVarsProvider to |
| 154 | // the list of MakeVarsProviders to run. |
| 155 | func registerSingletonMakeVarsProvider(config Config, singleton SingletonMakeVarsProvider) { |
| 156 | // Singletons are registered on the Context and may be different between different Contexts, |
| 157 | // for example when running multiple tests. Store the SingletonMakeVarsProviders in the |
| 158 | // Config so they are attached to the Context. |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 159 | singletonMakeVarsProviders := getSingletonMakevarsProviders(config) |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 160 | |
| 161 | *singletonMakeVarsProviders = append(*singletonMakeVarsProviders, |
| 162 | makeVarsProvider{pctx, singletonMakeVarsProviderAdapter(singleton)}) |
Colin Cross | ed023ec | 2019-02-19 12:38:45 -0800 | [diff] [blame] | 163 | } |
| 164 | |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 165 | // singletonMakeVarsProviderAdapter converts a SingletonMakeVarsProvider to a MakeVarsProvider. |
| 166 | func singletonMakeVarsProviderAdapter(singleton SingletonMakeVarsProvider) MakeVarsProvider { |
Colin Cross | ed023ec | 2019-02-19 12:38:45 -0800 | [diff] [blame] | 167 | return func(ctx MakeVarsContext) { singleton.MakeVars(ctx) } |
| 168 | } |
| 169 | |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 170 | // ModuleMakeVarsProvider is a Module with an extra method to provide extra values to be exported to Make. |
| 171 | type ModuleMakeVarsProvider interface { |
| 172 | Module |
| 173 | |
| 174 | // MakeVars uses a MakeVarsModuleContext to provide extra values to be exported to Make. |
| 175 | MakeVars(ctx MakeVarsModuleContext) |
| 176 | } |
| 177 | |
Paul Duffin | 8eb4573 | 2022-10-04 19:03:31 +0100 | [diff] [blame] | 178 | // ///////////////////////////////////////////////////////////////////////////// |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 179 | |
Colin Cross | 0875c52 | 2017-11-28 17:34:01 -0800 | [diff] [blame] | 180 | func makeVarsSingletonFunc() Singleton { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 181 | return &makeVarsSingleton{} |
| 182 | } |
| 183 | |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 184 | type makeVarsSingleton struct { |
Paul Duffin | 8eb4573 | 2022-10-04 19:03:31 +0100 | [diff] [blame] | 185 | varsForTesting []makeVarsVariable |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 186 | installsForTesting []byte |
| 187 | } |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 188 | |
| 189 | type makeVarsProvider struct { |
Colin Cross | 0875c52 | 2017-11-28 17:34:01 -0800 | [diff] [blame] | 190 | pctx PackageContext |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 191 | call MakeVarsProvider |
| 192 | } |
| 193 | |
Chris Parsons | f3c96ef | 2020-09-29 02:23:17 -0400 | [diff] [blame] | 194 | // Collection of makevars providers that are registered in init() methods. |
| 195 | var makeVarsInitProviders []makeVarsProvider |
| 196 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 197 | type makeVarsContext struct { |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 198 | SingletonContext |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 199 | config Config |
| 200 | pctx PackageContext |
| 201 | vars []makeVarsVariable |
| 202 | phonies []phony |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 203 | dists []dist |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 204 | } |
| 205 | |
| 206 | var _ MakeVarsContext = &makeVarsContext{} |
| 207 | |
| 208 | type makeVarsVariable struct { |
| 209 | name string |
| 210 | value string |
| 211 | sort bool |
| 212 | strict bool |
| 213 | } |
| 214 | |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 215 | type phony struct { |
| 216 | name string |
| 217 | deps []string |
| 218 | } |
| 219 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 220 | type dist struct { |
| 221 | goals []string |
| 222 | paths []string |
| 223 | } |
| 224 | |
Colin Cross | 0875c52 | 2017-11-28 17:34:01 -0800 | [diff] [blame] | 225 | func (s *makeVarsSingleton) GenerateBuildActions(ctx SingletonContext) { |
Jingwen Chen | cda22c9 | 2020-11-23 00:22:30 -0500 | [diff] [blame] | 226 | if !ctx.Config().KatiEnabled() { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 227 | return |
| 228 | } |
| 229 | |
Colin Cross | 988414c | 2020-01-11 01:11:46 +0000 | [diff] [blame] | 230 | outFile := absolutePath(PathForOutput(ctx, |
| 231 | "make_vars"+proptools.String(ctx.Config().productVariables.Make_suffix)+".mk").String()) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 232 | |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 233 | lateOutFile := absolutePath(PathForOutput(ctx, |
| 234 | "late"+proptools.String(ctx.Config().productVariables.Make_suffix)+".mk").String()) |
| 235 | |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 236 | installsFile := absolutePath(PathForOutput(ctx, |
| 237 | "installs"+proptools.String(ctx.Config().productVariables.Make_suffix)+".mk").String()) |
| 238 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 239 | if ctx.Failed() { |
| 240 | return |
| 241 | } |
| 242 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 243 | var vars []makeVarsVariable |
| 244 | var dists []dist |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 245 | var phonies []phony |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 246 | var katiInstalls []katiInstall |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 247 | var katiInitRcInstalls []katiInstall |
| 248 | var katiVintfManifestInstalls []katiInstall |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 249 | var katiSymlinks []katiInstall |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 250 | |
| 251 | providers := append([]makeVarsProvider(nil), makeVarsInitProviders...) |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 252 | providers = append(providers, *getSingletonMakevarsProviders(ctx.Config())...) |
Colin Cross | 06fa588 | 2020-10-29 18:21:38 -0700 | [diff] [blame] | 253 | |
| 254 | for _, provider := range providers { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 255 | mctx := &makeVarsContext{ |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 256 | SingletonContext: ctx, |
| 257 | pctx: provider.pctx, |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 258 | } |
| 259 | |
| 260 | provider.call(mctx) |
| 261 | |
| 262 | vars = append(vars, mctx.vars...) |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 263 | phonies = append(phonies, mctx.phonies...) |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 264 | dists = append(dists, mctx.dists...) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 265 | } |
| 266 | |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 267 | ctx.VisitAllModules(func(m Module) { |
Jiyong Park | f78531b | 2020-09-09 17:14:28 +0900 | [diff] [blame] | 268 | if provider, ok := m.(ModuleMakeVarsProvider); ok && m.Enabled() { |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 269 | mctx := &makeVarsContext{ |
| 270 | SingletonContext: ctx, |
| 271 | } |
| 272 | |
| 273 | provider.MakeVars(mctx) |
| 274 | |
| 275 | vars = append(vars, mctx.vars...) |
| 276 | phonies = append(phonies, mctx.phonies...) |
| 277 | dists = append(dists, mctx.dists...) |
| 278 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 279 | |
| 280 | if m.ExportedToMake() { |
| 281 | katiInstalls = append(katiInstalls, m.base().katiInstalls...) |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 282 | katiInitRcInstalls = append(katiInitRcInstalls, m.base().katiInitRcInstalls...) |
| 283 | katiVintfManifestInstalls = append(katiVintfManifestInstalls, m.base().katiVintfInstalls...) |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 284 | katiSymlinks = append(katiSymlinks, m.base().katiSymlinks...) |
| 285 | } |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 286 | }) |
| 287 | |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 288 | compareKatiInstalls := func(a, b katiInstall) int { |
| 289 | aTo, bTo := a.to.String(), b.to.String() |
| 290 | if cmpTo := cmp.Compare(aTo, bTo); cmpTo != 0 { |
| 291 | return cmpTo |
| 292 | } |
| 293 | |
| 294 | aFrom, bFrom := a.from.String(), b.from.String() |
| 295 | return cmp.Compare(aFrom, bFrom) |
| 296 | } |
| 297 | |
| 298 | slices.SortFunc(katiInitRcInstalls, compareKatiInstalls) |
| 299 | katiInitRcInstalls = slices.CompactFunc(katiInitRcInstalls, func(a, b katiInstall) bool { |
| 300 | return compareKatiInstalls(a, b) == 0 |
| 301 | }) |
| 302 | katiInstalls = append(katiInstalls, katiInitRcInstalls...) |
| 303 | |
| 304 | slices.SortFunc(katiVintfManifestInstalls, compareKatiInstalls) |
| 305 | katiVintfManifestInstalls = slices.CompactFunc(katiVintfManifestInstalls, func(a, b katiInstall) bool { |
| 306 | return compareKatiInstalls(a, b) == 0 |
| 307 | }) |
| 308 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 309 | if ctx.Failed() { |
| 310 | return |
| 311 | } |
| 312 | |
Dan Willemsen | 6a6478d | 2020-07-17 19:28:53 -0700 | [diff] [blame] | 313 | sort.Slice(vars, func(i, j int) bool { |
| 314 | return vars[i].name < vars[j].name |
| 315 | }) |
| 316 | sort.Slice(phonies, func(i, j int) bool { |
| 317 | return phonies[i].name < phonies[j].name |
| 318 | }) |
| 319 | lessArr := func(a, b []string) bool { |
| 320 | if len(a) == len(b) { |
| 321 | for i := range a { |
| 322 | if a[i] < b[i] { |
| 323 | return true |
| 324 | } |
| 325 | } |
| 326 | return false |
| 327 | } |
| 328 | return len(a) < len(b) |
| 329 | } |
| 330 | sort.Slice(dists, func(i, j int) bool { |
| 331 | return lessArr(dists[i].goals, dists[j].goals) || lessArr(dists[i].paths, dists[j].paths) |
| 332 | }) |
| 333 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 334 | outBytes := s.writeVars(vars) |
| 335 | |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 336 | if err := pathtools.WriteFileIfChanged(outFile, outBytes, 0666); err != nil { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 337 | ctx.Errorf(err.Error()) |
| 338 | } |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 339 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 340 | lateOutBytes := s.writeLate(phonies, dists) |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 341 | |
| 342 | if err := pathtools.WriteFileIfChanged(lateOutFile, lateOutBytes, 0666); err != nil { |
| 343 | ctx.Errorf(err.Error()) |
| 344 | } |
| 345 | |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 346 | installsBytes := s.writeInstalls(katiInstalls, katiSymlinks, katiVintfManifestInstalls) |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 347 | if err := pathtools.WriteFileIfChanged(installsFile, installsBytes, 0666); err != nil { |
| 348 | ctx.Errorf(err.Error()) |
| 349 | } |
Colin Cross | 6ac9576 | 2021-11-09 13:17:44 -0800 | [diff] [blame] | 350 | |
Paul Duffin | 8eb4573 | 2022-10-04 19:03:31 +0100 | [diff] [blame] | 351 | // Only save state for tests when testing. |
| 352 | if ctx.Config().RunningInsideUnitTest() { |
| 353 | s.varsForTesting = vars |
| 354 | s.installsForTesting = installsBytes |
| 355 | } |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 356 | } |
| 357 | |
| 358 | func (s *makeVarsSingleton) writeVars(vars []makeVarsVariable) []byte { |
| 359 | buf := &bytes.Buffer{} |
| 360 | |
Dan Willemsen | 59339a2 | 2018-07-22 21:18:45 -0700 | [diff] [blame] | 361 | fmt.Fprint(buf, `# Autogenerated file |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 362 | |
| 363 | # Compares SOONG_$(1) against $(1), and warns if they are not equal. |
| 364 | # |
| 365 | # If the original variable is empty, then just set it to the SOONG_ version. |
| 366 | # |
| 367 | # $(1): Name of the variable to check |
| 368 | # $(2): If not-empty, sort the values before comparing |
| 369 | # $(3): Extra snippet to run if it does not match |
| 370 | define soong-compare-var |
| 371 | ifneq ($$($(1)),) |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 372 | my_val_make := $$(strip $(if $(2),$$(sort $$($(1))),$$($(1)))) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 373 | my_val_soong := $(if $(2),$$(sort $$(SOONG_$(1))),$$(SOONG_$(1))) |
| 374 | ifneq ($$(my_val_make),$$(my_val_soong)) |
| 375 | $$(warning $(1) does not match between Make and Soong:) |
| 376 | $(if $(2),$$(warning Make adds: $$(filter-out $$(my_val_soong),$$(my_val_make))),$$(warning Make : $$(my_val_make))) |
| 377 | $(if $(2),$$(warning Soong adds: $$(filter-out $$(my_val_make),$$(my_val_soong))),$$(warning Soong: $$(my_val_soong))) |
| 378 | $(3) |
| 379 | endif |
| 380 | my_val_make := |
| 381 | my_val_soong := |
| 382 | else |
| 383 | $(1) := $$(SOONG_$(1)) |
| 384 | endif |
Dan Willemsen | de18f47 | 2016-09-30 10:16:38 -0700 | [diff] [blame] | 385 | .KATI_READONLY := $(1) SOONG_$(1) |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 386 | endef |
| 387 | |
| 388 | my_check_failed := false |
| 389 | |
| 390 | `) |
| 391 | |
| 392 | // Write all the strict checks out first so that if one of them errors, |
| 393 | // we get all of the strict errors printed, but not the non-strict |
| 394 | // warnings. |
| 395 | for _, v := range vars { |
| 396 | if !v.strict { |
| 397 | continue |
| 398 | } |
| 399 | |
| 400 | sort := "" |
| 401 | if v.sort { |
| 402 | sort = "true" |
| 403 | } |
| 404 | |
| 405 | fmt.Fprintf(buf, "SOONG_%s := %s\n", v.name, v.value) |
| 406 | fmt.Fprintf(buf, "$(eval $(call soong-compare-var,%s,%s,my_check_failed := true))\n\n", v.name, sort) |
| 407 | } |
| 408 | |
Dan Willemsen | 59339a2 | 2018-07-22 21:18:45 -0700 | [diff] [blame] | 409 | fmt.Fprint(buf, ` |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 410 | ifneq ($(my_check_failed),false) |
| 411 | $(error Soong variable check failed) |
| 412 | endif |
| 413 | my_check_failed := |
| 414 | |
| 415 | |
| 416 | `) |
| 417 | |
| 418 | for _, v := range vars { |
| 419 | if v.strict { |
| 420 | continue |
| 421 | } |
| 422 | |
| 423 | sort := "" |
| 424 | if v.sort { |
| 425 | sort = "true" |
| 426 | } |
| 427 | |
| 428 | fmt.Fprintf(buf, "SOONG_%s := %s\n", v.name, v.value) |
| 429 | fmt.Fprintf(buf, "$(eval $(call soong-compare-var,%s,%s))\n\n", v.name, sort) |
| 430 | } |
| 431 | |
| 432 | fmt.Fprintln(buf, "\nsoong-compare-var :=") |
| 433 | |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 434 | fmt.Fprintln(buf) |
| 435 | |
| 436 | return buf.Bytes() |
| 437 | } |
| 438 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 439 | func (s *makeVarsSingleton) writeLate(phonies []phony, dists []dist) []byte { |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 440 | buf := &bytes.Buffer{} |
| 441 | |
| 442 | fmt.Fprint(buf, `# Autogenerated file |
| 443 | |
| 444 | # Values written by Soong read after parsing all Android.mk files. |
| 445 | |
| 446 | |
| 447 | `) |
| 448 | |
| 449 | for _, phony := range phonies { |
| 450 | fmt.Fprintf(buf, ".PHONY: %s\n", phony.name) |
| 451 | fmt.Fprintf(buf, "%s: %s\n", phony.name, strings.Join(phony.deps, "\\\n ")) |
| 452 | } |
| 453 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 454 | fmt.Fprintln(buf) |
| 455 | |
| 456 | for _, dist := range dists { |
Colin Cross | b5399c8 | 2021-11-11 16:33:24 -0800 | [diff] [blame] | 457 | fmt.Fprintf(buf, ".PHONY: %s\n", strings.Join(dist.goals, " ")) |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 458 | fmt.Fprintf(buf, "$(call dist-for-goals,%s,%s)\n", |
| 459 | strings.Join(dist.goals, " "), strings.Join(dist.paths, " ")) |
| 460 | } |
| 461 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 462 | return buf.Bytes() |
| 463 | } |
| 464 | |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 465 | // writeInstalls writes the list of install rules generated by Soong to a makefile. The rules |
| 466 | // are exported to Make instead of written directly to the ninja file so that main.mk can add |
| 467 | // the dependencies from the `required` property that are hard to resolve in Soong. |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 468 | func (s *makeVarsSingleton) writeInstalls(installs, symlinks, katiVintfManifestInstalls []katiInstall) []byte { |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 469 | buf := &bytes.Buffer{} |
| 470 | |
| 471 | fmt.Fprint(buf, `# Autogenerated file |
| 472 | |
| 473 | # Values written by Soong to generate install rules that can be amended by Kati. |
| 474 | |
| 475 | |
| 476 | `) |
| 477 | |
| 478 | preserveSymlinksFlag := "-d" |
| 479 | if runtime.GOOS == "darwin" { |
| 480 | preserveSymlinksFlag = "-R" |
| 481 | } |
| 482 | |
| 483 | for _, install := range installs { |
| 484 | // Write a rule for each install request in the form: |
| 485 | // to: from [ deps ] [ | order only deps ] |
| 486 | // cp -f -d $< $@ [ && chmod +x $@ ] |
| 487 | fmt.Fprintf(buf, "%s: %s", install.to.String(), install.from.String()) |
| 488 | for _, dep := range install.implicitDeps { |
| 489 | fmt.Fprintf(buf, " %s", dep.String()) |
| 490 | } |
Colin Cross | 50ed1f9 | 2021-11-12 17:41:02 -0800 | [diff] [blame] | 491 | if extraFiles := install.extraFiles; extraFiles != nil { |
| 492 | fmt.Fprintf(buf, " %s", extraFiles.zip.String()) |
| 493 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 494 | if len(install.orderOnlyDeps) > 0 { |
| 495 | fmt.Fprintf(buf, " |") |
| 496 | } |
| 497 | for _, dep := range install.orderOnlyDeps { |
| 498 | fmt.Fprintf(buf, " %s", dep.String()) |
| 499 | } |
| 500 | fmt.Fprintln(buf) |
Alessandro Astone | 2b17a23 | 2022-10-25 11:44:59 +0200 | [diff] [blame] | 501 | fmt.Fprintln(buf, "\t@echo \"Install: $@\"") |
Colin Cross | 50ed1f9 | 2021-11-12 17:41:02 -0800 | [diff] [blame] | 502 | fmt.Fprintf(buf, "\trm -f $@ && cp -f %s $< $@\n", preserveSymlinksFlag) |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 503 | if install.executable { |
Colin Cross | 50ed1f9 | 2021-11-12 17:41:02 -0800 | [diff] [blame] | 504 | fmt.Fprintf(buf, "\tchmod +x $@\n") |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 505 | } |
Colin Cross | 50ed1f9 | 2021-11-12 17:41:02 -0800 | [diff] [blame] | 506 | if extraFiles := install.extraFiles; extraFiles != nil { |
Romain Jobredeaux | 1cef629 | 2022-05-19 11:11:51 -0400 | [diff] [blame] | 507 | fmt.Fprintf(buf, "\t( unzip -qDD -d '%s' '%s' 2>&1 | grep -v \"zipfile is empty\"; exit $${PIPESTATUS[0]} ) || \\\n", extraFiles.dir.String(), extraFiles.zip.String()) |
| 508 | fmt.Fprintf(buf, "\t ( code=$$?; if [ $$code -ne 0 -a $$code -ne 1 ]; then exit $$code; fi )\n") |
Colin Cross | 50ed1f9 | 2021-11-12 17:41:02 -0800 | [diff] [blame] | 509 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 510 | fmt.Fprintln(buf) |
| 511 | } |
| 512 | |
| 513 | for _, symlink := range symlinks { |
| 514 | fmt.Fprintf(buf, "%s:", symlink.to.String()) |
Colin Cross | 64002af | 2021-11-09 16:37:52 -0800 | [diff] [blame] | 515 | if symlink.from != nil { |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 516 | // The katiVintfManifestInstall doesn't need updating when the target is modified, but we sometimes |
| 517 | // have a dependency on a katiVintfManifestInstall to a binary instead of to the binary directly, and |
| 518 | // the mtime of the katiVintfManifestInstall must be updated when the binary is modified, so use a |
Colin Cross | 64002af | 2021-11-09 16:37:52 -0800 | [diff] [blame] | 519 | // normal dependency here instead of an order-only dependency. |
| 520 | fmt.Fprintf(buf, " %s", symlink.from.String()) |
| 521 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 522 | for _, dep := range symlink.implicitDeps { |
| 523 | fmt.Fprintf(buf, " %s", dep.String()) |
| 524 | } |
Colin Cross | 64002af | 2021-11-09 16:37:52 -0800 | [diff] [blame] | 525 | if len(symlink.orderOnlyDeps) > 0 { |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 526 | fmt.Fprintf(buf, " |") |
| 527 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 528 | for _, dep := range symlink.orderOnlyDeps { |
| 529 | fmt.Fprintf(buf, " %s", dep.String()) |
| 530 | } |
| 531 | fmt.Fprintln(buf) |
| 532 | |
| 533 | fromStr := "" |
| 534 | if symlink.from != nil { |
| 535 | rel, err := filepath.Rel(filepath.Dir(symlink.to.String()), symlink.from.String()) |
| 536 | if err != nil { |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 537 | panic(fmt.Errorf("failed to find relative path for katiVintfManifestInstall from %q to %q: %w", |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 538 | symlink.from.String(), symlink.to.String(), err)) |
| 539 | } |
| 540 | fromStr = rel |
| 541 | } else { |
| 542 | fromStr = symlink.absFrom |
| 543 | } |
| 544 | |
Alessandro Astone | 2b17a23 | 2022-10-25 11:44:59 +0200 | [diff] [blame] | 545 | fmt.Fprintln(buf, "\t@echo \"Symlink: $@\"") |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 546 | fmt.Fprintf(buf, "\trm -f $@ && ln -sfn %s $@", fromStr) |
| 547 | fmt.Fprintln(buf) |
| 548 | fmt.Fprintln(buf) |
| 549 | } |
| 550 | |
Colin Cross | d9bbf4b | 2023-11-17 16:23:48 -0800 | [diff] [blame] | 551 | for _, install := range katiVintfManifestInstalls { |
| 552 | // Write a rule for each vintf install request that calls the copy-vintf-manifest-chedk make function. |
| 553 | fmt.Fprintf(buf, "$(eval $(call copy-vintf-manifest-checked, %s, %s))\n", install.from.String(), install.to.String()) |
| 554 | |
| 555 | if len(install.implicitDeps) > 0 { |
| 556 | panic(fmt.Errorf("unsupported implicitDeps %q in vintf install rule %q", install.implicitDeps, install.to)) |
| 557 | } |
| 558 | if len(install.orderOnlyDeps) > 0 { |
| 559 | panic(fmt.Errorf("unsupported orderOnlyDeps %q in vintf install rule %q", install.orderOnlyDeps, install.to)) |
| 560 | } |
| 561 | |
| 562 | fmt.Fprintln(buf) |
| 563 | } |
Colin Cross | 6301c3c | 2021-09-28 17:40:21 -0700 | [diff] [blame] | 564 | return buf.Bytes() |
| 565 | } |
| 566 | |
Dan Willemsen | 3fb1fae | 2018-03-12 15:30:26 -0700 | [diff] [blame] | 567 | func (c *makeVarsContext) DeviceConfig() DeviceConfig { |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 568 | return DeviceConfig{c.Config().deviceConfig} |
Jiyong Park | 374510b | 2018-03-19 18:23:01 +0900 | [diff] [blame] | 569 | } |
| 570 | |
Colin Cross | 3165695 | 2018-05-24 16:11:20 -0700 | [diff] [blame] | 571 | var ninjaDescaper = strings.NewReplacer("$$", "$") |
| 572 | |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 573 | func (c *makeVarsContext) Eval(ninjaStr string) (string, error) { |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 574 | s, err := c.SingletonContext.Eval(c.pctx, ninjaStr) |
Colin Cross | 3165695 | 2018-05-24 16:11:20 -0700 | [diff] [blame] | 575 | if err != nil { |
| 576 | return "", err |
| 577 | } |
| 578 | // SingletonContext.Eval returns an exapnded string that is valid for a ninja file, de-escape $$ to $ for use |
| 579 | // in a Makefile |
| 580 | return ninjaDescaper.Replace(s), nil |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 581 | } |
| 582 | |
| 583 | func (c *makeVarsContext) addVariableRaw(name, value string, strict, sort bool) { |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 584 | c.vars = append(c.vars, makeVarsVariable{ |
| 585 | name: name, |
| 586 | value: value, |
| 587 | strict: strict, |
| 588 | sort: sort, |
| 589 | }) |
| 590 | } |
| 591 | |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 592 | func (c *makeVarsContext) addVariable(name, ninjaStr string, strict, sort bool) { |
| 593 | value, err := c.Eval(ninjaStr) |
| 594 | if err != nil { |
Colin Cross | 65494b9 | 2019-02-07 14:25:51 -0800 | [diff] [blame] | 595 | c.SingletonContext.Errorf(err.Error()) |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 596 | } |
| 597 | c.addVariableRaw(name, value, strict, sort) |
| 598 | } |
| 599 | |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 600 | func (c *makeVarsContext) addPhony(name string, deps []string) { |
| 601 | c.phonies = append(c.phonies, phony{name, deps}) |
| 602 | } |
| 603 | |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 604 | func (c *makeVarsContext) addDist(goals []string, paths []string) { |
| 605 | c.dists = append(c.dists, dist{ |
| 606 | goals: goals, |
| 607 | paths: paths, |
| 608 | }) |
| 609 | } |
| 610 | |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 611 | func (c *makeVarsContext) Strict(name, ninjaStr string) { |
| 612 | c.addVariable(name, ninjaStr, true, false) |
| 613 | } |
| 614 | func (c *makeVarsContext) StrictSorted(name, ninjaStr string) { |
| 615 | c.addVariable(name, ninjaStr, true, true) |
| 616 | } |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 617 | func (c *makeVarsContext) StrictRaw(name, value string) { |
| 618 | c.addVariableRaw(name, value, true, false) |
| 619 | } |
Dan Willemsen | 4b7d5de | 2016-01-12 23:20:28 -0800 | [diff] [blame] | 620 | |
| 621 | func (c *makeVarsContext) Check(name, ninjaStr string) { |
| 622 | c.addVariable(name, ninjaStr, false, false) |
| 623 | } |
| 624 | func (c *makeVarsContext) CheckSorted(name, ninjaStr string) { |
| 625 | c.addVariable(name, ninjaStr, false, true) |
| 626 | } |
Dan Willemsen | 558e517 | 2016-05-19 16:58:46 -0700 | [diff] [blame] | 627 | func (c *makeVarsContext) CheckRaw(name, value string) { |
| 628 | c.addVariableRaw(name, value, false, false) |
| 629 | } |
Colin Cross | c3d87d3 | 2020-06-04 13:25:17 -0700 | [diff] [blame] | 630 | |
| 631 | func (c *makeVarsContext) Phony(name string, deps ...Path) { |
| 632 | c.addPhony(name, Paths(deps).Strings()) |
| 633 | } |
Colin Cross | 3cda0d8 | 2019-09-24 13:40:07 -0700 | [diff] [blame] | 634 | |
| 635 | func (c *makeVarsContext) DistForGoal(goal string, paths ...Path) { |
| 636 | c.DistForGoals([]string{goal}, paths...) |
| 637 | } |
| 638 | |
| 639 | func (c *makeVarsContext) DistForGoalWithFilename(goal string, path Path, filename string) { |
| 640 | c.DistForGoalsWithFilename([]string{goal}, path, filename) |
| 641 | } |
| 642 | |
| 643 | func (c *makeVarsContext) DistForGoals(goals []string, paths ...Path) { |
| 644 | c.addDist(goals, Paths(paths).Strings()) |
| 645 | } |
| 646 | |
| 647 | func (c *makeVarsContext) DistForGoalsWithFilename(goals []string, path Path, filename string) { |
| 648 | c.addDist(goals, []string{path.String() + ":" + filename}) |
| 649 | } |