| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 1 | // Copyright 2017 Google Inc. All rights reserved. | 
|  | 2 | // | 
|  | 3 | // Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 4 | // you may not use this file except in compliance with the License. | 
|  | 5 | // You may obtain a copy of the License at | 
|  | 6 | // | 
|  | 7 | //     http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 8 | // | 
|  | 9 | // Unless required by applicable law or agreed to in writing, software | 
|  | 10 | // distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 12 | // See the License for the specific language governing permissions and | 
|  | 13 | // limitations under the License. | 
|  | 14 |  | 
|  | 15 | package python | 
|  | 16 |  | 
|  | 17 | // This file contains the "Base" module type for building Python program. | 
|  | 18 |  | 
|  | 19 | import ( | 
|  | 20 | "fmt" | 
|  | 21 | "path/filepath" | 
|  | 22 | "regexp" | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 23 | "strings" | 
|  | 24 |  | 
|  | 25 | "github.com/google/blueprint" | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 26 | "github.com/google/blueprint/proptools" | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 27 |  | 
|  | 28 | "android/soong/android" | 
|  | 29 | ) | 
|  | 30 |  | 
|  | 31 | func init() { | 
| Paul Duffin | d089045 | 2021-03-17 21:57:08 +0000 | [diff] [blame] | 32 | registerPythonMutators(android.InitRegistrationContext) | 
|  | 33 | } | 
|  | 34 |  | 
|  | 35 | func registerPythonMutators(ctx android.RegistrationContext) { | 
|  | 36 | ctx.PreDepsMutators(RegisterPythonPreDepsMutators) | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 37 | } | 
|  | 38 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 39 | // Exported to support other packages using Python modules in tests. | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 40 | func RegisterPythonPreDepsMutators(ctx android.RegisterMutatorsContext) { | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 41 | ctx.BottomUp("python_version", versionSplitMutator()).Parallel() | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 42 | } | 
|  | 43 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 44 | // the version-specific properties that apply to python modules. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 45 | type VersionProperties struct { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 46 | // whether the module is required to be built with this version. | 
|  | 47 | // Defaults to true for Python 3, and false otherwise. | 
| Liz Kammer | 59c0eae | 2021-09-17 17:48:05 -0400 | [diff] [blame] | 48 | Enabled *bool | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 49 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 50 | // list of source files specific to this Python version. | 
|  | 51 | // Using the syntax ":module", srcs may reference the outputs of other modules that produce source files, | 
|  | 52 | // e.g. genrule or filegroup. | 
| Colin Cross | 27b922f | 2019-03-04 22:35:41 -0800 | [diff] [blame] | 53 | Srcs []string `android:"path,arch_variant"` | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 54 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 55 | // list of source files that should not be used to build the Python module for this version. | 
|  | 56 | // This is most useful to remove files that are not common to all Python versions. | 
| Colin Cross | 27b922f | 2019-03-04 22:35:41 -0800 | [diff] [blame] | 57 | Exclude_srcs []string `android:"path,arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 58 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 59 | // list of the Python libraries used only for this Python version. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 60 | Libs []string `android:"arch_variant"` | 
|  | 61 |  | 
| Cole Faust | f09101e | 2024-04-18 18:33:15 +0000 | [diff] [blame] | 62 | // whether the binary is required to be built with embedded launcher for this version, defaults to true. | 
| Liz Kammer | 59c0eae | 2021-09-17 17:48:05 -0400 | [diff] [blame] | 63 | Embedded_launcher *bool // TODO(b/174041232): Remove this property | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 64 | } | 
|  | 65 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 66 | // properties that apply to all python modules | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 67 | type BaseProperties struct { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 68 | // the package path prefix within the output artifact at which to place the source/data | 
|  | 69 | // files of the current module. | 
|  | 70 | // eg. Pkg_path = "a/b/c"; Other packages can reference this module by using | 
|  | 71 | // (from a.b.c import ...) statement. | 
| Nan Zhang | bea0975 | 2018-05-31 12:49:33 -0700 | [diff] [blame] | 72 | // if left unspecified, all the source/data files path is unchanged within zip file. | 
| Liz Kammer | 59c0eae | 2021-09-17 17:48:05 -0400 | [diff] [blame] | 73 | Pkg_path *string | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 74 |  | 
|  | 75 | // true, if the Python module is used internally, eg, Python std libs. | 
| Liz Kammer | 59c0eae | 2021-09-17 17:48:05 -0400 | [diff] [blame] | 76 | Is_internal *bool | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 77 |  | 
|  | 78 | // list of source (.py) files compatible both with Python2 and Python3 used to compile the | 
|  | 79 | // Python module. | 
|  | 80 | // srcs may reference the outputs of other modules that produce source files like genrule | 
|  | 81 | // or filegroup using the syntax ":module". | 
|  | 82 | // Srcs has to be non-empty. | 
| Colin Cross | 27b922f | 2019-03-04 22:35:41 -0800 | [diff] [blame] | 83 | Srcs []string `android:"path,arch_variant"` | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 84 |  | 
|  | 85 | // list of source files that should not be used to build the C/C++ module. | 
|  | 86 | // This is most useful in the arch/multilib variants to remove non-common files | 
| Colin Cross | 27b922f | 2019-03-04 22:35:41 -0800 | [diff] [blame] | 87 | Exclude_srcs []string `android:"path,arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 88 |  | 
|  | 89 | // list of files or filegroup modules that provide data that should be installed alongside | 
|  | 90 | // the test. the file extension can be arbitrary except for (.py). | 
| Colin Cross | 27b922f | 2019-03-04 22:35:41 -0800 | [diff] [blame] | 91 | Data []string `android:"path,arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 92 |  | 
| Colin Cross | 1bc6393 | 2020-11-22 20:12:45 -0800 | [diff] [blame] | 93 | // list of java modules that provide data that should be installed alongside the test. | 
|  | 94 | Java_data []string | 
|  | 95 |  | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 96 | // list of the Python libraries compatible both with Python2 and Python3. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 97 | Libs []string `android:"arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 98 |  | 
|  | 99 | Version struct { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 100 | // Python2-specific properties, including whether Python2 is supported for this module | 
|  | 101 | // and version-specific sources, exclusions and dependencies. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 102 | Py2 VersionProperties `android:"arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 103 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 104 | // Python3-specific properties, including whether Python3 is supported for this module | 
|  | 105 | // and version-specific sources, exclusions and dependencies. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 106 | Py3 VersionProperties `android:"arch_variant"` | 
|  | 107 | } `android:"arch_variant"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 108 |  | 
|  | 109 | // the actual version each module uses after variations created. | 
|  | 110 | // this property name is hidden from users' perspectives, and soong will populate it during | 
|  | 111 | // runtime. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 112 | Actual_version string `blueprint:"mutated"` | 
| Liz Kammer | 7e93e5b | 2020-10-30 15:44:09 -0700 | [diff] [blame] | 113 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 114 | // whether the module is required to be built with actual_version. | 
|  | 115 | // this is set by the python version mutator based on version-specific properties | 
| Liz Kammer | 7e93e5b | 2020-10-30 15:44:09 -0700 | [diff] [blame] | 116 | Enabled *bool `blueprint:"mutated"` | 
|  | 117 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 118 | // whether the binary is required to be built with embedded launcher for this actual_version. | 
|  | 119 | // this is set by the python version mutator based on version-specific properties | 
| Liz Kammer | 7e93e5b | 2020-10-30 15:44:09 -0700 | [diff] [blame] | 120 | Embedded_launcher *bool `blueprint:"mutated"` | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 121 | } | 
|  | 122 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 123 | // Used to store files of current module after expanding dependencies | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 124 | type pathMapping struct { | 
|  | 125 | dest string | 
|  | 126 | src  android.Path | 
|  | 127 | } | 
|  | 128 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 129 | type PythonLibraryModule struct { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 130 | android.ModuleBase | 
| Nan Zhang | a3fc4ba | 2017-07-20 17:43:37 -0700 | [diff] [blame] | 131 | android.DefaultableModuleBase | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 132 |  | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 133 | properties      BaseProperties | 
|  | 134 | protoProperties android.ProtoProperties | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 135 |  | 
|  | 136 | // initialize before calling Init | 
|  | 137 | hod      android.HostOrDeviceSupported | 
|  | 138 | multilib android.Multilib | 
|  | 139 |  | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 140 | // the Python files of current module after expanding source dependencies. | 
|  | 141 | // pathMapping: <dest: runfile_path, src: source_path> | 
|  | 142 | srcsPathMappings []pathMapping | 
|  | 143 |  | 
|  | 144 | // the data files of current module after expanding source dependencies. | 
|  | 145 | // pathMapping: <dest: runfile_path, src: source_path> | 
|  | 146 | dataPathMappings []pathMapping | 
|  | 147 |  | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 148 | // The zip file containing the current module's source/data files. | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 149 | srcsZip android.Path | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 150 |  | 
|  | 151 | // The zip file containing the current module's source/data files, with the | 
|  | 152 | // source files precompiled. | 
|  | 153 | precompiledSrcsZip android.Path | 
| Ronald Braunstein | c4cd7a1 | 2024-04-16 16:39:48 -0700 | [diff] [blame] | 154 |  | 
|  | 155 | sourceProperties android.SourceProperties | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 156 | } | 
|  | 157 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 158 | // newModule generates new Python base module | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 159 | func newModule(hod android.HostOrDeviceSupported, multilib android.Multilib) *PythonLibraryModule { | 
|  | 160 | return &PythonLibraryModule{ | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 161 | hod:      hod, | 
|  | 162 | multilib: multilib, | 
|  | 163 | } | 
|  | 164 | } | 
|  | 165 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 166 | // interface implemented by Python modules to provide source and data mappings and zip to python | 
|  | 167 | // modules that depend on it | 
|  | 168 | type pythonDependency interface { | 
|  | 169 | getSrcsPathMappings() []pathMapping | 
|  | 170 | getDataPathMappings() []pathMapping | 
|  | 171 | getSrcsZip() android.Path | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 172 | getPrecompiledSrcsZip() android.Path | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 173 | getPkgPath() string | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 174 | } | 
|  | 175 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 176 | // getSrcsPathMappings gets this module's path mapping of src source path : runfiles destination | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 177 | func (p *PythonLibraryModule) getSrcsPathMappings() []pathMapping { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 178 | return p.srcsPathMappings | 
|  | 179 | } | 
|  | 180 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 181 | // getSrcsPathMappings gets this module's path mapping of data source path : runfiles destination | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 182 | func (p *PythonLibraryModule) getDataPathMappings() []pathMapping { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 183 | return p.dataPathMappings | 
|  | 184 | } | 
|  | 185 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 186 | // getSrcsZip returns the filepath where the current module's source/data files are zipped. | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 187 | func (p *PythonLibraryModule) getSrcsZip() android.Path { | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 188 | return p.srcsZip | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 189 | } | 
|  | 190 |  | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 191 | // getSrcsZip returns the filepath where the current module's source/data files are zipped. | 
|  | 192 | func (p *PythonLibraryModule) getPrecompiledSrcsZip() android.Path { | 
|  | 193 | return p.precompiledSrcsZip | 
|  | 194 | } | 
|  | 195 |  | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 196 | // getPkgPath returns the pkg_path value | 
|  | 197 | func (p *PythonLibraryModule) getPkgPath() string { | 
|  | 198 | return String(p.properties.Pkg_path) | 
|  | 199 | } | 
|  | 200 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 201 | func (p *PythonLibraryModule) getBaseProperties() *BaseProperties { | 
|  | 202 | return &p.properties | 
|  | 203 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 204 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 205 | var _ pythonDependency = (*PythonLibraryModule)(nil) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 206 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 207 | func (p *PythonLibraryModule) init() android.Module { | 
| Ronald Braunstein | c4cd7a1 | 2024-04-16 16:39:48 -0700 | [diff] [blame] | 208 | p.AddProperties(&p.properties, &p.protoProperties, &p.sourceProperties) | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 209 | android.InitAndroidArchModule(p, p.hod, p.multilib) | 
| Nan Zhang | a3fc4ba | 2017-07-20 17:43:37 -0700 | [diff] [blame] | 210 | android.InitDefaultableModule(p) | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 211 | return p | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 212 | } | 
|  | 213 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 214 | // Python-specific tag to transfer information on the purpose of a dependency. | 
|  | 215 | // This is used when adding a dependency on a module, which can later be accessed when visiting | 
|  | 216 | // dependencies. | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 217 | type dependencyTag struct { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 218 | blueprint.BaseDependencyTag | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 219 | name string | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 220 | } | 
|  | 221 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 222 | // Python-specific tag that indicates that installed files of this module should depend on installed | 
|  | 223 | // files of the dependency | 
| Colin Cross | e9fe294 | 2020-11-10 18:12:15 -0800 | [diff] [blame] | 224 | type installDependencyTag struct { | 
|  | 225 | blueprint.BaseDependencyTag | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 226 | // embedding this struct provides the installation dependency requirement | 
| Colin Cross | e9fe294 | 2020-11-10 18:12:15 -0800 | [diff] [blame] | 227 | android.InstallAlwaysNeededDependencyTag | 
|  | 228 | name string | 
|  | 229 | } | 
|  | 230 |  | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 231 | var ( | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 232 | pythonLibTag = dependencyTag{name: "pythonLib"} | 
|  | 233 | javaDataTag  = dependencyTag{name: "javaData"} | 
|  | 234 | // The python interpreter, with soong module name "py3-launcher" or "py3-launcher-autorun". | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 235 | launcherTag          = dependencyTag{name: "launcher"} | 
|  | 236 | launcherSharedLibTag = installDependencyTag{name: "launcherSharedLib"} | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 237 | // The python interpreter built for host so that we can precompile python sources. | 
|  | 238 | // This only works because the precompiled sources don't vary by architecture. | 
|  | 239 | // The soong module name is "py3-launcher". | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 240 | hostLauncherTag          = dependencyTag{name: "hostLauncher"} | 
|  | 241 | hostlauncherSharedLibTag = dependencyTag{name: "hostlauncherSharedLib"} | 
|  | 242 | hostStdLibTag            = dependencyTag{name: "hostStdLib"} | 
|  | 243 | pathComponentRegexp      = regexp.MustCompile(`^[a-zA-Z_][a-zA-Z0-9_-]*$`) | 
|  | 244 | pyExt                    = ".py" | 
|  | 245 | protoExt                 = ".proto" | 
|  | 246 | pyVersion2               = "PY2" | 
|  | 247 | pyVersion3               = "PY3" | 
| Cole Faust | d82f036 | 2023-04-12 17:32:19 -0700 | [diff] [blame] | 248 | pyVersion2And3           = "PY2ANDPY3" | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 249 | internalPath             = "internal" | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 250 | ) | 
|  | 251 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 252 | type basePropertiesProvider interface { | 
|  | 253 | getBaseProperties() *BaseProperties | 
|  | 254 | } | 
|  | 255 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 256 | // versionSplitMutator creates version variants for modules and appends the version-specific | 
|  | 257 | // properties for a given variant to the properties in the variant module | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 258 | func versionSplitMutator() func(android.BottomUpMutatorContext) { | 
|  | 259 | return func(mctx android.BottomUpMutatorContext) { | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 260 | if base, ok := mctx.Module().(basePropertiesProvider); ok { | 
|  | 261 | props := base.getBaseProperties() | 
|  | 262 | var versionNames []string | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 263 | // collect version specific properties, so that we can merge version-specific properties | 
|  | 264 | // into the module's overall properties | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 265 | var versionProps []VersionProperties | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 266 | // PY3 is first so that we alias the PY3 variant rather than PY2 if both | 
|  | 267 | // are available | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 268 | if proptools.BoolDefault(props.Version.Py3.Enabled, true) { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 269 | versionNames = append(versionNames, pyVersion3) | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 270 | versionProps = append(versionProps, props.Version.Py3) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 271 | } | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 272 | if proptools.BoolDefault(props.Version.Py2.Enabled, false) { | 
| Cole Faust | edc4c50 | 2022-09-09 19:39:25 -0700 | [diff] [blame] | 273 | if !mctx.DeviceConfig().BuildBrokenUsesSoongPython2Modules() && | 
| Cole Faust | edc4c50 | 2022-09-09 19:39:25 -0700 | [diff] [blame] | 274 | mctx.ModuleName() != "py2-cmd" && | 
|  | 275 | mctx.ModuleName() != "py2-stdlib" { | 
|  | 276 | mctx.PropertyErrorf("version.py2.enabled", "Python 2 is no longer supported, please convert to python 3. This error can be temporarily overridden by setting BUILD_BROKEN_USES_SOONG_PYTHON2_MODULES := true in the product configuration") | 
|  | 277 | } | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 278 | versionNames = append(versionNames, pyVersion2) | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 279 | versionProps = append(versionProps, props.Version.Py2) | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 280 | } | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 281 | modules := mctx.CreateLocalVariations(versionNames...) | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 282 | // Alias module to the first variant | 
| Liz Kammer | dd849a8 | 2020-06-12 16:38:45 -0700 | [diff] [blame] | 283 | if len(versionNames) > 0 { | 
|  | 284 | mctx.AliasVariation(versionNames[0]) | 
|  | 285 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 286 | for i, v := range versionNames { | 
|  | 287 | // set the actual version for Python module. | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 288 | newProps := modules[i].(basePropertiesProvider).getBaseProperties() | 
|  | 289 | newProps.Actual_version = v | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 290 | // append versioned properties for the Python module to the overall properties | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 291 | err := proptools.AppendMatchingProperties([]interface{}{newProps}, &versionProps[i], nil) | 
| Liz Kammer | 7e93e5b | 2020-10-30 15:44:09 -0700 | [diff] [blame] | 292 | if err != nil { | 
|  | 293 | panic(err) | 
|  | 294 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 295 | } | 
|  | 296 | } | 
|  | 297 | } | 
|  | 298 | } | 
|  | 299 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 300 | func anyHasExt(paths []string, ext string) bool { | 
|  | 301 | for _, p := range paths { | 
|  | 302 | if filepath.Ext(p) == ext { | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 303 | return true | 
|  | 304 | } | 
|  | 305 | } | 
|  | 306 |  | 
|  | 307 | return false | 
|  | 308 | } | 
|  | 309 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 310 | func (p *PythonLibraryModule) anySrcHasExt(ctx android.BottomUpMutatorContext, ext string) bool { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 311 | return anyHasExt(p.properties.Srcs, ext) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 312 | } | 
|  | 313 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 314 | // DepsMutator mutates dependencies for this module: | 
| Colin Cross | d079e0b | 2022-08-16 10:27:33 -0700 | [diff] [blame] | 315 | //   - handles proto dependencies, | 
|  | 316 | //   - if required, specifies launcher and adds launcher dependencies, | 
|  | 317 | //   - applies python version mutations to Python dependencies | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 318 | func (p *PythonLibraryModule) DepsMutator(ctx android.BottomUpMutatorContext) { | 
| Colin Cross | fe17f6f | 2019-03-28 19:30:56 -0700 | [diff] [blame] | 319 | android.ProtoDeps(ctx, &p.protoProperties) | 
|  | 320 |  | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 321 | versionVariation := []blueprint.Variation{ | 
|  | 322 | {"python_version", p.properties.Actual_version}, | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 323 | } | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 324 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 325 | // If sources contain a proto file, add dependency on libprotobuf-python | 
|  | 326 | if p.anySrcHasExt(ctx, protoExt) && p.Name() != "libprotobuf-python" { | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 327 | ctx.AddVariationDependencies(versionVariation, pythonLibTag, "libprotobuf-python") | 
|  | 328 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 329 |  | 
|  | 330 | // Add python library dependencies for this python version variation | 
| Colin Cross | e20113d | 2020-11-22 19:37:44 -0800 | [diff] [blame] | 331 | ctx.AddVariationDependencies(versionVariation, pythonLibTag, android.LastUniqueStrings(p.properties.Libs)...) | 
| Liz Kammer | 7e93e5b | 2020-10-30 15:44:09 -0700 | [diff] [blame] | 332 |  | 
| Colin Cross | 1bc6393 | 2020-11-22 20:12:45 -0800 | [diff] [blame] | 333 | // Emulate the data property for java_data but with the arch variation overridden to "common" | 
|  | 334 | // so that it can point to java modules. | 
|  | 335 | javaDataVariation := []blueprint.Variation{{"arch", android.Common.String()}} | 
|  | 336 | ctx.AddVariationDependencies(javaDataVariation, javaDataTag, p.properties.Java_data...) | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 337 |  | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 338 | p.AddDepsOnPythonLauncherAndStdlib(ctx, hostStdLibTag, hostLauncherTag, hostlauncherSharedLibTag, false, ctx.Config().BuildOSTarget) | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 339 | } | 
|  | 340 |  | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 341 | // AddDepsOnPythonLauncherAndStdlib will make the current module depend on the python stdlib, | 
|  | 342 | // launcher (interpreter), and the launcher's shared libraries. If autorun is true, it will use | 
|  | 343 | // the autorun launcher instead of the regular one. This function acceps a targetForDeps argument | 
|  | 344 | // as the target to use for these dependencies. For embedded launcher python binaries, the launcher | 
|  | 345 | // that will be embedded will be under the same target as the python module itself. But when | 
|  | 346 | // precompiling python code, we need to get the python launcher built for host, even if we're | 
|  | 347 | // compiling the python module for device, so we pass a different target to this function. | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 348 | func (p *PythonLibraryModule) AddDepsOnPythonLauncherAndStdlib(ctx android.BottomUpMutatorContext, | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 349 | stdLibTag, launcherTag, launcherSharedLibTag blueprint.DependencyTag, | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 350 | autorun bool, targetForDeps android.Target) { | 
|  | 351 | var stdLib string | 
|  | 352 | var launcherModule string | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 353 | // Add launcher shared lib dependencies. Ideally, these should be | 
|  | 354 | // derived from the `shared_libs` property of the launcher. TODO: read these from | 
|  | 355 | // the python launcher itself using ctx.OtherModuleProvider() or similar on the result | 
|  | 356 | // of ctx.AddFarVariationDependencies() | 
|  | 357 | launcherSharedLibDeps := []string{ | 
|  | 358 | "libsqlite", | 
|  | 359 | } | 
|  | 360 | // Add launcher-specific dependencies for bionic | 
|  | 361 | if targetForDeps.Os.Bionic() { | 
|  | 362 | launcherSharedLibDeps = append(launcherSharedLibDeps, "libc", "libdl", "libm") | 
|  | 363 | } | 
|  | 364 | if targetForDeps.Os == android.LinuxMusl && !ctx.Config().HostStaticBinaries() { | 
|  | 365 | launcherSharedLibDeps = append(launcherSharedLibDeps, "libc_musl") | 
|  | 366 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 367 |  | 
|  | 368 | switch p.properties.Actual_version { | 
|  | 369 | case pyVersion2: | 
|  | 370 | stdLib = "py2-stdlib" | 
|  | 371 |  | 
|  | 372 | launcherModule = "py2-launcher" | 
|  | 373 | if autorun { | 
|  | 374 | launcherModule = "py2-launcher-autorun" | 
|  | 375 | } | 
|  | 376 |  | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 377 | launcherSharedLibDeps = append(launcherSharedLibDeps, "libc++") | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 378 | case pyVersion3: | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 379 | var prebuiltStdLib bool | 
|  | 380 | if targetForDeps.Os.Bionic() { | 
|  | 381 | prebuiltStdLib = false | 
|  | 382 | } else if ctx.Config().VendorConfig("cpython3").Bool("force_build_host") { | 
|  | 383 | prebuiltStdLib = false | 
|  | 384 | } else { | 
|  | 385 | prebuiltStdLib = true | 
|  | 386 | } | 
|  | 387 |  | 
|  | 388 | if prebuiltStdLib { | 
|  | 389 | stdLib = "py3-stdlib-prebuilt" | 
|  | 390 | } else { | 
|  | 391 | stdLib = "py3-stdlib" | 
|  | 392 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 393 |  | 
|  | 394 | launcherModule = "py3-launcher" | 
|  | 395 | if autorun { | 
|  | 396 | launcherModule = "py3-launcher-autorun" | 
|  | 397 | } | 
|  | 398 | if ctx.Config().HostStaticBinaries() && targetForDeps.Os == android.LinuxMusl { | 
|  | 399 | launcherModule += "-static" | 
|  | 400 | } | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 401 | if ctx.Device() { | 
|  | 402 | launcherSharedLibDeps = append(launcherSharedLibDeps, "liblog") | 
|  | 403 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 404 | default: | 
|  | 405 | panic(fmt.Errorf("unknown Python Actual_version: %q for module: %q.", | 
|  | 406 | p.properties.Actual_version, ctx.ModuleName())) | 
|  | 407 | } | 
|  | 408 | targetVariations := targetForDeps.Variations() | 
|  | 409 | if ctx.ModuleName() != stdLib { | 
|  | 410 | stdLibVariations := make([]blueprint.Variation, 0, len(targetVariations)+1) | 
|  | 411 | stdLibVariations = append(stdLibVariations, blueprint.Variation{Mutator: "python_version", Variation: p.properties.Actual_version}) | 
|  | 412 | stdLibVariations = append(stdLibVariations, targetVariations...) | 
|  | 413 | // Using AddFarVariationDependencies for all of these because they can be for a different | 
|  | 414 | // platform, like if the python module itself was being compiled for device, we may want | 
|  | 415 | // the python interpreter built for host so that we can precompile python sources. | 
|  | 416 | ctx.AddFarVariationDependencies(stdLibVariations, stdLibTag, stdLib) | 
|  | 417 | } | 
|  | 418 | ctx.AddFarVariationDependencies(targetVariations, launcherTag, launcherModule) | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 419 | ctx.AddFarVariationDependencies(targetVariations, launcherSharedLibTag, launcherSharedLibDeps...) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 420 | } | 
|  | 421 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 422 | // GenerateAndroidBuildActions performs build actions common to all Python modules | 
|  | 423 | func (p *PythonLibraryModule) GenerateAndroidBuildActions(ctx android.ModuleContext) { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 424 | expandedSrcs := android.PathsForModuleSrcExcludes(ctx, p.properties.Srcs, p.properties.Exclude_srcs) | 
| Colin Cross | 4021302 | 2023-12-13 15:19:49 -0800 | [diff] [blame] | 425 | android.SetProvider(ctx, blueprint.SrcsFileProviderKey, blueprint.SrcsFileProviderData{SrcPaths: expandedSrcs.Strings()}) | 
| Ronald Braunstein | c4cd7a1 | 2024-04-16 16:39:48 -0700 | [diff] [blame] | 426 | // Keep before any early returns. | 
|  | 427 | android.SetProvider(ctx, android.TestOnlyProviderKey, android.TestModuleInformation{ | 
|  | 428 | TestOnly:       Bool(p.sourceProperties.Test_only), | 
|  | 429 | TopLevelTarget: p.sourceProperties.Top_level_test_target, | 
|  | 430 | }) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 431 |  | 
|  | 432 | // expand data files from "data" property. | 
| Colin Cross | 8a49795 | 2019-03-05 22:25:09 -0800 | [diff] [blame] | 433 | expandedData := android.PathsForModuleSrc(ctx, p.properties.Data) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 434 |  | 
| Colin Cross | 1bc6393 | 2020-11-22 20:12:45 -0800 | [diff] [blame] | 435 | // Emulate the data property for java_data dependencies. | 
|  | 436 | for _, javaData := range ctx.GetDirectDepsWithTag(javaDataTag) { | 
|  | 437 | expandedData = append(expandedData, android.OutputFilesForModule(ctx, javaData, "")...) | 
|  | 438 | } | 
|  | 439 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 440 | // Validate pkg_path property | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 441 | pkgPath := String(p.properties.Pkg_path) | 
|  | 442 | if pkgPath != "" { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 443 | // TODO: export validation from android/paths.go handling to replace this duplicated functionality | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 444 | pkgPath = filepath.Clean(String(p.properties.Pkg_path)) | 
|  | 445 | if pkgPath == ".." || strings.HasPrefix(pkgPath, "../") || | 
|  | 446 | strings.HasPrefix(pkgPath, "/") { | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 447 | ctx.PropertyErrorf("pkg_path", | 
|  | 448 | "%q must be a relative path contained in par file.", | 
| Nan Zhang | ea568a4 | 2017-11-08 21:20:04 -0800 | [diff] [blame] | 449 | String(p.properties.Pkg_path)) | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 450 | return | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 451 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 452 | } | 
|  | 453 | // If property Is_internal is set, prepend pkgPath with internalPath | 
|  | 454 | if proptools.BoolDefault(p.properties.Is_internal, false) { | 
|  | 455 | pkgPath = filepath.Join(internalPath, pkgPath) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 456 | } | 
|  | 457 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 458 | // generate src:destination path mappings for this module | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 459 | p.genModulePathMappings(ctx, pkgPath, expandedSrcs, expandedData) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 460 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 461 | // generate the zipfile of all source and data files | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 462 | p.srcsZip = p.createSrcsZip(ctx, pkgPath) | 
| Dan Willemsen | fe2dafc | 2023-08-24 22:59:16 +0000 | [diff] [blame] | 463 | p.precompiledSrcsZip = p.precompileSrcs(ctx) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 464 | } | 
|  | 465 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 466 | func isValidPythonPath(path string) error { | 
|  | 467 | identifiers := strings.Split(strings.TrimSuffix(path, filepath.Ext(path)), "/") | 
|  | 468 | for _, token := range identifiers { | 
|  | 469 | if !pathComponentRegexp.MatchString(token) { | 
|  | 470 | return fmt.Errorf("the path %q contains invalid subpath %q. "+ | 
|  | 471 | "Subpaths must be at least one character long. "+ | 
|  | 472 | "The first character must an underscore or letter. "+ | 
|  | 473 | "Following characters may be any of: letter, digit, underscore, hyphen.", | 
|  | 474 | path, token) | 
|  | 475 | } | 
|  | 476 | } | 
|  | 477 | return nil | 
|  | 478 | } | 
|  | 479 |  | 
|  | 480 | // For this module, generate unique pathMappings: <dest: runfiles_path, src: source_path> | 
|  | 481 | // for python/data files expanded from properties. | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 482 | func (p *PythonLibraryModule) genModulePathMappings(ctx android.ModuleContext, pkgPath string, | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 483 | expandedSrcs, expandedData android.Paths) { | 
|  | 484 | // fetch <runfiles_path, source_path> pairs from "src" and "data" properties to | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 485 | // check current module duplicates. | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 486 | destToPySrcs := make(map[string]string) | 
|  | 487 | destToPyData := make(map[string]string) | 
|  | 488 |  | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 489 | // Disable path checks for the stdlib, as it includes a "." in the version string | 
|  | 490 | isInternal := proptools.BoolDefault(p.properties.Is_internal, false) | 
|  | 491 |  | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 492 | for _, s := range expandedSrcs { | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 493 | if s.Ext() != pyExt && s.Ext() != protoExt { | 
|  | 494 | ctx.PropertyErrorf("srcs", "found non (.py|.proto) file: %q!", s.String()) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 495 | continue | 
|  | 496 | } | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 497 | runfilesPath := filepath.Join(pkgPath, s.Rel()) | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 498 | if !isInternal { | 
|  | 499 | if err := isValidPythonPath(runfilesPath); err != nil { | 
|  | 500 | ctx.PropertyErrorf("srcs", err.Error()) | 
|  | 501 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 502 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 503 | if !checkForDuplicateOutputPath(ctx, destToPySrcs, runfilesPath, s.String(), p.Name(), p.Name()) { | 
|  | 504 | p.srcsPathMappings = append(p.srcsPathMappings, pathMapping{dest: runfilesPath, src: s}) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 505 | } | 
|  | 506 | } | 
|  | 507 |  | 
|  | 508 | for _, d := range expandedData { | 
| Raphael Blistein | 5985846 | 2024-05-08 16:15:53 +0000 | [diff] [blame] | 509 | if d.Ext() == pyExt { | 
|  | 510 | ctx.PropertyErrorf("data", "found (.py) file: %q!", d.String()) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 511 | continue | 
|  | 512 | } | 
| Nan Zhang | 1db8540 | 2017-12-18 13:20:23 -0800 | [diff] [blame] | 513 | runfilesPath := filepath.Join(pkgPath, d.Rel()) | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 514 | if !checkForDuplicateOutputPath(ctx, destToPyData, runfilesPath, d.String(), p.Name(), p.Name()) { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 515 | p.dataPathMappings = append(p.dataPathMappings, | 
|  | 516 | pathMapping{dest: runfilesPath, src: d}) | 
|  | 517 | } | 
|  | 518 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 519 | } | 
|  | 520 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 521 | // createSrcsZip registers build actions to zip current module's sources and data. | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 522 | func (p *PythonLibraryModule) createSrcsZip(ctx android.ModuleContext, pkgPath string) android.Path { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 523 | relativeRootMap := make(map[string]android.Paths) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 524 | var protoSrcs android.Paths | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 525 | addPathMapping := func(path pathMapping) { | 
| Raphael Blistein | 5985846 | 2024-05-08 16:15:53 +0000 | [diff] [blame] | 526 | relativeRoot := strings.TrimSuffix(path.src.String(), path.src.Rel()) | 
|  | 527 | relativeRootMap[relativeRoot] = append(relativeRootMap[relativeRoot], path.src) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 528 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 529 |  | 
|  | 530 | // "srcs" or "data" properties may contain filegroups so it might happen that | 
|  | 531 | // the root directory for each source path is different. | 
|  | 532 | for _, path := range p.srcsPathMappings { | 
| Raphael Blistein | 5985846 | 2024-05-08 16:15:53 +0000 | [diff] [blame] | 533 | // handle proto sources separately | 
|  | 534 | if path.src.Ext() == protoExt { | 
|  | 535 | protoSrcs = append(protoSrcs, path.src) | 
|  | 536 | } else { | 
|  | 537 | addPathMapping(path) | 
|  | 538 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 539 | } | 
|  | 540 | for _, path := range p.dataPathMappings { | 
|  | 541 | addPathMapping(path) | 
|  | 542 | } | 
|  | 543 |  | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 544 | var zips android.Paths | 
|  | 545 | if len(protoSrcs) > 0 { | 
| Colin Cross | 19878da | 2019-03-28 14:45:07 -0700 | [diff] [blame] | 546 | protoFlags := android.GetProtoFlags(ctx, &p.protoProperties) | 
|  | 547 | protoFlags.OutTypeFlag = "--python_out" | 
|  | 548 |  | 
| Cole Faust | caf766b | 2022-10-21 16:07:56 -0700 | [diff] [blame] | 549 | if pkgPath != "" { | 
| Cole Faust | 43ac21f | 2022-09-19 11:19:52 -0700 | [diff] [blame] | 550 | pkgPathStagingDir := android.PathForModuleGen(ctx, "protos_staged_for_pkg_path") | 
|  | 551 | rule := android.NewRuleBuilder(pctx, ctx) | 
|  | 552 | var stagedProtoSrcs android.Paths | 
|  | 553 | for _, srcFile := range protoSrcs { | 
|  | 554 | stagedProtoSrc := pkgPathStagingDir.Join(ctx, pkgPath, srcFile.Rel()) | 
| Cole Faust | 43ac21f | 2022-09-19 11:19:52 -0700 | [diff] [blame] | 555 | rule.Command().Text("cp -f").Input(srcFile).Output(stagedProtoSrc) | 
|  | 556 | stagedProtoSrcs = append(stagedProtoSrcs, stagedProtoSrc) | 
|  | 557 | } | 
|  | 558 | rule.Build("stage_protos_for_pkg_path", "Stage protos for pkg_path") | 
|  | 559 | protoSrcs = stagedProtoSrcs | 
| Cole Faust | 43ac21f | 2022-09-19 11:19:52 -0700 | [diff] [blame] | 560 | } | 
|  | 561 |  | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 562 | for _, srcFile := range protoSrcs { | 
| Cole Faust | caf766b | 2022-10-21 16:07:56 -0700 | [diff] [blame] | 563 | zip := genProto(ctx, srcFile, protoFlags) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 564 | zips = append(zips, zip) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 565 | } | 
|  | 566 | } | 
|  | 567 |  | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 568 | if len(relativeRootMap) > 0 { | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 569 | // in order to keep stable order of soong_zip params, we sort the keys here. | 
| Cole Faust | 18994c7 | 2023-02-28 16:02:16 -0800 | [diff] [blame] | 570 | roots := android.SortedKeys(relativeRootMap) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 571 |  | 
| Cole Faust | 0124336 | 2022-06-02 12:11:12 -0700 | [diff] [blame] | 572 | // Use -symlinks=false so that the symlinks in the bazel output directory are followed | 
|  | 573 | parArgs := []string{"-symlinks=false"} | 
| Nan Zhang | f0c4e43 | 2018-05-22 14:50:18 -0700 | [diff] [blame] | 574 | if pkgPath != "" { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 575 | // use package path as path prefix | 
| Nan Zhang | f0c4e43 | 2018-05-22 14:50:18 -0700 | [diff] [blame] | 576 | parArgs = append(parArgs, `-P `+pkgPath) | 
|  | 577 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 578 | paths := android.Paths{} | 
|  | 579 | for _, root := range roots { | 
|  | 580 | // specify relative root of file in following -f arguments | 
|  | 581 | parArgs = append(parArgs, `-C `+root) | 
|  | 582 | for _, path := range relativeRootMap[root] { | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 583 | parArgs = append(parArgs, `-f `+path.String()) | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 584 | paths = append(paths, path) | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 585 | } | 
|  | 586 | } | 
|  | 587 |  | 
|  | 588 | origSrcsZip := android.PathForModuleOut(ctx, ctx.ModuleName()+".py.srcszip") | 
|  | 589 | ctx.Build(pctx, android.BuildParams{ | 
|  | 590 | Rule:        zip, | 
|  | 591 | Description: "python library archive", | 
|  | 592 | Output:      origSrcsZip, | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 593 | // as zip rule does not use $in, there is no real need to distinguish between Inputs and Implicits | 
|  | 594 | Implicits: paths, | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 595 | Args: map[string]string{ | 
|  | 596 | "args": strings.Join(parArgs, " "), | 
|  | 597 | }, | 
|  | 598 | }) | 
|  | 599 | zips = append(zips, origSrcsZip) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 600 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 601 | // we may have multiple zips due to separate handling of proto source files | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 602 | if len(zips) == 1 { | 
|  | 603 | return zips[0] | 
|  | 604 | } else { | 
|  | 605 | combinedSrcsZip := android.PathForModuleOut(ctx, ctx.ModuleName()+".srcszip") | 
|  | 606 | ctx.Build(pctx, android.BuildParams{ | 
|  | 607 | Rule:        combineZip, | 
|  | 608 | Description: "combine python library archive", | 
|  | 609 | Output:      combinedSrcsZip, | 
|  | 610 | Inputs:      zips, | 
|  | 611 | }) | 
|  | 612 | return combinedSrcsZip | 
|  | 613 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 614 | } | 
|  | 615 |  | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 616 | func (p *PythonLibraryModule) precompileSrcs(ctx android.ModuleContext) android.Path { | 
|  | 617 | // To precompile the python sources, we need a python interpreter and stdlib built | 
|  | 618 | // for host. We then use those to compile the python sources, which may be used on either | 
|  | 619 | // host of device. Python bytecode is architecture agnostic, so we're essentially | 
|  | 620 | // "cross compiling" for device here purely by virtue of host and device python bytecode | 
|  | 621 | // being the same. | 
|  | 622 | var stdLib android.Path | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 623 | var stdLibPkg string | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 624 | var launcher android.Path | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 625 | if proptools.BoolDefault(p.properties.Is_internal, false) { | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 626 | stdLib = p.srcsZip | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 627 | stdLibPkg = p.getPkgPath() | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 628 | } else { | 
|  | 629 | ctx.VisitDirectDepsWithTag(hostStdLibTag, func(module android.Module) { | 
|  | 630 | if dep, ok := module.(pythonDependency); ok { | 
|  | 631 | stdLib = dep.getPrecompiledSrcsZip() | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 632 | stdLibPkg = dep.getPkgPath() | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 633 | } | 
|  | 634 | }) | 
|  | 635 | } | 
|  | 636 | ctx.VisitDirectDepsWithTag(hostLauncherTag, func(module android.Module) { | 
|  | 637 | if dep, ok := module.(IntermPathProvider); ok { | 
|  | 638 | optionalLauncher := dep.IntermPathForModuleOut() | 
|  | 639 | if optionalLauncher.Valid() { | 
|  | 640 | launcher = optionalLauncher.Path() | 
|  | 641 | } | 
| Cole Faust | 909d237 | 2023-02-13 23:17:40 +0000 | [diff] [blame] | 642 | } | 
|  | 643 | }) | 
|  | 644 | var launcherSharedLibs android.Paths | 
|  | 645 | var ldLibraryPath []string | 
|  | 646 | ctx.VisitDirectDepsWithTag(hostlauncherSharedLibTag, func(module android.Module) { | 
|  | 647 | if dep, ok := module.(IntermPathProvider); ok { | 
|  | 648 | optionalPath := dep.IntermPathForModuleOut() | 
|  | 649 | if optionalPath.Valid() { | 
|  | 650 | launcherSharedLibs = append(launcherSharedLibs, optionalPath.Path()) | 
|  | 651 | ldLibraryPath = append(ldLibraryPath, filepath.Dir(optionalPath.Path().String())) | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 652 | } | 
|  | 653 | } | 
|  | 654 | }) | 
|  | 655 |  | 
|  | 656 | out := android.PathForModuleOut(ctx, ctx.ModuleName()+".srcszipprecompiled") | 
|  | 657 | if stdLib == nil || launcher == nil { | 
|  | 658 | // This shouldn't happen in a real build because we'll error out when adding dependencies | 
|  | 659 | // on the stdlib and launcher if they don't exist. But some tests set | 
|  | 660 | // AllowMissingDependencies. | 
|  | 661 | return out | 
|  | 662 | } | 
|  | 663 | ctx.Build(pctx, android.BuildParams{ | 
|  | 664 | Rule:        precompile, | 
|  | 665 | Input:       p.srcsZip, | 
|  | 666 | Output:      out, | 
|  | 667 | Implicits:   launcherSharedLibs, | 
|  | 668 | Description: "Precompile the python sources of " + ctx.ModuleName(), | 
|  | 669 | Args: map[string]string{ | 
|  | 670 | "stdlibZip":     stdLib.String(), | 
| Dan Willemsen | 339a63f | 2023-08-15 22:17:03 -0400 | [diff] [blame] | 671 | "stdlibPkg":     stdLibPkg, | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 672 | "launcher":      launcher.String(), | 
|  | 673 | "ldLibraryPath": strings.Join(ldLibraryPath, ":"), | 
|  | 674 | }, | 
|  | 675 | }) | 
|  | 676 | return out | 
|  | 677 | } | 
|  | 678 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 679 | // isPythonLibModule returns whether the given module is a Python library PythonLibraryModule or not | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 680 | func isPythonLibModule(module blueprint.Module) bool { | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 681 | if _, ok := module.(*PythonLibraryModule); ok { | 
|  | 682 | if _, ok := module.(*PythonBinaryModule); !ok { | 
|  | 683 | return true | 
|  | 684 | } | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 685 | } | 
|  | 686 | return false | 
|  | 687 | } | 
|  | 688 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 689 | // collectPathsFromTransitiveDeps checks for source/data files for duplicate paths | 
|  | 690 | // for module and its transitive dependencies and collects list of data/source file | 
|  | 691 | // zips for transitive dependencies. | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 692 | func (p *PythonLibraryModule) collectPathsFromTransitiveDeps(ctx android.ModuleContext, precompiled bool) android.Paths { | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 693 | // fetch <runfiles_path, source_path> pairs from "src" and "data" properties to | 
|  | 694 | // check duplicates. | 
|  | 695 | destToPySrcs := make(map[string]string) | 
|  | 696 | destToPyData := make(map[string]string) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 697 | for _, path := range p.srcsPathMappings { | 
|  | 698 | destToPySrcs[path.dest] = path.src.String() | 
|  | 699 | } | 
|  | 700 | for _, path := range p.dataPathMappings { | 
|  | 701 | destToPyData[path.dest] = path.src.String() | 
|  | 702 | } | 
|  | 703 |  | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 704 | seen := make(map[android.Module]bool) | 
|  | 705 |  | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 706 | var result android.Paths | 
|  | 707 |  | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 708 | // visit all its dependencies in depth first. | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 709 | ctx.WalkDeps(func(child, parent android.Module) bool { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 710 | // we only collect dependencies tagged as python library deps | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 711 | if ctx.OtherModuleDependencyTag(child) != pythonLibTag { | 
|  | 712 | return false | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 713 | } | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 714 | if seen[child] { | 
|  | 715 | return false | 
|  | 716 | } | 
|  | 717 | seen[child] = true | 
| Nan Zhang | b8fa197 | 2017-12-22 16:12:00 -0800 | [diff] [blame] | 718 | // Python modules only can depend on Python libraries. | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 719 | if !isPythonLibModule(child) { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 720 | ctx.PropertyErrorf("libs", | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 721 | "the dependency %q of module %q is not Python library!", | 
| Alex Márquez Pérez Muñíz Díaz Púras Thaureaux | d75507f | 2021-08-20 21:02:43 +0000 | [diff] [blame] | 722 | ctx.OtherModuleName(child), ctx.ModuleName()) | 
| Nan Zhang | d4e641b | 2017-07-12 12:55:28 -0700 | [diff] [blame] | 723 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 724 | // collect source and data paths, checking that there are no duplicate output file conflicts | 
|  | 725 | if dep, ok := child.(pythonDependency); ok { | 
|  | 726 | srcs := dep.getSrcsPathMappings() | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 727 | for _, path := range srcs { | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 728 | checkForDuplicateOutputPath(ctx, destToPySrcs, | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 729 | path.dest, path.src.String(), ctx.ModuleName(), ctx.OtherModuleName(child)) | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 730 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 731 | data := dep.getDataPathMappings() | 
|  | 732 | for _, path := range data { | 
|  | 733 | checkForDuplicateOutputPath(ctx, destToPyData, | 
|  | 734 | path.dest, path.src.String(), ctx.ModuleName(), ctx.OtherModuleName(child)) | 
|  | 735 | } | 
| Cole Faust | 5c503d1 | 2023-01-24 11:48:08 -0800 | [diff] [blame] | 736 | if precompiled { | 
|  | 737 | result = append(result, dep.getPrecompiledSrcsZip()) | 
|  | 738 | } else { | 
|  | 739 | result = append(result, dep.getSrcsZip()) | 
|  | 740 | } | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 741 | } | 
| Colin Cross | 6b75360 | 2018-06-21 13:03:07 -0700 | [diff] [blame] | 742 | return true | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 743 | }) | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 744 | return result | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 745 | } | 
|  | 746 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 747 | // chckForDuplicateOutputPath checks whether outputPath has already been included in map m, which | 
|  | 748 | // would result in two files being placed in the same location. | 
|  | 749 | // If there is a duplicate path, an error is thrown and true is returned | 
|  | 750 | // Otherwise, outputPath: srcPath is added to m and returns false | 
|  | 751 | func checkForDuplicateOutputPath(ctx android.ModuleContext, m map[string]string, outputPath, srcPath, curModule, otherModule string) bool { | 
|  | 752 | if oldSrcPath, found := m[outputPath]; found { | 
| Nan Zhang | bea0975 | 2018-05-31 12:49:33 -0700 | [diff] [blame] | 753 | ctx.ModuleErrorf("found two files to be placed at the same location within zip %q."+ | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 754 | " First file: in module %s at path %q."+ | 
|  | 755 | " Second file: in module %s at path %q.", | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 756 | outputPath, curModule, oldSrcPath, otherModule, srcPath) | 
|  | 757 | return true | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 758 | } | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 759 | m[outputPath] = srcPath | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 760 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 761 | return false | 
| Nan Zhang | db0b9a3 | 2017-02-27 10:12:13 -0800 | [diff] [blame] | 762 | } | 
| Nan Zhang | ea568a4 | 2017-11-08 21:20:04 -0800 | [diff] [blame] | 763 |  | 
| Liz Kammer | d737d02 | 2020-11-16 15:42:51 -0800 | [diff] [blame] | 764 | // InstallInData returns true as Python is not supported in the system partition | 
| Cole Faust | 4d247e6 | 2023-01-23 10:14:58 -0800 | [diff] [blame] | 765 | func (p *PythonLibraryModule) InstallInData() bool { | 
| Nan Zhang | d9ec5e7 | 2017-12-01 20:00:31 +0000 | [diff] [blame] | 766 | return true | 
|  | 767 | } | 
|  | 768 |  | 
| Nan Zhang | ea568a4 | 2017-11-08 21:20:04 -0800 | [diff] [blame] | 769 | var Bool = proptools.Bool | 
| Dan Willemsen | 6ca390f | 2019-02-14 23:17:08 -0800 | [diff] [blame] | 770 | var BoolDefault = proptools.BoolDefault | 
| Nan Zhang | ea568a4 | 2017-11-08 21:20:04 -0800 | [diff] [blame] | 771 | var String = proptools.String |