Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | # |
| 3 | # Copyright (c) 2019, Nordic Semiconductor ASA |
| 4 | # |
| 5 | # SPDX-License-Identifier: Apache-2.0 |
| 6 | |
| 7 | '''Tool for parsing a list of projects to determine if they are Zephyr |
| 8 | projects. If no projects are given then the output from `west list` will be |
| 9 | used as project list. |
| 10 | |
| 11 | Include file is generated for Kconfig using --kconfig-out. |
| 12 | A <name>:<path> text file is generated for use with CMake using --cmake-out. |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 13 | |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 14 | Using --twister-out <filename> an argument file for twister script will |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 15 | be generated which would point to test and sample roots available in modules |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 16 | that can be included during a twister run. This allows testing code |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 17 | maintained in modules in addition to what is available in the main Zephyr tree. |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 18 | ''' |
| 19 | |
| 20 | import argparse |
| 21 | import os |
| 22 | import sys |
| 23 | import yaml |
| 24 | import pykwalify.core |
Torsten Rasmussen | b3da9ef | 2019-12-13 09:42:13 +0100 | [diff] [blame] | 25 | from pathlib import Path, PurePath |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 26 | from collections import namedtuple |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 27 | |
| 28 | METADATA_SCHEMA = ''' |
| 29 | ## A pykwalify schema for basic validation of the structure of a |
| 30 | ## metadata YAML file. |
| 31 | ## |
| 32 | # The zephyr/module.yml file is a simple list of key value pairs to be used by |
| 33 | # the build system. |
| 34 | type: map |
| 35 | mapping: |
| 36 | build: |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 37 | required: false |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 38 | type: map |
| 39 | mapping: |
| 40 | cmake: |
| 41 | required: false |
| 42 | type: str |
| 43 | kconfig: |
| 44 | required: false |
| 45 | type: str |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 46 | depends: |
| 47 | required: false |
| 48 | type: seq |
| 49 | sequence: |
| 50 | - type: str |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 51 | settings: |
| 52 | required: false |
| 53 | type: map |
| 54 | mapping: |
| 55 | board_root: |
| 56 | required: false |
| 57 | type: str |
| 58 | dts_root: |
| 59 | required: false |
| 60 | type: str |
| 61 | soc_root: |
| 62 | required: false |
| 63 | type: str |
| 64 | arch_root: |
| 65 | required: false |
| 66 | type: str |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 67 | tests: |
| 68 | required: false |
| 69 | type: seq |
| 70 | sequence: |
| 71 | - type: str |
| 72 | samples: |
| 73 | required: false |
| 74 | type: seq |
| 75 | sequence: |
| 76 | - type: str |
| 77 | boards: |
| 78 | required: false |
| 79 | type: seq |
| 80 | sequence: |
| 81 | - type: str |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 82 | ''' |
| 83 | |
| 84 | schema = yaml.safe_load(METADATA_SCHEMA) |
| 85 | |
| 86 | |
| 87 | def validate_setting(setting, module_path, filename=None): |
| 88 | if setting is not None: |
| 89 | if filename is not None: |
| 90 | checkfile = os.path.join(module_path, setting, filename) |
| 91 | else: |
| 92 | checkfile = os.path.join(module_path, setting) |
| 93 | if not os.path.isfile(checkfile): |
| 94 | return False |
| 95 | return True |
| 96 | |
| 97 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 98 | def process_module(module): |
Torsten Rasmussen | b3da9ef | 2019-12-13 09:42:13 +0100 | [diff] [blame] | 99 | module_path = PurePath(module) |
| 100 | module_yml = module_path.joinpath('zephyr/module.yml') |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 101 | |
Sebastian Bøe | bb95dce | 2019-12-19 15:49:22 +0100 | [diff] [blame] | 102 | # The input is a module if zephyr/module.yml is a valid yaml file |
| 103 | # or if both zephyr/CMakeLists.txt and zephyr/Kconfig are present. |
| 104 | |
Torsten Rasmussen | b3da9ef | 2019-12-13 09:42:13 +0100 | [diff] [blame] | 105 | if Path(module_yml).is_file(): |
| 106 | with Path(module_yml).open('r') as f: |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 107 | meta = yaml.safe_load(f.read()) |
| 108 | |
| 109 | try: |
| 110 | pykwalify.core.Core(source_data=meta, schema_data=schema)\ |
| 111 | .validate() |
| 112 | except pykwalify.errors.SchemaError as e: |
Ulf Magnusson | 50b9b12 | 2019-09-07 14:41:01 +0200 | [diff] [blame] | 113 | sys.exit('ERROR: Malformed "build" section in file: {}\n{}' |
Torsten Rasmussen | b3da9ef | 2019-12-13 09:42:13 +0100 | [diff] [blame] | 114 | .format(module_yml.as_posix(), e)) |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 115 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 116 | return meta |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 117 | |
Sebastian Bøe | bb95dce | 2019-12-19 15:49:22 +0100 | [diff] [blame] | 118 | if Path(module_path.joinpath('zephyr/CMakeLists.txt')).is_file() and \ |
| 119 | Path(module_path.joinpath('zephyr/Kconfig')).is_file(): |
| 120 | return {'build': {'cmake': 'zephyr', 'kconfig': 'zephyr/Kconfig'}} |
| 121 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 122 | return None |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 123 | |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 124 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 125 | def process_cmake(module, meta): |
| 126 | section = meta.get('build', dict()) |
| 127 | module_path = PurePath(module) |
| 128 | module_yml = module_path.joinpath('zephyr/module.yml') |
| 129 | cmake_setting = section.get('cmake', None) |
| 130 | if not validate_setting(cmake_setting, module, 'CMakeLists.txt'): |
| 131 | sys.exit('ERROR: "cmake" key in {} has folder value "{}" which ' |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 132 | 'does not contain a CMakeLists.txt file.' |
| 133 | .format(module_yml.as_posix(), cmake_setting)) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 134 | |
| 135 | cmake_path = os.path.join(module, cmake_setting or 'zephyr') |
| 136 | cmake_file = os.path.join(cmake_path, 'CMakeLists.txt') |
| 137 | if os.path.isfile(cmake_file): |
Torsten Rasmussen | ab7ec17 | 2020-08-25 13:32:33 +0200 | [diff] [blame] | 138 | return('\"{}\":\"{}\":\"{}\"\n' |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 139 | .format(module_path.name, |
Torsten Rasmussen | ab7ec17 | 2020-08-25 13:32:33 +0200 | [diff] [blame] | 140 | module_path.as_posix(), |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 141 | Path(cmake_path).resolve().as_posix())) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 142 | else: |
Torsten Rasmussen | ab7ec17 | 2020-08-25 13:32:33 +0200 | [diff] [blame] | 143 | return('\"{}\":\"{}\":\"\"\n' |
| 144 | .format(module_path.name, |
| 145 | module_path.as_posix())) |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 146 | |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 147 | def process_settings(module, meta): |
| 148 | section = meta.get('build', dict()) |
| 149 | build_settings = section.get('settings', None) |
| 150 | out_text = "" |
| 151 | |
| 152 | if build_settings is not None: |
| 153 | for root in ['board', 'dts', 'soc', 'arch']: |
| 154 | setting = build_settings.get(root+'_root', None) |
| 155 | if setting is not None: |
| 156 | root_path = PurePath(module) / setting |
Torsten Rasmussen | fef7879 | 2020-10-15 22:16:41 +0200 | [diff] [blame] | 157 | out_text += f'"{root.upper()}_ROOT":"{root_path.as_posix()}"\n' |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 158 | |
| 159 | return out_text |
| 160 | |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 161 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 162 | def process_kconfig(module, meta): |
| 163 | section = meta.get('build', dict()) |
| 164 | module_path = PurePath(module) |
| 165 | module_yml = module_path.joinpath('zephyr/module.yml') |
| 166 | |
| 167 | kconfig_setting = section.get('kconfig', None) |
| 168 | if not validate_setting(kconfig_setting, module): |
| 169 | sys.exit('ERROR: "kconfig" key in {} has value "{}" which does ' |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 170 | 'not point to a valid Kconfig file.' |
| 171 | .format(module_yml, kconfig_setting)) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 172 | |
| 173 | kconfig_file = os.path.join(module, kconfig_setting or 'zephyr/Kconfig') |
| 174 | if os.path.isfile(kconfig_file): |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 175 | return 'osource "{}"\n\n'.format(Path(kconfig_file) |
| 176 | .resolve().as_posix()) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 177 | else: |
| 178 | return "" |
| 179 | |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 180 | def process_twister(module, meta): |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 181 | |
| 182 | out = "" |
| 183 | tests = meta.get('tests', []) |
| 184 | samples = meta.get('samples', []) |
| 185 | boards = meta.get('boards', []) |
| 186 | |
| 187 | for pth in tests + samples: |
| 188 | if pth: |
| 189 | dir = os.path.join(module, pth) |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 190 | out += '-T\n{}\n'.format(PurePath(os.path.abspath(dir)) |
| 191 | .as_posix()) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 192 | |
| 193 | for pth in boards: |
| 194 | if pth: |
| 195 | dir = os.path.join(module, pth) |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 196 | out += '--board-root\n{}\n'.format(PurePath(os.path.abspath(dir)) |
| 197 | .as_posix()) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 198 | |
| 199 | return out |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 200 | |
| 201 | |
| 202 | def main(): |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 203 | parser = argparse.ArgumentParser(description=''' |
| 204 | Process a list of projects and create Kconfig / CMake include files for |
| 205 | projects which are also a Zephyr module''') |
| 206 | |
| 207 | parser.add_argument('--kconfig-out', |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 208 | help="""File to write with resulting KConfig import |
| 209 | statements.""") |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 210 | parser.add_argument('--twister-out', |
| 211 | help="""File to write with resulting twister |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 212 | parameters.""") |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 213 | parser.add_argument('--cmake-out', |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 214 | help="""File to write with resulting <name>:<path> |
| 215 | values to use for including in CMake""") |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 216 | parser.add_argument('--settings-out', |
| 217 | help="""File to write with resulting <name>:<value> |
| 218 | values to use for including in CMake""") |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 219 | parser.add_argument('-m', '--modules', nargs='+', |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 220 | help="""List of modules to parse instead of using `west |
| 221 | list`""") |
Martí Bolívar | 34346c4 | 2020-01-29 12:38:58 -0800 | [diff] [blame] | 222 | parser.add_argument('-x', '--extra-modules', nargs='+', default=[], |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 223 | help='List of extra modules to parse') |
Torsten Rasmussen | 39cd4c8 | 2020-02-19 12:01:39 +0100 | [diff] [blame] | 224 | parser.add_argument('-z', '--zephyr-base', |
| 225 | help='Path to zephyr repository') |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 226 | args = parser.parse_args() |
| 227 | |
| 228 | if args.modules is None: |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 229 | # West is imported here, as it is optional |
| 230 | # (and thus maybe not installed) |
Torsten Rasmussen | ef3c5e5 | 2020-06-08 21:09:15 +0200 | [diff] [blame] | 231 | # if user is providing a specific modules list. |
| 232 | from west.manifest import Manifest |
| 233 | from west.util import WestNotFound |
| 234 | try: |
| 235 | manifest = Manifest.from_file() |
| 236 | projects = [p.posixpath for p in manifest.get_projects([])] |
| 237 | except WestNotFound: |
| 238 | # Only accept WestNotFound, meaning we are not in a west |
| 239 | # workspace. Such setup is allowed, as west may be installed |
| 240 | # but the project is not required to use west. |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 241 | projects = [] |
Carles Cufi | 02dea92 | 2020-07-20 14:09:12 +0200 | [diff] [blame] | 242 | else: |
| 243 | projects = args.modules.copy() |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 244 | |
Martí Bolívar | 34346c4 | 2020-01-29 12:38:58 -0800 | [diff] [blame] | 245 | projects += args.extra_modules |
| 246 | extra_modules = set(args.extra_modules) |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 247 | |
| 248 | kconfig = "" |
| 249 | cmake = "" |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 250 | settings = "" |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 251 | twister = "" |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 252 | |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 253 | Module = namedtuple('Module', ['project', 'meta', 'depends']) |
| 254 | # dep_modules is a list of all modules that has an unresolved dependency |
| 255 | dep_modules = [] |
| 256 | # start_modules is a list modules with no depends left (no incoming edge) |
| 257 | start_modules = [] |
| 258 | # sorted_modules is a topological sorted list of the modules |
| 259 | sorted_modules = [] |
| 260 | |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 261 | for project in projects: |
| 262 | # Avoid including Zephyr base project as module. |
Torsten Rasmussen | ef3c5e5 | 2020-06-08 21:09:15 +0200 | [diff] [blame] | 263 | if project == args.zephyr_base: |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 264 | continue |
| 265 | |
| 266 | meta = process_module(project) |
| 267 | if meta: |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 268 | section = meta.get('build', dict()) |
| 269 | deps = section.get('depends', []) |
| 270 | if not deps: |
| 271 | start_modules.append(Module(project, meta, [])) |
| 272 | else: |
| 273 | dep_modules.append(Module(project, meta, deps)) |
Martí Bolívar | 34346c4 | 2020-01-29 12:38:58 -0800 | [diff] [blame] | 274 | elif project in extra_modules: |
| 275 | sys.exit(f'{project}, given in ZEPHYR_EXTRA_MODULES, ' |
| 276 | 'is not a valid zephyr module') |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 277 | |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 278 | # This will do a topological sort to ensure the modules are ordered |
| 279 | # according to dependency settings. |
| 280 | while start_modules: |
| 281 | node = start_modules.pop(0) |
| 282 | sorted_modules.append(node) |
| 283 | node_name = PurePath(node.project).name |
| 284 | to_remove = [] |
| 285 | for module in dep_modules: |
| 286 | if node_name in module.depends: |
| 287 | module.depends.remove(node_name) |
| 288 | if not module.depends: |
| 289 | start_modules.append(module) |
| 290 | to_remove.append(module) |
| 291 | for module in to_remove: |
| 292 | dep_modules.remove(module) |
| 293 | |
| 294 | if dep_modules: |
| 295 | # If there are any modules with unresolved dependencies, then the |
| 296 | # modules contains unmet or cyclic dependencies. Error out. |
| 297 | error = 'Unmet or cyclic dependencies in modules:\n' |
| 298 | for module in dep_modules: |
| 299 | error += f'{module.project} depends on: {module.depends}\n' |
| 300 | sys.exit(error) |
| 301 | |
| 302 | for module in sorted_modules: |
| 303 | kconfig += process_kconfig(module.project, module.meta) |
| 304 | cmake += process_cmake(module.project, module.meta) |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 305 | settings += process_settings(module.project, module.meta) |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 306 | twister += process_twister(module.project, module.meta) |
Torsten Rasmussen | 3917ee5 | 2020-05-18 22:34:49 +0200 | [diff] [blame] | 307 | |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 308 | if args.kconfig_out: |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 309 | with open(args.kconfig_out, 'w', encoding="utf-8") as fp: |
| 310 | fp.write(kconfig) |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 311 | |
| 312 | if args.cmake_out: |
Anas Nashif | 286a9ed | 2019-12-11 10:13:23 -0500 | [diff] [blame] | 313 | with open(args.cmake_out, 'w', encoding="utf-8") as fp: |
| 314 | fp.write(cmake) |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 315 | |
Torsten Rasmussen | 25d57ba | 2020-07-07 17:29:56 +0200 | [diff] [blame] | 316 | if args.settings_out: |
| 317 | with open(args.settings_out, 'w', encoding="utf-8") as fp: |
| 318 | fp.write(settings) |
| 319 | |
Anas Nashif | 5471398 | 2020-12-07 14:52:10 -0500 | [diff] [blame] | 320 | if args.twister_out: |
| 321 | with open(args.twister_out, 'w', encoding="utf-8") as fp: |
| 322 | fp.write(twister) |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 323 | |
Torsten Rasmussen | 25c3f8e | 2020-08-25 14:02:04 +0200 | [diff] [blame] | 324 | |
Torsten Rasmussen | bd7569f | 2019-03-19 10:38:18 +0100 | [diff] [blame] | 325 | if __name__ == "__main__": |
| 326 | main() |