Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 1 | // Copyright 2018 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 android |
| 16 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 17 | import ( |
Colin Cross | cefa94bd | 2019-06-03 15:07:03 -0700 | [diff] [blame] | 18 | "sort" |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 19 | "sync" |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 20 | ) |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 21 | |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 22 | // ApexModule is the interface that a module type is expected to implement if |
| 23 | // the module has to be built differently depending on whether the module |
| 24 | // is destined for an apex or not (installed to one of the regular partitions). |
| 25 | // |
| 26 | // Native shared libraries are one such module type; when it is built for an |
| 27 | // APEX, it should depend only on stable interfaces such as NDK, stable AIDL, |
| 28 | // or C APIs from other APEXs. |
| 29 | // |
| 30 | // A module implementing this interface will be mutated into multiple |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 31 | // variations by apex.apexMutator if it is directly or indirectly included |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 32 | // in one or more APEXs. Specifically, if a module is included in apex.foo and |
| 33 | // apex.bar then three apex variants are created: platform, apex.foo and |
| 34 | // apex.bar. The platform variant is for the regular partitions |
| 35 | // (e.g., /system or /vendor, etc.) while the other two are for the APEXs, |
| 36 | // respectively. |
| 37 | type ApexModule interface { |
| 38 | Module |
| 39 | apexModuleBase() *ApexModuleBase |
| 40 | |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 41 | // Marks that this module should be built for the APEXes of the specified names. |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 42 | // Call this before apex.apexMutator is run. |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 43 | BuildForApexes(apexNames []string) |
| 44 | |
| 45 | // Returns the name of the APEXes that this modoule will be built for |
| 46 | ApexVariations() []string |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 47 | |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 48 | // Returns the name of APEX that this module will be built for. Empty string |
| 49 | // is returned when 'IsForPlatform() == true'. Note that a module can be |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 50 | // included in multiple APEXes, in which case, the module is mutated into |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 51 | // multiple modules each of which for an APEX. This method returns the |
| 52 | // name of the APEX that a variant module is for. |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 53 | // Call this after apex.apexMutator is run. |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 54 | ApexName() string |
| 55 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 56 | // Tests whether this module will be built for the platform or not. |
| 57 | // This is a shortcut for ApexName() == "" |
| 58 | IsForPlatform() bool |
| 59 | |
| 60 | // Tests if this module could have APEX variants. APEX variants are |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 61 | // created only for the modules that returns true here. This is useful |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 62 | // for not creating APEX variants for certain types of shared libraries |
| 63 | // such as NDK stubs. |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 64 | CanHaveApexVariants() bool |
| 65 | |
| 66 | // Tests if this module can be installed to APEX as a file. For example, |
| 67 | // this would return true for shared libs while return false for static |
| 68 | // libs. |
| 69 | IsInstallableToApex() bool |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 70 | |
| 71 | // Mutate this module into one or more variants each of which is built |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 72 | // for an APEX marked via BuildForApexes(). |
Colin Cross | 43b92e0 | 2019-11-18 15:28:57 -0800 | [diff] [blame] | 73 | CreateApexVariations(mctx BottomUpMutatorContext) []Module |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 74 | |
| 75 | // Sets the name of the apex variant of this module. Called inside |
| 76 | // CreateApexVariations. |
| 77 | setApexName(apexName string) |
Jiyong Park | 4f7dd9b | 2019-08-12 10:37:49 +0900 | [diff] [blame] | 78 | |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 79 | // Tests if this module is available for the specified APEX or ":platform" |
| 80 | AvailableFor(what string) bool |
Jiyong Park | a7bc8ad | 2019-10-15 15:20:07 +0900 | [diff] [blame] | 81 | |
| 82 | // DepIsInSameApex tests if the other module 'dep' is installed to the same |
| 83 | // APEX as this module |
| 84 | DepIsInSameApex(ctx BaseModuleContext, dep Module) bool |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 85 | } |
| 86 | |
| 87 | type ApexProperties struct { |
Martin Stjernholm | 06ca82d | 2020-01-17 13:02:56 +0000 | [diff] [blame] | 88 | // Availability of this module in APEXes. Only the listed APEXes can contain |
| 89 | // this module. If the module has stubs then other APEXes and the platform may |
| 90 | // access it through them (subject to visibility). |
| 91 | // |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 92 | // "//apex_available:anyapex" is a pseudo APEX name that matches to any APEX. |
| 93 | // "//apex_available:platform" refers to non-APEX partitions like "system.img". |
| 94 | // Default is ["//apex_available:platform", "//apex_available:anyapex"]. |
| 95 | // TODO(b/128708192) change the default to ["//apex_available:platform"] |
| 96 | Apex_available []string |
| 97 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 98 | // Name of the apex variant that this module is mutated into |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 99 | ApexName string `blueprint:"mutated"` |
| 100 | } |
| 101 | |
| 102 | // Provides default implementation for the ApexModule interface. APEX-aware |
| 103 | // modules are expected to include this struct and call InitApexModule(). |
| 104 | type ApexModuleBase struct { |
| 105 | ApexProperties ApexProperties |
| 106 | |
| 107 | canHaveApexVariants bool |
Colin Cross | cefa94bd | 2019-06-03 15:07:03 -0700 | [diff] [blame] | 108 | |
| 109 | apexVariationsLock sync.Mutex // protects apexVariations during parallel apexDepsMutator |
| 110 | apexVariations []string |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 111 | } |
| 112 | |
| 113 | func (m *ApexModuleBase) apexModuleBase() *ApexModuleBase { |
| 114 | return m |
| 115 | } |
| 116 | |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 117 | func (m *ApexModuleBase) BuildForApexes(apexNames []string) { |
Colin Cross | cefa94bd | 2019-06-03 15:07:03 -0700 | [diff] [blame] | 118 | m.apexVariationsLock.Lock() |
| 119 | defer m.apexVariationsLock.Unlock() |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 120 | for _, apexName := range apexNames { |
| 121 | if !InList(apexName, m.apexVariations) { |
| 122 | m.apexVariations = append(m.apexVariations, apexName) |
| 123 | } |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 124 | } |
| 125 | } |
| 126 | |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 127 | func (m *ApexModuleBase) ApexVariations() []string { |
| 128 | return m.apexVariations |
| 129 | } |
| 130 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 131 | func (m *ApexModuleBase) ApexName() string { |
| 132 | return m.ApexProperties.ApexName |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 133 | } |
| 134 | |
| 135 | func (m *ApexModuleBase) IsForPlatform() bool { |
| 136 | return m.ApexProperties.ApexName == "" |
| 137 | } |
| 138 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 139 | func (m *ApexModuleBase) setApexName(apexName string) { |
| 140 | m.ApexProperties.ApexName = apexName |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 141 | } |
| 142 | |
| 143 | func (m *ApexModuleBase) CanHaveApexVariants() bool { |
| 144 | return m.canHaveApexVariants |
| 145 | } |
| 146 | |
| 147 | func (m *ApexModuleBase) IsInstallableToApex() bool { |
| 148 | // should be overriden if needed |
| 149 | return false |
| 150 | } |
| 151 | |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 152 | const ( |
Jiyong Park | b02bb40 | 2019-12-03 00:43:57 +0900 | [diff] [blame] | 153 | AvailableToPlatform = "//apex_available:platform" |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 154 | availableToAnyApex = "//apex_available:anyapex" |
| 155 | ) |
| 156 | |
Jiyong Park | a90ca00 | 2019-10-07 15:47:24 +0900 | [diff] [blame] | 157 | func CheckAvailableForApex(what string, apex_available []string) bool { |
| 158 | if len(apex_available) == 0 { |
Anton Hansson | eec79eb | 2020-01-10 15:12:39 +0000 | [diff] [blame] | 159 | // apex_available defaults to ["//apex_available:platform"], |
| 160 | // which means 'available to the platform but no apexes'. |
| 161 | return what == AvailableToPlatform |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 162 | } |
Jiyong Park | a90ca00 | 2019-10-07 15:47:24 +0900 | [diff] [blame] | 163 | return InList(what, apex_available) || |
Jiyong Park | b02bb40 | 2019-12-03 00:43:57 +0900 | [diff] [blame] | 164 | (what != AvailableToPlatform && InList(availableToAnyApex, apex_available)) |
Jiyong Park | a90ca00 | 2019-10-07 15:47:24 +0900 | [diff] [blame] | 165 | } |
| 166 | |
| 167 | func (m *ApexModuleBase) AvailableFor(what string) bool { |
| 168 | return CheckAvailableForApex(what, m.ApexProperties.Apex_available) |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 169 | } |
| 170 | |
Jiyong Park | a7bc8ad | 2019-10-15 15:20:07 +0900 | [diff] [blame] | 171 | func (m *ApexModuleBase) DepIsInSameApex(ctx BaseModuleContext, dep Module) bool { |
| 172 | // By default, if there is a dependency from A to B, we try to include both in the same APEX, |
| 173 | // unless B is explicitly from outside of the APEX (i.e. a stubs lib). Thus, returning true. |
| 174 | // This is overridden by some module types like apex.ApexBundle, cc.Module, java.Module, etc. |
| 175 | return true |
| 176 | } |
| 177 | |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 178 | func (m *ApexModuleBase) checkApexAvailableProperty(mctx BaseModuleContext) { |
| 179 | for _, n := range m.ApexProperties.Apex_available { |
Jiyong Park | b02bb40 | 2019-12-03 00:43:57 +0900 | [diff] [blame] | 180 | if n == AvailableToPlatform || n == availableToAnyApex { |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 181 | continue |
| 182 | } |
Orion Hodson | 4b5438a | 2019-10-08 10:40:51 +0100 | [diff] [blame] | 183 | if !mctx.OtherModuleExists(n) && !mctx.Config().AllowMissingDependencies() { |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 184 | mctx.PropertyErrorf("apex_available", "%q is not a valid module name", n) |
| 185 | } |
| 186 | } |
| 187 | } |
| 188 | |
Colin Cross | 43b92e0 | 2019-11-18 15:28:57 -0800 | [diff] [blame] | 189 | func (m *ApexModuleBase) CreateApexVariations(mctx BottomUpMutatorContext) []Module { |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 190 | if len(m.apexVariations) > 0 { |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 191 | m.checkApexAvailableProperty(mctx) |
Jiyong Park | 0f80c18 | 2020-01-31 02:49:53 +0900 | [diff] [blame] | 192 | |
Colin Cross | cefa94bd | 2019-06-03 15:07:03 -0700 | [diff] [blame] | 193 | sort.Strings(m.apexVariations) |
Jiyong Park | 127b40b | 2019-09-30 16:04:35 +0900 | [diff] [blame] | 194 | variations := []string{} |
Jiyong Park | 0f80c18 | 2020-01-31 02:49:53 +0900 | [diff] [blame] | 195 | variations = append(variations, "") // Original variation for platform |
Logan Chien | 3aeedc9 | 2018-12-26 15:32:21 +0800 | [diff] [blame] | 196 | variations = append(variations, m.apexVariations...) |
| 197 | |
Jiyong Park | 3ff1699 | 2019-12-27 14:11:47 +0900 | [diff] [blame] | 198 | defaultVariation := "" |
| 199 | mctx.SetDefaultDependencyVariation(&defaultVariation) |
Jiyong Park | 0f80c18 | 2020-01-31 02:49:53 +0900 | [diff] [blame] | 200 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 201 | modules := mctx.CreateVariations(variations...) |
| 202 | for i, m := range modules { |
Jiyong Park | 0f80c18 | 2020-01-31 02:49:53 +0900 | [diff] [blame] | 203 | platformVariation := i == 0 |
| 204 | if platformVariation && !mctx.Host() && !m.(ApexModule).AvailableFor(AvailableToPlatform) { |
| 205 | m.SkipInstall() |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 206 | } |
| 207 | m.(ApexModule).setApexName(variations[i]) |
| 208 | } |
| 209 | return modules |
| 210 | } |
| 211 | return nil |
| 212 | } |
| 213 | |
| 214 | var apexData OncePer |
| 215 | var apexNamesMapMutex sync.Mutex |
Colin Cross | 571cccf | 2019-02-04 11:22:08 -0800 | [diff] [blame] | 216 | var apexNamesKey = NewOnceKey("apexNames") |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 217 | |
| 218 | // This structure maintains the global mapping in between modules and APEXes. |
| 219 | // Examples: |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 220 | // |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 221 | // apexNamesMap()["foo"]["bar"] == true: module foo is directly depended on by APEX bar |
| 222 | // apexNamesMap()["foo"]["bar"] == false: module foo is indirectly depended on by APEX bar |
| 223 | // apexNamesMap()["foo"]["bar"] doesn't exist: foo is not built for APEX bar |
| 224 | func apexNamesMap() map[string]map[string]bool { |
Colin Cross | 571cccf | 2019-02-04 11:22:08 -0800 | [diff] [blame] | 225 | return apexData.Once(apexNamesKey, func() interface{} { |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 226 | return make(map[string]map[string]bool) |
| 227 | }).(map[string]map[string]bool) |
| 228 | } |
| 229 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 230 | // Update the map to mark that a module named moduleName is directly or indirectly |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 231 | // depended on by the specified APEXes. Directly depending means that a module |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 232 | // is explicitly listed in the build definition of the APEX via properties like |
| 233 | // native_shared_libs, java_libs, etc. |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 234 | func UpdateApexDependency(apexNames []string, moduleName string, directDep bool) { |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 235 | apexNamesMapMutex.Lock() |
| 236 | defer apexNamesMapMutex.Unlock() |
Jiyong Park | f760cae | 2020-02-12 07:53:12 +0900 | [diff] [blame^] | 237 | for _, apexName := range apexNames { |
| 238 | apexesForModule, ok := apexNamesMap()[moduleName] |
| 239 | if !ok { |
| 240 | apexesForModule = make(map[string]bool) |
| 241 | apexNamesMap()[moduleName] = apexesForModule |
| 242 | } |
| 243 | apexesForModule[apexName] = apexesForModule[apexName] || directDep |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 244 | } |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 245 | } |
| 246 | |
Jooyung Han | 671f1ce | 2019-12-17 12:47:13 +0900 | [diff] [blame] | 247 | // TODO(b/146393795): remove this when b/146393795 is fixed |
| 248 | func ClearApexDependency() { |
| 249 | m := apexNamesMap() |
| 250 | for k := range m { |
| 251 | delete(m, k) |
| 252 | } |
| 253 | } |
| 254 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 255 | // Tests whether a module named moduleName is directly depended on by an APEX |
| 256 | // named apexName. |
| 257 | func DirectlyInApex(apexName string, moduleName string) bool { |
| 258 | apexNamesMapMutex.Lock() |
| 259 | defer apexNamesMapMutex.Unlock() |
| 260 | if apexNames, ok := apexNamesMap()[moduleName]; ok { |
| 261 | return apexNames[apexName] |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 262 | } |
| 263 | return false |
| 264 | } |
| 265 | |
Nicolas Geoffray | c22c1bf | 2019-01-15 19:53:23 +0000 | [diff] [blame] | 266 | type hostContext interface { |
| 267 | Host() bool |
| 268 | } |
| 269 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 270 | // Tests whether a module named moduleName is directly depended on by any APEX. |
Nicolas Geoffray | c22c1bf | 2019-01-15 19:53:23 +0000 | [diff] [blame] | 271 | func DirectlyInAnyApex(ctx hostContext, moduleName string) bool { |
| 272 | if ctx.Host() { |
| 273 | // Host has no APEX. |
| 274 | return false |
| 275 | } |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 276 | apexNamesMapMutex.Lock() |
| 277 | defer apexNamesMapMutex.Unlock() |
| 278 | if apexNames, ok := apexNamesMap()[moduleName]; ok { |
| 279 | for an := range apexNames { |
| 280 | if apexNames[an] { |
Jiyong Park | 25fc6a9 | 2018-11-18 18:02:45 +0900 | [diff] [blame] | 281 | return true |
| 282 | } |
| 283 | } |
| 284 | } |
| 285 | return false |
| 286 | } |
| 287 | |
Jiyong Park | 0ddfcd1 | 2018-12-11 01:35:25 +0900 | [diff] [blame] | 288 | // Tests whether a module named module is depended on (including both |
| 289 | // direct and indirect dependencies) by any APEX. |
| 290 | func InAnyApex(moduleName string) bool { |
| 291 | apexNamesMapMutex.Lock() |
| 292 | defer apexNamesMapMutex.Unlock() |
| 293 | apexNames, ok := apexNamesMap()[moduleName] |
| 294 | return ok && len(apexNames) > 0 |
| 295 | } |
| 296 | |
| 297 | func GetApexesForModule(moduleName string) []string { |
| 298 | ret := []string{} |
| 299 | apexNamesMapMutex.Lock() |
| 300 | defer apexNamesMapMutex.Unlock() |
| 301 | if apexNames, ok := apexNamesMap()[moduleName]; ok { |
| 302 | for an := range apexNames { |
| 303 | ret = append(ret, an) |
| 304 | } |
| 305 | } |
| 306 | return ret |
Jiyong Park | de866cb | 2018-12-07 23:08:36 +0900 | [diff] [blame] | 307 | } |
| 308 | |
Jiyong Park | 9d45299 | 2018-10-03 00:38:19 +0900 | [diff] [blame] | 309 | func InitApexModule(m ApexModule) { |
| 310 | base := m.apexModuleBase() |
| 311 | base.canHaveApexVariants = true |
| 312 | |
| 313 | m.AddProperties(&base.ApexProperties) |
| 314 | } |