| 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 | } |