blob: 700e11718ec98ff45e7243cfdace3d8ef22ee4ee [file] [log] [blame]
Andrew Jeffery89b64b62020-03-13 12:15:48 +10301#!/usr/bin/env python3
Matthew Barthccb7f852016-11-23 17:43:02 -06002
3"""
4This script determines the given package's openbmc dependencies from its
5configure.ac file where it downloads, configures, builds, and installs each of
6these dependencies. Then the given package is configured, built, and installed
7prior to executing its unit tests.
8"""
9
Patrick Williamse08ffba2022-12-05 10:33:46 -060010import argparse
Andrew Jefferybf83a1b2024-05-15 22:05:32 +093011import json
Patrick Williamse08ffba2022-12-05 10:33:46 -060012import multiprocessing
13import os
14import platform
15import re
16import shutil
17import subprocess
18import sys
19from subprocess import CalledProcessError, check_call
20from tempfile import TemporaryDirectory
21from urllib.parse import urljoin
22
Matthew Barthd1810372016-12-19 16:57:21 -060023from git import Repo
Patrick Williamse795dfe2022-12-06 10:07:02 -060024
William A. Kennington III3992d102021-05-17 01:41:04 -070025# interpreter is not used directly but this resolves dependency ordering
26# that would be broken if we didn't include it.
Patrick Williamse08ffba2022-12-05 10:33:46 -060027from mesonbuild import interpreter # noqa: F401
William A. Kennington IIIfcd70772020-06-04 00:50:23 -070028from mesonbuild import coredata, optinterpreter
Patrick Williamse95626d2021-04-13 11:13:53 -050029from mesonbuild.mesonlib import OptionKey
Patrick Williams95095f12021-04-14 14:42:35 -050030from mesonbuild.mesonlib import version_compare as meson_version_compare
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -050031
32
Patrick Williamse08ffba2022-12-05 10:33:46 -060033class DepTree:
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -050034 """
35 Represents package dependency tree, where each node is a DepTree with a
36 name and DepTree children.
37 """
38
39 def __init__(self, name):
40 """
41 Create new DepTree.
42
43 Parameter descriptions:
44 name Name of new tree node.
45 """
46 self.name = name
47 self.children = list()
48
49 def AddChild(self, name):
50 """
51 Add new child node to current node.
52
53 Parameter descriptions:
54 name Name of new child
55 """
56 new_child = DepTree(name)
57 self.children.append(new_child)
58 return new_child
59
60 def AddChildNode(self, node):
61 """
62 Add existing child node to current node.
63
64 Parameter descriptions:
65 node Tree node to add
66 """
67 self.children.append(node)
68
69 def RemoveChild(self, name):
70 """
71 Remove child node.
72
73 Parameter descriptions:
74 name Name of child to remove
75 """
76 for child in self.children:
77 if child.name == name:
78 self.children.remove(child)
79 return
80
81 def GetNode(self, name):
82 """
83 Return node with matching name. Return None if not found.
84
85 Parameter descriptions:
86 name Name of node to return
87 """
88 if self.name == name:
89 return self
90 for child in self.children:
91 node = child.GetNode(name)
92 if node:
93 return node
94 return None
95
96 def GetParentNode(self, name, parent_node=None):
97 """
98 Return parent of node with matching name. Return none if not found.
99
100 Parameter descriptions:
101 name Name of node to get parent of
102 parent_node Parent of current node
103 """
104 if self.name == name:
105 return parent_node
106 for child in self.children:
107 found_node = child.GetParentNode(name, self)
108 if found_node:
109 return found_node
110 return None
111
112 def GetPath(self, name, path=None):
113 """
114 Return list of node names from head to matching name.
115 Return None if not found.
116
117 Parameter descriptions:
118 name Name of node
119 path List of node names from head to current node
120 """
121 if not path:
122 path = []
123 if self.name == name:
124 path.append(self.name)
125 return path
126 for child in self.children:
127 match = child.GetPath(name, path + [self.name])
128 if match:
129 return match
130 return None
131
132 def GetPathRegex(self, name, regex_str, path=None):
133 """
134 Return list of node paths that end in name, or match regex_str.
135 Return empty list if not found.
136
137 Parameter descriptions:
138 name Name of node to search for
139 regex_str Regex string to match node names
140 path Path of node names from head to current node
141 """
142 new_paths = []
143 if not path:
144 path = []
145 match = re.match(regex_str, self.name)
146 if (self.name == name) or (match):
147 new_paths.append(path + [self.name])
148 for child in self.children:
149 return_paths = None
150 full_path = path + [self.name]
151 return_paths = child.GetPathRegex(name, regex_str, full_path)
152 for i in return_paths:
153 new_paths.append(i)
154 return new_paths
155
156 def MoveNode(self, from_name, to_name):
157 """
158 Mode existing from_name node to become child of to_name node.
159
160 Parameter descriptions:
161 from_name Name of node to make a child of to_name
162 to_name Name of node to make parent of from_name
163 """
164 parent_from_node = self.GetParentNode(from_name)
165 from_node = self.GetNode(from_name)
166 parent_from_node.RemoveChild(from_name)
167 to_node = self.GetNode(to_name)
168 to_node.AddChildNode(from_node)
169
170 def ReorderDeps(self, name, regex_str):
171 """
172 Reorder dependency tree. If tree contains nodes with names that
173 match 'name' and 'regex_str', move 'regex_str' nodes that are
174 to the right of 'name' node, so that they become children of the
175 'name' node.
176
177 Parameter descriptions:
178 name Name of node to look for
179 regex_str Regex string to match names to
180 """
181 name_path = self.GetPath(name)
182 if not name_path:
183 return
184 paths = self.GetPathRegex(name, regex_str)
185 is_name_in_paths = False
186 name_index = 0
187 for i in range(len(paths)):
188 path = paths[i]
189 if path[-1] == name:
190 is_name_in_paths = True
191 name_index = i
192 break
193 if not is_name_in_paths:
194 return
195 for i in range(name_index + 1, len(paths)):
196 path = paths[i]
197 if name in path:
198 continue
199 from_name = path[-1]
200 self.MoveNode(from_name, name)
201
202 def GetInstallList(self):
203 """
204 Return post-order list of node names.
205
206 Parameter descriptions:
207 """
208 install_list = []
209 for child in self.children:
210 child_install_list = child.GetInstallList()
211 install_list.extend(child_install_list)
212 install_list.append(self.name)
213 return install_list
214
215 def PrintTree(self, level=0):
216 """
217 Print pre-order node names with indentation denoting node depth level.
218
219 Parameter descriptions:
220 level Current depth level
221 """
222 INDENT_PER_LEVEL = 4
Patrick Williamse08ffba2022-12-05 10:33:46 -0600223 print(" " * (level * INDENT_PER_LEVEL) + self.name)
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500224 for child in self.children:
225 child.PrintTree(level + 1)
Matthew Barth33df8792016-12-19 14:30:17 -0600226
227
Andrew Jeffery71924fd2023-04-12 23:27:05 +0930228def check_call_cmd(*cmd, **kwargs):
Matthew Barth33df8792016-12-19 14:30:17 -0600229 """
230 Verbose prints the directory location the given command is called from and
231 the command, then executes the command using check_call.
232
233 Parameter descriptions:
234 dir Directory location command is to be called from
235 cmd List of parameters constructing the complete command
236 """
William A. Kennington III1fddb972019-02-06 18:03:53 -0800237 printline(os.getcwd(), ">", " ".join(cmd))
Andrew Jeffery71924fd2023-04-12 23:27:05 +0930238 check_call(cmd, **kwargs)
Matthew Barthccb7f852016-11-23 17:43:02 -0600239
240
Andrew Geisslera61acb52019-01-03 16:32:44 -0600241def clone_pkg(pkg, branch):
Matthew Barth33df8792016-12-19 14:30:17 -0600242 """
243 Clone the given openbmc package's git repository from gerrit into
244 the WORKSPACE location
245
246 Parameter descriptions:
247 pkg Name of the package to clone
Andrew Geisslera61acb52019-01-03 16:32:44 -0600248 branch Branch to clone from pkg
Matthew Barth33df8792016-12-19 14:30:17 -0600249 """
Andrew Jeffery7be94ca2018-03-08 13:15:33 +1030250 pkg_dir = os.path.join(WORKSPACE, pkg)
Patrick Williamse08ffba2022-12-05 10:33:46 -0600251 if os.path.exists(os.path.join(pkg_dir, ".git")):
Andrew Jeffery7be94ca2018-03-08 13:15:33 +1030252 return pkg_dir
Patrick Williamse08ffba2022-12-05 10:33:46 -0600253 pkg_repo = urljoin("https://gerrit.openbmc.org/openbmc/", pkg)
Andrew Jeffery7be94ca2018-03-08 13:15:33 +1030254 os.mkdir(pkg_dir)
Andrew Geisslera61acb52019-01-03 16:32:44 -0600255 printline(pkg_dir, "> git clone", pkg_repo, branch, "./")
256 try:
257 # first try the branch
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030258 clone = Repo.clone_from(pkg_repo, pkg_dir, branch=branch)
259 repo_inst = clone.working_dir
Patrick Williamse08ffba2022-12-05 10:33:46 -0600260 except Exception:
Andrew Geisslera61acb52019-01-03 16:32:44 -0600261 printline("Input branch not found, default to master")
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030262 clone = Repo.clone_from(pkg_repo, pkg_dir, branch="master")
263 repo_inst = clone.working_dir
Andrew Geisslera61acb52019-01-03 16:32:44 -0600264 return repo_inst
Matthew Barth33df8792016-12-19 14:30:17 -0600265
266
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030267def make_target_exists(target):
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800268 """
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030269 Runs a check against the makefile in the current directory to determine
270 if the target exists so that it can be built.
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800271
272 Parameter descriptions:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030273 target The make target we are checking
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800274 """
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030275 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600276 cmd = ["make", "-n", target]
277 with open(os.devnull, "w") as devnull:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030278 check_call(cmd, stdout=devnull, stderr=devnull)
279 return True
280 except CalledProcessError:
281 return False
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800282
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800283
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700284make_parallel = [
Patrick Williamse08ffba2022-12-05 10:33:46 -0600285 "make",
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700286 # Run enough jobs to saturate all the cpus
Patrick Williamse08ffba2022-12-05 10:33:46 -0600287 "-j",
288 str(multiprocessing.cpu_count()),
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700289 # Don't start more jobs if the load avg is too high
Patrick Williamse08ffba2022-12-05 10:33:46 -0600290 "-l",
291 str(multiprocessing.cpu_count()),
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700292 # Synchronize the output so logs aren't intermixed in stdout / stderr
Patrick Williamse08ffba2022-12-05 10:33:46 -0600293 "-O",
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700294]
295
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800296
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030297def build_and_install(name, build_for_testing=False):
William A. Kennington III780ec092018-12-06 14:46:50 -0800298 """
299 Builds and installs the package in the environment. Optionally
300 builds the examples and test cases for package.
301
302 Parameter description:
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030303 name The name of the package we are building
William A. Kennington IIIa0454912018-12-06 14:47:16 -0800304 build_for_testing Enable options related to testing on the package?
William A. Kennington III780ec092018-12-06 14:46:50 -0800305 """
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030306 os.chdir(os.path.join(WORKSPACE, name))
William A. Kennington III54d4faf2018-12-06 17:46:24 -0800307
308 # Refresh dynamic linker run time bindings for dependencies
Patrick Williamse08ffba2022-12-05 10:33:46 -0600309 check_call_cmd("sudo", "-n", "--", "ldconfig")
William A. Kennington III54d4faf2018-12-06 17:46:24 -0800310
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030311 pkg = Package()
312 if build_for_testing:
313 pkg.test()
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800314 else:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030315 pkg.install()
316
William A. Kennington III780ec092018-12-06 14:46:50 -0800317
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030318def build_dep_tree(name, pkgdir, dep_added, head, branch, dep_tree=None):
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500319 """
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030320 For each package (name), starting with the package to be unit tested,
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030321 extract its dependencies. For each package dependency defined, recursively
322 apply the same strategy
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500323
324 Parameter descriptions:
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030325 name Name of the package
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500326 pkgdir Directory where package source is located
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800327 dep_added Current dict of dependencies and added status
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500328 head Head node of the dependency tree
Andrew Geisslera61acb52019-01-03 16:32:44 -0600329 branch Branch to clone from pkg
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500330 dep_tree Current dependency tree node
331 """
332 if not dep_tree:
333 dep_tree = head
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800334
William A. Kennington IIIbe6aab22018-12-06 15:01:54 -0800335 with open("/tmp/depcache", "r") as depcache:
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800336 cache = depcache.readline()
337
338 # Read out pkg dependencies
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030339 pkg = Package(name, pkgdir)
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800340
Patrick Williams38c6b7d2021-02-18 07:26:14 -0600341 build = pkg.build_system()
Patrick Williamse08ffba2022-12-05 10:33:46 -0600342 if not build:
Patrick Williams38c6b7d2021-02-18 07:26:14 -0600343 raise Exception(f"Unable to find build system for {name}.")
344
345 for dep in set(build.dependencies()):
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800346 if dep in cache:
347 continue
348 # Dependency package not already known
349 if dep_added.get(dep) is None:
Patrick Williams38c6b7d2021-02-18 07:26:14 -0600350 print(f"Adding {dep} dependency to {name}.")
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800351 # Dependency package not added
352 new_child = dep_tree.AddChild(dep)
353 dep_added[dep] = False
Andrew Jeffery3b92fdd2020-03-13 11:49:18 +1030354 dep_pkgdir = clone_pkg(dep, branch)
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800355 # Determine this dependency package's
356 # dependencies and add them before
357 # returning to add this package
Patrick Williamse08ffba2022-12-05 10:33:46 -0600358 dep_added = build_dep_tree(
359 dep, dep_pkgdir, dep_added, head, branch, new_child
360 )
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800361 else:
362 # Dependency package known and added
363 if dep_added[dep]:
Andrew Jeffery2cb0c7a2018-03-08 13:19:08 +1030364 continue
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500365 else:
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800366 # Cyclic dependency failure
Patrick Williamse08ffba2022-12-05 10:33:46 -0600367 raise Exception("Cyclic dependencies found in " + name)
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500368
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030369 if not dep_added[name]:
370 dep_added[name] = True
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500371
372 return dep_added
Matthew Barthccb7f852016-11-23 17:43:02 -0600373
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700374
William A. Kennington III90b106a2019-02-06 18:08:24 -0800375def run_cppcheck():
Ed Tanousc7198552022-07-01 08:15:50 -0700376 if not os.path.exists(os.path.join("build", "compile_commands.json")):
Brad Bishop48424d42020-01-07 13:01:31 -0500377 return None
378
Patrick Williams485a0922022-08-12 11:44:55 -0500379 with TemporaryDirectory() as cpp_dir:
Patrick Williams485a0922022-08-12 11:44:55 -0500380 # http://cppcheck.sourceforge.net/manual.pdf
381 try:
382 check_call_cmd(
Patrick Williamse08ffba2022-12-05 10:33:46 -0600383 "cppcheck",
384 "-j",
385 str(multiprocessing.cpu_count()),
386 "--enable=style,performance,portability,missingInclude",
Brad Bishop688f8372023-03-02 22:56:31 -0500387 "--inline-suppr",
Patrick Williamse08ffba2022-12-05 10:33:46 -0600388 "--suppress=useStlAlgorithm",
389 "--suppress=unusedStructMember",
390 "--suppress=postfixOperator",
391 "--suppress=unreadVariable",
392 "--suppress=knownConditionTrueFalse",
393 "--library=googletest",
394 "--project=build/compile_commands.json",
395 f"--cppcheck-build-dir={cpp_dir}",
Patrick Williams485a0922022-08-12 11:44:55 -0500396 )
397 except subprocess.CalledProcessError:
398 print("cppcheck found errors")
Lei YUdbd7cd62022-07-19 19:24:01 +0800399
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030400
William A. Kennington III37a89a22018-12-13 14:32:02 -0800401def is_valgrind_safe():
402 """
403 Returns whether it is safe to run valgrind on our platform
404 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600405 src = "unit-test-vg.c"
406 exe = "./unit-test-vg"
407 with open(src, "w") as h:
408 h.write("#include <errno.h>\n")
409 h.write("#include <stdio.h>\n")
410 h.write("#include <stdlib.h>\n")
411 h.write("#include <string.h>\n")
412 h.write("int main() {\n")
413 h.write("char *heap_str = malloc(16);\n")
William A. Kennington III0326ded2019-02-07 00:33:28 -0800414 h.write('strcpy(heap_str, "RandString");\n')
415 h.write('int res = strcmp("RandString", heap_str);\n')
Patrick Williamse08ffba2022-12-05 10:33:46 -0600416 h.write("free(heap_str);\n")
417 h.write("char errstr[64];\n")
418 h.write("strerror_r(EINVAL, errstr, sizeof(errstr));\n")
William A. Kennington IIIafb0f982019-04-26 17:30:28 -0700419 h.write('printf("%s\\n", errstr);\n')
Patrick Williamse08ffba2022-12-05 10:33:46 -0600420 h.write("return res;\n")
421 h.write("}\n")
William A. Kennington III0326ded2019-02-07 00:33:28 -0800422 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600423 with open(os.devnull, "w") as devnull:
424 check_call(
425 ["gcc", "-O2", "-o", exe, src], stdout=devnull, stderr=devnull
426 )
427 check_call(
428 ["valgrind", "--error-exitcode=99", exe],
429 stdout=devnull,
430 stderr=devnull,
431 )
William A. Kennington III0326ded2019-02-07 00:33:28 -0800432 return True
Patrick Williamse08ffba2022-12-05 10:33:46 -0600433 except Exception:
William A. Kennington III0326ded2019-02-07 00:33:28 -0800434 sys.stderr.write("###### Platform is not valgrind safe ######\n")
435 return False
436 finally:
437 os.remove(src)
438 os.remove(exe)
William A. Kennington III37a89a22018-12-13 14:32:02 -0800439
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030440
William A. Kennington III282e3302019-02-04 16:55:05 -0800441def is_sanitize_safe():
442 """
443 Returns whether it is safe to run sanitizers on our platform
444 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600445 src = "unit-test-sanitize.c"
446 exe = "./unit-test-sanitize"
447 with open(src, "w") as h:
448 h.write("int main() { return 0; }\n")
William A. Kennington III0b7fb2b2019-02-07 00:33:42 -0800449 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600450 with open(os.devnull, "w") as devnull:
451 check_call(
452 [
453 "gcc",
454 "-O2",
455 "-fsanitize=address",
456 "-fsanitize=undefined",
457 "-o",
458 exe,
459 src,
460 ],
461 stdout=devnull,
462 stderr=devnull,
463 )
William A. Kennington III0b7fb2b2019-02-07 00:33:42 -0800464 check_call([exe], stdout=devnull, stderr=devnull)
Andrew Geisslercd9578b2021-05-04 10:18:01 -0500465
466 # TODO - Sanitizer not working on ppc64le
467 # https://github.com/openbmc/openbmc-build-scripts/issues/31
Patrick Williamse08ffba2022-12-05 10:33:46 -0600468 if platform.processor() == "ppc64le":
Andrew Geisslercd9578b2021-05-04 10:18:01 -0500469 sys.stderr.write("###### ppc64le is not sanitize safe ######\n")
470 return False
471 else:
472 return True
Patrick Williamse08ffba2022-12-05 10:33:46 -0600473 except Exception:
William A. Kennington III0b7fb2b2019-02-07 00:33:42 -0800474 sys.stderr.write("###### Platform is not sanitize safe ######\n")
475 return False
476 finally:
477 os.remove(src)
478 os.remove(exe)
William A. Kennington III282e3302019-02-04 16:55:05 -0800479
William A. Kennington III49d4e592019-02-06 17:59:27 -0800480
William A. Kennington IIIeaff24a2019-02-06 16:57:42 -0800481def maybe_make_valgrind():
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700482 """
483 Potentially runs the unit tests through valgrind for the package
484 via `make check-valgrind`. If the package does not have valgrind testing
485 then it just skips over this.
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700486 """
William A. Kennington III4e1d0a12018-07-16 12:04:03 -0700487 # Valgrind testing is currently broken by an aggressive strcmp optimization
488 # that is inlined into optimized code for POWER by gcc 7+. Until we find
489 # a workaround, just don't run valgrind tests on POWER.
490 # https://github.com/openbmc/openbmc/issues/3315
William A. Kennington III37a89a22018-12-13 14:32:02 -0800491 if not is_valgrind_safe():
William A. Kennington III75130192019-02-07 00:34:14 -0800492 sys.stderr.write("###### Skipping valgrind ######\n")
William A. Kennington III4e1d0a12018-07-16 12:04:03 -0700493 return
Patrick Williamse08ffba2022-12-05 10:33:46 -0600494 if not make_target_exists("check-valgrind"):
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700495 return
496
497 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600498 cmd = make_parallel + ["check-valgrind"]
William A. Kennington III1fddb972019-02-06 18:03:53 -0800499 check_call_cmd(*cmd)
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700500 except CalledProcessError:
William A. Kennington III90b106a2019-02-06 18:08:24 -0800501 for root, _, files in os.walk(os.getcwd()):
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700502 for f in files:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600503 if re.search("test-suite-[a-z]+.log", f) is None:
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700504 continue
Patrick Williamse08ffba2022-12-05 10:33:46 -0600505 check_call_cmd("cat", os.path.join(root, f))
506 raise Exception("Valgrind tests failed")
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700507
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030508
William A. Kennington IIIeaff24a2019-02-06 16:57:42 -0800509def maybe_make_coverage():
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700510 """
511 Potentially runs the unit tests through code coverage for the package
512 via `make check-code-coverage`. If the package does not have code coverage
513 testing then it just skips over this.
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700514 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600515 if not make_target_exists("check-code-coverage"):
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700516 return
517
518 # Actually run code coverage
519 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600520 cmd = make_parallel + ["check-code-coverage"]
William A. Kennington III1fddb972019-02-06 18:03:53 -0800521 check_call_cmd(*cmd)
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700522 except CalledProcessError:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600523 raise Exception("Code coverage failed")
Matthew Barthccb7f852016-11-23 17:43:02 -0600524
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030525
526class BuildSystem(object):
527 """
528 Build systems generally provide the means to configure, build, install and
529 test software. The BuildSystem class defines a set of interfaces on top of
530 which Autotools, Meson, CMake and possibly other build system drivers can
531 be implemented, separating out the phases to control whether a package
532 should merely be installed or also tested and analyzed.
533 """
Lei YU08d2b922022-04-25 11:21:36 +0800534
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030535 def __init__(self, package, path):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600536 """Initialise the driver with properties independent of the build
537 system
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030538
539 Keyword arguments:
540 package: The name of the package. Derived from the path if None
541 path: The path to the package. Set to the working directory if None
542 """
543 self.path = "." if not path else path
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030544 realpath = os.path.realpath(self.path)
545 self.package = package if package else os.path.basename(realpath)
Andrew Jeffery3f8e5a72020-03-13 11:47:56 +1030546 self.build_for_testing = False
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030547
548 def probe(self):
549 """Test if the build system driver can be applied to the package
550
551 Return True if the driver can drive the package's build system,
552 otherwise False.
553
554 Generally probe() is implemented by testing for the presence of the
555 build system's configuration file(s).
556 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600557 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030558
559 def dependencies(self):
560 """Provide the package's dependencies
561
562 Returns a list of dependencies. If no dependencies are required then an
563 empty list must be returned.
564
565 Generally dependencies() is implemented by analysing and extracting the
566 data from the build system configuration.
567 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600568 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030569
570 def configure(self, build_for_testing):
571 """Configure the source ready for building
572
573 Should raise an exception if configuration failed.
574
575 Keyword arguments:
576 build_for_testing: Mark the package as being built for testing rather
577 than for installation as a dependency for the
578 package under test. Setting to True generally
579 implies that the package will be configured to build
580 with debug information, at a low level of
581 optimisation and possibly with sanitizers enabled.
582
583 Generally configure() is implemented by invoking the build system
584 tooling to generate Makefiles or equivalent.
585 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600586 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030587
588 def build(self):
589 """Build the software ready for installation and/or testing
590
591 Should raise an exception if the build fails
592
593 Generally build() is implemented by invoking `make` or `ninja`.
594 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600595 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030596
597 def install(self):
598 """Install the software ready for use
599
600 Should raise an exception if installation fails
601
602 Like build(), install() is generally implemented by invoking `make` or
603 `ninja`.
604 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600605 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030606
607 def test(self):
608 """Build and run the test suite associated with the package
609
610 Should raise an exception if the build or testing fails.
611
612 Like install(), test() is generally implemented by invoking `make` or
613 `ninja`.
614 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600615 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030616
617 def analyze(self):
618 """Run any supported analysis tools over the codebase
619
620 Should raise an exception if analysis fails.
621
622 Some analysis tools such as scan-build need injection into the build
623 system. analyze() provides the necessary hook to implement such
624 behaviour. Analyzers independent of the build system can also be
625 specified here but at the cost of possible duplication of code between
626 the build system driver implementations.
627 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600628 raise NotImplementedError
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030629
630
631class Autotools(BuildSystem):
632 def __init__(self, package=None, path=None):
633 super(Autotools, self).__init__(package, path)
634
635 def probe(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600636 return os.path.isfile(os.path.join(self.path, "configure.ac"))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030637
638 def dependencies(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600639 configure_ac = os.path.join(self.path, "configure.ac")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030640
Patrick Williamse08ffba2022-12-05 10:33:46 -0600641 contents = ""
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030642 # Prepend some special function overrides so we can parse out
643 # dependencies
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030644 for macro in DEPENDENCIES.keys():
Patrick Williamse08ffba2022-12-05 10:33:46 -0600645 contents += (
646 "m4_define(["
647 + macro
648 + "], ["
649 + macro
650 + "_START$"
651 + str(DEPENDENCIES_OFFSET[macro] + 1)
652 + macro
653 + "_END])\n"
654 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030655 with open(configure_ac, "rt") as f:
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030656 contents += f.read()
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030657
Patrick Williamse08ffba2022-12-05 10:33:46 -0600658 autoconf_cmdline = ["autoconf", "-Wno-undefined", "-"]
659 autoconf_process = subprocess.Popen(
660 autoconf_cmdline,
661 stdin=subprocess.PIPE,
662 stdout=subprocess.PIPE,
663 stderr=subprocess.PIPE,
664 )
665 document = contents.encode("utf-8")
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030666 (stdout, stderr) = autoconf_process.communicate(input=document)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030667 if not stdout:
668 print(stderr)
669 raise Exception("Failed to run autoconf for parsing dependencies")
670
671 # Parse out all of the dependency text
672 matches = []
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030673 for macro in DEPENDENCIES.keys():
Patrick Williamse08ffba2022-12-05 10:33:46 -0600674 pattern = "(" + macro + ")_START(.*?)" + macro + "_END"
675 for match in re.compile(pattern).finditer(stdout.decode("utf-8")):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030676 matches.append((match.group(1), match.group(2)))
677
678 # Look up dependencies from the text
679 found_deps = []
680 for macro, deptext in matches:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600681 for potential_dep in deptext.split(" "):
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030682 for known_dep in DEPENDENCIES[macro].keys():
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030683 if potential_dep.startswith(known_dep):
684 found_deps.append(DEPENDENCIES[macro][known_dep])
685
686 return found_deps
687
688 def _configure_feature(self, flag, enabled):
689 """
690 Returns an configure flag as a string
691
692 Parameters:
693 flag The name of the flag
694 enabled Whether the flag is enabled or disabled
695 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600696 return "--" + ("enable" if enabled else "disable") + "-" + flag
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030697
698 def configure(self, build_for_testing):
699 self.build_for_testing = build_for_testing
700 conf_flags = [
Patrick Williamse08ffba2022-12-05 10:33:46 -0600701 self._configure_feature("silent-rules", False),
702 self._configure_feature("examples", build_for_testing),
703 self._configure_feature("tests", build_for_testing),
704 self._configure_feature("itests", INTEGRATION_TEST),
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030705 ]
Patrick Williamse08ffba2022-12-05 10:33:46 -0600706 conf_flags.extend(
707 [
708 self._configure_feature("code-coverage", build_for_testing),
709 self._configure_feature("valgrind", build_for_testing),
710 ]
711 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030712 # Add any necessary configure flags for package
713 if CONFIGURE_FLAGS.get(self.package) is not None:
714 conf_flags.extend(CONFIGURE_FLAGS.get(self.package))
Patrick Williamse08ffba2022-12-05 10:33:46 -0600715 for bootstrap in ["bootstrap.sh", "bootstrap", "autogen.sh"]:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030716 if os.path.exists(bootstrap):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600717 check_call_cmd("./" + bootstrap)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030718 break
Patrick Williamse08ffba2022-12-05 10:33:46 -0600719 check_call_cmd("./configure", *conf_flags)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030720
721 def build(self):
722 check_call_cmd(*make_parallel)
723
724 def install(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600725 check_call_cmd("sudo", "-n", "--", *(make_parallel + ["install"]))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030726
727 def test(self):
728 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600729 cmd = make_parallel + ["check"]
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030730 for i in range(0, args.repeat):
731 check_call_cmd(*cmd)
Andrew Jefferyd0809692021-05-14 16:23:57 +0930732
733 maybe_make_valgrind()
734 maybe_make_coverage()
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030735 except CalledProcessError:
736 for root, _, files in os.walk(os.getcwd()):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600737 if "test-suite.log" not in files:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030738 continue
Patrick Williamse08ffba2022-12-05 10:33:46 -0600739 check_call_cmd("cat", os.path.join(root, "test-suite.log"))
740 raise Exception("Unit tests failed")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030741
742 def analyze(self):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030743 run_cppcheck()
744
745
746class CMake(BuildSystem):
747 def __init__(self, package=None, path=None):
748 super(CMake, self).__init__(package, path)
749
750 def probe(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600751 return os.path.isfile(os.path.join(self.path, "CMakeLists.txt"))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030752
753 def dependencies(self):
754 return []
755
756 def configure(self, build_for_testing):
757 self.build_for_testing = build_for_testing
Ramin Izadpanah298d56b2020-11-17 16:55:28 +0000758 if INTEGRATION_TEST:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600759 check_call_cmd(
760 "cmake",
761 "-DCMAKE_EXPORT_COMPILE_COMMANDS=ON",
762 "-DITESTS=ON",
763 ".",
764 )
Ramin Izadpanah298d56b2020-11-17 16:55:28 +0000765 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600766 check_call_cmd("cmake", "-DCMAKE_EXPORT_COMPILE_COMMANDS=ON", ".")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030767
768 def build(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600769 check_call_cmd(
770 "cmake",
771 "--build",
772 ".",
773 "--",
774 "-j",
775 str(multiprocessing.cpu_count()),
776 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030777
778 def install(self):
Michael Shen87ab5702023-01-13 07:41:11 +0000779 check_call_cmd("sudo", "cmake", "--install", ".")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030780
781 def test(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600782 if make_target_exists("test"):
783 check_call_cmd("ctest", ".")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030784
785 def analyze(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600786 if os.path.isfile(".clang-tidy"):
787 with TemporaryDirectory(prefix="build", dir=".") as build_dir:
Nan Zhou82aaba02022-09-16 00:21:07 +0000788 # clang-tidy needs to run on a clang-specific build
Patrick Williamse08ffba2022-12-05 10:33:46 -0600789 check_call_cmd(
790 "cmake",
791 "-DCMAKE_C_COMPILER=clang",
792 "-DCMAKE_CXX_COMPILER=clang++",
793 "-DCMAKE_EXPORT_COMPILE_COMMANDS=ON",
794 "-H.",
795 "-B" + build_dir,
796 )
Nan Zhou524ebee2022-08-27 23:26:13 +0000797
Patrick Williamse08ffba2022-12-05 10:33:46 -0600798 check_call_cmd(
799 "run-clang-tidy", "-header-filter=.*", "-p", build_dir
800 )
Ed Tanous662890f2020-09-28 17:35:20 -0700801
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030802 maybe_make_valgrind()
803 maybe_make_coverage()
804 run_cppcheck()
805
806
807class Meson(BuildSystem):
Andrew Jefferybf83a1b2024-05-15 22:05:32 +0930808 @staticmethod
809 def _project_name(path):
810 doc = subprocess.check_output(
811 ["meson", "introspect", "--projectinfo", path],
812 stderr=subprocess.STDOUT,
813 ).decode("utf-8")
814 return json.loads(doc)["descriptive_name"]
815
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030816 def __init__(self, package=None, path=None):
817 super(Meson, self).__init__(package, path)
818
819 def probe(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600820 return os.path.isfile(os.path.join(self.path, "meson.build"))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030821
822 def dependencies(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600823 meson_build = os.path.join(self.path, "meson.build")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030824 if not os.path.exists(meson_build):
825 return []
826
827 found_deps = []
828 for root, dirs, files in os.walk(self.path):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600829 if "meson.build" not in files:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030830 continue
Patrick Williamse08ffba2022-12-05 10:33:46 -0600831 with open(os.path.join(root, "meson.build"), "rt") as f:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030832 build_contents = f.read()
Nan Zhouef13d532020-07-07 09:52:02 -0700833 pattern = r"dependency\('([^']*)'.*?\),?\n"
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030834 for match in re.finditer(pattern, build_contents):
835 group = match.group(1)
Patrick Williamse08ffba2022-12-05 10:33:46 -0600836 maybe_dep = DEPENDENCIES["PKG_CHECK_MODULES"].get(group)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030837 if maybe_dep is not None:
838 found_deps.append(maybe_dep)
839
840 return found_deps
841
842 def _parse_options(self, options_file):
843 """
844 Returns a set of options defined in the provides meson_options.txt file
845
846 Parameters:
847 options_file The file containing options
848 """
Patrick Williamse08ffba2022-12-05 10:33:46 -0600849 oi = optinterpreter.OptionInterpreter("")
William A. Kennington III0ecd0bc2020-06-04 00:44:22 -0700850 oi.process(options_file)
851 return oi.options
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030852
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700853 def _configure_boolean(self, val):
854 """
855 Returns the meson flag which signifies the value
856
857 True is true which requires the boolean.
858 False is false which disables the boolean.
859
860 Parameters:
861 val The value being converted
862 """
863 if val is True:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600864 return "true"
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700865 elif val is False:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600866 return "false"
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700867 else:
868 raise Exception("Bad meson boolean value")
869
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030870 def _configure_feature(self, val):
871 """
872 Returns the meson flag which signifies the value
873
874 True is enabled which requires the feature.
875 False is disabled which disables the feature.
876 None is auto which autodetects the feature.
877
878 Parameters:
879 val The value being converted
880 """
881 if val is True:
882 return "enabled"
883 elif val is False:
884 return "disabled"
885 elif val is None:
886 return "auto"
887 else:
888 raise Exception("Bad meson feature value")
889
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700890 def _configure_option(self, opts, key, val):
891 """
892 Returns the meson flag which signifies the value
893 based on the type of the opt
894
895 Parameters:
896 opt The meson option which we are setting
897 val The value being converted
898 """
899 if isinstance(opts[key], coredata.UserBooleanOption):
900 str_val = self._configure_boolean(val)
901 elif isinstance(opts[key], coredata.UserFeatureOption):
902 str_val = self._configure_feature(val)
903 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600904 raise Exception("Unknown meson option type")
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700905 return "-D{}={}".format(key, str_val)
906
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030907 def configure(self, build_for_testing):
908 self.build_for_testing = build_for_testing
William A. Kennington III0ecd0bc2020-06-04 00:44:22 -0700909 meson_options = {}
William A. Kennington III08fa1d52023-11-29 17:01:19 -0800910 if os.path.exists("meson.options"):
911 meson_options = self._parse_options("meson.options")
912 elif os.path.exists("meson_options.txt"):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030913 meson_options = self._parse_options("meson_options.txt")
914 meson_flags = [
Patrick Williamse08ffba2022-12-05 10:33:46 -0600915 "-Db_colorout=never",
916 "-Dwerror=true",
917 "-Dwarning_level=3",
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030918 ]
919 if build_for_testing:
Andrew Jeffery9d43ecf2023-05-25 10:31:08 +0930920 # -Ddebug=true -Doptimization=g is helpful for abi-dumper but isn't a combination that
921 # is supported by meson's build types. Configure it manually.
922 meson_flags.append("-Ddebug=true")
923 meson_flags.append("-Doptimization=g")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030924 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600925 meson_flags.append("--buildtype=debugoptimized")
926 if OptionKey("tests") in meson_options:
927 meson_flags.append(
928 self._configure_option(
929 meson_options, OptionKey("tests"), build_for_testing
930 )
931 )
932 if OptionKey("examples") in meson_options:
933 meson_flags.append(
934 self._configure_option(
935 meson_options, OptionKey("examples"), build_for_testing
936 )
937 )
938 if OptionKey("itests") in meson_options:
939 meson_flags.append(
940 self._configure_option(
941 meson_options, OptionKey("itests"), INTEGRATION_TEST
942 )
943 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030944 if MESON_FLAGS.get(self.package) is not None:
945 meson_flags.extend(MESON_FLAGS.get(self.package))
946 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600947 check_call_cmd(
948 "meson", "setup", "--reconfigure", "build", *meson_flags
949 )
950 except Exception:
951 shutil.rmtree("build", ignore_errors=True)
952 check_call_cmd("meson", "setup", "build", *meson_flags)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030953
Andrew Jefferybf83a1b2024-05-15 22:05:32 +0930954 self.package = Meson._project_name("build")
955
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030956 def build(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600957 check_call_cmd("ninja", "-C", "build")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030958
959 def install(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -0600960 check_call_cmd("sudo", "-n", "--", "ninja", "-C", "build", "install")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030961
962 def test(self):
Patrick Williams95095f12021-04-14 14:42:35 -0500963 # It is useful to check various settings of the meson.build file
964 # for compatibility, such as meson_version checks. We shouldn't
965 # do this in the configure path though because it affects subprojects
966 # and dependencies as well, but we only want this applied to the
967 # project-under-test (otherwise an upstream dependency could fail
968 # this check without our control).
969 self._extra_meson_checks()
970
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030971 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600972 test_args = ("--repeat", str(args.repeat), "-C", "build")
973 check_call_cmd("meson", "test", "--print-errorlogs", *test_args)
Brad Bishop7b8cef22020-08-26 15:58:09 -0400974
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030975 except CalledProcessError:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600976 raise Exception("Unit tests failed")
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030977
978 def _setup_exists(self, setup):
979 """
980 Returns whether the meson build supports the named test setup.
981
982 Parameter descriptions:
983 setup The setup target to check
984 """
985 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -0600986 with open(os.devnull, "w"):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030987 output = subprocess.check_output(
Patrick Williamse08ffba2022-12-05 10:33:46 -0600988 [
989 "meson",
990 "test",
991 "-C",
992 "build",
993 "--setup",
994 setup,
995 "-t",
996 "0",
997 ],
998 stderr=subprocess.STDOUT,
999 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301000 except CalledProcessError as e:
1001 output = e.output
Patrick Williamse08ffba2022-12-05 10:33:46 -06001002 output = output.decode("utf-8")
1003 return not re.search("Test setup .* not found from project", output)
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301004
1005 def _maybe_valgrind(self):
1006 """
1007 Potentially runs the unit tests through valgrind for the package
Andrew Jeffery47fbfa52020-03-13 12:05:09 +10301008 via `meson test`. The package can specify custom valgrind
1009 configurations by utilizing add_test_setup() in a meson.build
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301010 """
1011 if not is_valgrind_safe():
1012 sys.stderr.write("###### Skipping valgrind ######\n")
1013 return
1014 try:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001015 if self._setup_exists("valgrind"):
1016 check_call_cmd(
1017 "meson",
1018 "test",
1019 "-t",
1020 "10",
1021 "-C",
1022 "build",
1023 "--print-errorlogs",
1024 "--setup",
1025 "valgrind",
1026 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301027 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001028 check_call_cmd(
1029 "meson",
1030 "test",
1031 "-t",
1032 "10",
1033 "-C",
1034 "build",
1035 "--print-errorlogs",
1036 "--wrapper",
1037 "valgrind",
1038 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301039 except CalledProcessError:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001040 raise Exception("Valgrind tests failed")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301041
1042 def analyze(self):
1043 self._maybe_valgrind()
1044
1045 # Run clang-tidy only if the project has a configuration
Patrick Williamse08ffba2022-12-05 10:33:46 -06001046 if os.path.isfile(".clang-tidy"):
Manojkiran Eda1aa91992020-10-02 14:11:53 +05301047 os.environ["CXX"] = "clang++"
Patrick Williamse08ffba2022-12-05 10:33:46 -06001048 with TemporaryDirectory(prefix="build", dir=".") as build_dir:
1049 check_call_cmd("meson", "setup", build_dir)
Ed Tanous29e02312022-12-19 16:20:34 -08001050 if not os.path.isfile(".openbmc-no-clang"):
1051 check_call_cmd("meson", "compile", "-C", build_dir)
Nan Zhou82aaba02022-09-16 00:21:07 +00001052 try:
Lei YUac9c9c72023-12-07 07:29:31 +00001053 check_call_cmd("ninja", "-C", build_dir, "clang-tidy-fix")
Nan Zhou82aaba02022-09-16 00:21:07 +00001054 except subprocess.CalledProcessError:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001055 check_call_cmd(
1056 "git", "-C", CODE_SCAN_DIR, "--no-pager", "diff"
1057 )
Nan Zhou82aaba02022-09-16 00:21:07 +00001058 raise
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301059 # Run the basic clang static analyzer otherwise
1060 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001061 check_call_cmd("ninja", "-C", "build", "scan-build")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301062
1063 # Run tests through sanitizers
1064 # b_lundef is needed if clang++ is CXX since it resolves the
1065 # asan symbols at runtime only. We don't want to set it earlier
1066 # in the build process to ensure we don't have undefined
1067 # runtime code.
1068 if is_sanitize_safe():
Patrick Williamse08ffba2022-12-05 10:33:46 -06001069 check_call_cmd(
1070 "meson",
1071 "configure",
1072 "build",
1073 "-Db_sanitize=address,undefined",
1074 "-Db_lundef=false",
1075 )
1076 check_call_cmd(
1077 "meson",
1078 "test",
1079 "-C",
1080 "build",
1081 "--print-errorlogs",
1082 "--logbase",
1083 "testlog-ubasan",
Andrew Jeffery71924fd2023-04-12 23:27:05 +09301084 env=os.environ | {"UBSAN_OPTIONS": "halt_on_error=1"},
Patrick Williamse08ffba2022-12-05 10:33:46 -06001085 )
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301086 # TODO: Fix memory sanitizer
Andrew Jeffery6015ca12020-03-13 11:50:10 +10301087 # check_call_cmd('meson', 'configure', 'build',
1088 # '-Db_sanitize=memory')
1089 # check_call_cmd('meson', 'test', '-C', 'build'
1090 # '--logbase', 'testlog-msan')
Patrick Williamse08ffba2022-12-05 10:33:46 -06001091 check_call_cmd("meson", "configure", "build", "-Db_sanitize=none")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301092 else:
1093 sys.stderr.write("###### Skipping sanitizers ######\n")
1094
1095 # Run coverage checks
Patrick Williamse08ffba2022-12-05 10:33:46 -06001096 check_call_cmd("meson", "configure", "build", "-Db_coverage=true")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301097 self.test()
1098 # Only build coverage HTML if coverage files were produced
Patrick Williamse08ffba2022-12-05 10:33:46 -06001099 for root, dirs, files in os.walk("build"):
1100 if any([f.endswith(".gcda") for f in files]):
1101 check_call_cmd("ninja", "-C", "build", "coverage-html")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301102 break
Patrick Williamse08ffba2022-12-05 10:33:46 -06001103 check_call_cmd("meson", "configure", "build", "-Db_coverage=false")
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301104 run_cppcheck()
1105
Patrick Williams95095f12021-04-14 14:42:35 -05001106 def _extra_meson_checks(self):
Patrick Williamse08ffba2022-12-05 10:33:46 -06001107 with open(os.path.join(self.path, "meson.build"), "rt") as f:
Patrick Williams95095f12021-04-14 14:42:35 -05001108 build_contents = f.read()
1109
1110 # Find project's specified meson_version.
1111 meson_version = None
1112 pattern = r"meson_version:[^']*'([^']*)'"
1113 for match in re.finditer(pattern, build_contents):
1114 group = match.group(1)
1115 meson_version = group
1116
1117 # C++20 requires at least Meson 0.57 but Meson itself doesn't
1118 # identify this. Add to our unit-test checks so that we don't
1119 # get a meson.build missing this.
1120 pattern = r"'cpp_std=c\+\+20'"
1121 for match in re.finditer(pattern, build_contents):
Patrick Williamse08ffba2022-12-05 10:33:46 -06001122 if not meson_version or not meson_version_compare(
1123 meson_version, ">=0.57"
1124 ):
Patrick Williams95095f12021-04-14 14:42:35 -05001125 raise Exception(
1126 "C++20 support requires specifying in meson.build: "
1127 + "meson_version: '>=0.57'"
1128 )
1129
Patrick Williamsb9e07122023-07-12 10:16:28 -05001130 # C++23 requires at least Meson 1.1.1 but Meson itself doesn't
1131 # identify this. Add to our unit-test checks so that we don't
1132 # get a meson.build missing this.
1133 pattern = r"'cpp_std=c\+\+23'"
1134 for match in re.finditer(pattern, build_contents):
1135 if not meson_version or not meson_version_compare(
1136 meson_version, ">=1.1.1"
1137 ):
1138 raise Exception(
1139 "C++23 support requires specifying in meson.build: "
1140 + "meson_version: '>=1.1.1'"
1141 )
1142
Patrick Williamscebaea22023-04-12 10:45:23 -05001143 if "get_variable(" in build_contents:
1144 if not meson_version or not meson_version_compare(
1145 meson_version, ">=0.58"
1146 ):
1147 raise Exception(
1148 "dep.get_variable() with positional argument requires "
1149 + "meson_Version: '>=0.58'"
1150 )
1151
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301152
1153class Package(object):
1154 def __init__(self, name=None, path=None):
Manojkiran Eda29d28252020-06-15 18:58:09 +05301155 self.supported = [Meson, Autotools, CMake]
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301156 self.name = name
1157 self.path = path
1158 self.test_only = False
1159
1160 def build_systems(self):
Andrew Jeffery881041f2020-03-13 11:46:30 +10301161 instances = (system(self.name, self.path) for system in self.supported)
1162 return (instance for instance in instances if instance.probe())
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301163
1164 def build_system(self, preferred=None):
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001165 systems = list(self.build_systems())
1166
1167 if not systems:
1168 return None
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301169
1170 if preferred:
Andrew Jeffery881041f2020-03-13 11:46:30 +10301171 return {type(system): system for system in systems}[preferred]
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301172
1173 return next(iter(systems))
1174
1175 def install(self, system=None):
1176 if not system:
1177 system = self.build_system()
1178
1179 system.configure(False)
1180 system.build()
1181 system.install()
1182
Andrew Jeffery19d75672020-03-13 10:42:08 +10301183 def _test_one(self, system):
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301184 system.configure(True)
1185 system.build()
1186 system.install()
1187 system.test()
Andrew Jefferyd0809692021-05-14 16:23:57 +09301188 if not TEST_ONLY:
1189 system.analyze()
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301190
Andrew Jeffery19d75672020-03-13 10:42:08 +10301191 def test(self):
1192 for system in self.build_systems():
1193 self._test_one(system)
1194
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301195
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001196def find_file(filename, basedir):
1197 """
Patrick Williams55448ad2020-12-14 14:28:28 -06001198 Finds all occurrences of a file (or list of files) in the base
1199 directory and passes them back with their relative paths.
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001200
1201 Parameter descriptions:
Patrick Williams55448ad2020-12-14 14:28:28 -06001202 filename The name of the file (or list of files) to
1203 find
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001204 basedir The base directory search in
1205 """
1206
Patrick Williams55448ad2020-12-14 14:28:28 -06001207 if not isinstance(filename, list):
Lei YU08d2b922022-04-25 11:21:36 +08001208 filename = [filename]
Patrick Williams55448ad2020-12-14 14:28:28 -06001209
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001210 filepaths = []
1211 for root, dirs, files in os.walk(basedir):
Patrick Williamse08ffba2022-12-05 10:33:46 -06001212 if os.path.split(root)[-1] == "subprojects":
Brad Bishopeb667262021-08-06 13:54:58 -04001213 for f in files:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001214 subproject = ".".join(f.split(".")[0:-1])
1215 if f.endswith(".wrap") and subproject in dirs:
Brad Bishopeb667262021-08-06 13:54:58 -04001216 # don't find files in meson subprojects with wraps
1217 dirs.remove(subproject)
Patrick Williams55448ad2020-12-14 14:28:28 -06001218 for f in filename:
1219 if f in files:
1220 filepaths.append(os.path.join(root, f))
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001221 return filepaths
1222
Andrew Jeffery9cc97d82020-03-13 11:57:44 +10301223
Patrick Williamse08ffba2022-12-05 10:33:46 -06001224if __name__ == "__main__":
Matthew Barthccb7f852016-11-23 17:43:02 -06001225 # CONFIGURE_FLAGS = [GIT REPO]:[CONFIGURE FLAGS]
1226 CONFIGURE_FLAGS = {
Patrick Williamse08ffba2022-12-05 10:33:46 -06001227 "phosphor-logging": [
1228 "--enable-metadata-processing",
1229 "--enable-openpower-pel-extension",
1230 "YAML_DIR=/usr/local/share/phosphor-dbus-yaml/yaml",
1231 ]
Matthew Barthccb7f852016-11-23 17:43:02 -06001232 }
1233
William A. Kennington III3f1d1202018-12-06 18:02:07 -08001234 # MESON_FLAGS = [GIT REPO]:[MESON FLAGS]
1235 MESON_FLAGS = {
Patrick Williamse08ffba2022-12-05 10:33:46 -06001236 "phosphor-dbus-interfaces": [
1237 "-Ddata_com_ibm=true",
1238 "-Ddata_org_open_power=true",
1239 ],
1240 "phosphor-logging": ["-Dopenpower-pel-extension=enabled"],
William A. Kennington III3f1d1202018-12-06 18:02:07 -08001241 }
1242
Matthew Barthccb7f852016-11-23 17:43:02 -06001243 # DEPENDENCIES = [MACRO]:[library/header]:[GIT REPO]
1244 DEPENDENCIES = {
Patrick Williamse08ffba2022-12-05 10:33:46 -06001245 "AC_CHECK_LIB": {"mapper": "phosphor-objmgr"},
1246 "AC_CHECK_HEADER": {
1247 "host-ipmid": "phosphor-host-ipmid",
1248 "blobs-ipmid": "phosphor-ipmi-blobs",
1249 "sdbusplus": "sdbusplus",
1250 "sdeventplus": "sdeventplus",
1251 "stdplus": "stdplus",
1252 "gpioplus": "gpioplus",
1253 "phosphor-logging/log.hpp": "phosphor-logging",
Patrick Williamseab8a372017-01-30 11:21:32 -06001254 },
Patrick Williamse08ffba2022-12-05 10:33:46 -06001255 "AC_PATH_PROG": {"sdbus++": "sdbusplus"},
1256 "PKG_CHECK_MODULES": {
1257 "phosphor-dbus-interfaces": "phosphor-dbus-interfaces",
1258 "libipmid": "phosphor-host-ipmid",
1259 "libipmid-host": "phosphor-host-ipmid",
1260 "sdbusplus": "sdbusplus",
1261 "sdeventplus": "sdeventplus",
1262 "stdplus": "stdplus",
1263 "gpioplus": "gpioplus",
1264 "phosphor-logging": "phosphor-logging",
1265 "phosphor-snmp": "phosphor-snmp",
1266 "ipmiblob": "ipmi-blob-tool",
1267 "hei": "openpower-libhei",
1268 "phosphor-ipmi-blobs": "phosphor-ipmi-blobs",
1269 "libcr51sign": "google-misc",
Brad Bishopebb49112017-02-13 11:07:26 -05001270 },
Matthew Barthccb7f852016-11-23 17:43:02 -06001271 }
1272
William A. Kennington IIIe67f5fc2018-12-06 17:40:30 -08001273 # Offset into array of macro parameters MACRO(0, 1, ...N)
1274 DEPENDENCIES_OFFSET = {
Patrick Williamse08ffba2022-12-05 10:33:46 -06001275 "AC_CHECK_LIB": 0,
1276 "AC_CHECK_HEADER": 0,
1277 "AC_PATH_PROG": 1,
1278 "PKG_CHECK_MODULES": 1,
William A. Kennington IIIe67f5fc2018-12-06 17:40:30 -08001279 }
1280
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -05001281 # DEPENDENCIES_REGEX = [GIT REPO]:[REGEX STRING]
Patrick Williamse08ffba2022-12-05 10:33:46 -06001282 DEPENDENCIES_REGEX = {"phosphor-logging": r"\S+-dbus-interfaces$"}
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -05001283
Matthew Barth33df8792016-12-19 14:30:17 -06001284 # Set command line arguments
1285 parser = argparse.ArgumentParser()
Patrick Williamse08ffba2022-12-05 10:33:46 -06001286 parser.add_argument(
1287 "-w",
1288 "--workspace",
1289 dest="WORKSPACE",
1290 required=True,
1291 help="Workspace directory location(i.e. /home)",
1292 )
1293 parser.add_argument(
1294 "-p",
1295 "--package",
1296 dest="PACKAGE",
1297 required=True,
1298 help="OpenBMC package to be unit tested",
1299 )
1300 parser.add_argument(
1301 "-t",
1302 "--test-only",
1303 dest="TEST_ONLY",
1304 action="store_true",
1305 required=False,
1306 default=False,
1307 help="Only run test cases, no other validation",
1308 )
Ramin Izadpanah298d56b2020-11-17 16:55:28 +00001309 arg_inttests = parser.add_mutually_exclusive_group()
Patrick Williamse08ffba2022-12-05 10:33:46 -06001310 arg_inttests.add_argument(
1311 "--integration-tests",
1312 dest="INTEGRATION_TEST",
1313 action="store_true",
1314 required=False,
1315 default=True,
1316 help="Enable integration tests [default].",
1317 )
1318 arg_inttests.add_argument(
1319 "--no-integration-tests",
1320 dest="INTEGRATION_TEST",
1321 action="store_false",
1322 required=False,
1323 help="Disable integration tests.",
1324 )
1325 parser.add_argument(
1326 "-v",
1327 "--verbose",
1328 action="store_true",
1329 help="Print additional package status messages",
1330 )
1331 parser.add_argument(
1332 "-r", "--repeat", help="Repeat tests N times", type=int, default=1
1333 )
1334 parser.add_argument(
1335 "-b",
1336 "--branch",
1337 dest="BRANCH",
1338 required=False,
1339 help="Branch to target for dependent repositories",
1340 default="master",
1341 )
1342 parser.add_argument(
1343 "-n",
1344 "--noformat",
1345 dest="FORMAT",
1346 action="store_false",
1347 required=False,
1348 help="Whether or not to run format code",
1349 )
Matthew Barth33df8792016-12-19 14:30:17 -06001350 args = parser.parse_args(sys.argv[1:])
1351 WORKSPACE = args.WORKSPACE
1352 UNIT_TEST_PKG = args.PACKAGE
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001353 TEST_ONLY = args.TEST_ONLY
Ramin Izadpanah298d56b2020-11-17 16:55:28 +00001354 INTEGRATION_TEST = args.INTEGRATION_TEST
Andrew Geisslera61acb52019-01-03 16:32:44 -06001355 BRANCH = args.BRANCH
Lei YU7ef93302019-11-06 13:53:21 +08001356 FORMAT_CODE = args.FORMAT
Matthew Barth33df8792016-12-19 14:30:17 -06001357 if args.verbose:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001358
Matthew Barth33df8792016-12-19 14:30:17 -06001359 def printline(*line):
1360 for arg in line:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001361 print(arg, end=" ")
Andrew Jeffery89b64b62020-03-13 12:15:48 +10301362 print()
Patrick Williamse08ffba2022-12-05 10:33:46 -06001363
Matthew Barth33df8792016-12-19 14:30:17 -06001364 else:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001365
Andrew Jefferyfce557f2020-03-13 12:12:19 +10301366 def printline(*line):
1367 pass
Matthew Barthccb7f852016-11-23 17:43:02 -06001368
Patrick Williamsb6535952020-12-15 06:40:10 -06001369 CODE_SCAN_DIR = os.path.join(WORKSPACE, UNIT_TEST_PKG)
Lei YU7ef93302019-11-06 13:53:21 +08001370
Patrick Williams330b0772022-11-28 06:14:06 -06001371 # Run format-code.sh, which will in turn call any repo-level formatters.
Lei YU7ef93302019-11-06 13:53:21 +08001372 if FORMAT_CODE:
Patrick Williamse08ffba2022-12-05 10:33:46 -06001373 check_call_cmd(
1374 os.path.join(
1375 WORKSPACE, "openbmc-build-scripts", "scripts", "format-code.sh"
1376 ),
1377 CODE_SCAN_DIR,
1378 )
Andrew Geisslera28286d2018-01-10 11:00:00 -08001379
Ed Tanous32768b82022-01-05 14:14:06 -08001380 # Check to see if any files changed
Patrick Williamse08ffba2022-12-05 10:33:46 -06001381 check_call_cmd(
1382 "git", "-C", CODE_SCAN_DIR, "--no-pager", "diff", "--exit-code"
1383 )
Ed Tanous32768b82022-01-05 14:14:06 -08001384
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001385 # Check if this repo has a supported make infrastructure
Patrick Williamsb6535952020-12-15 06:40:10 -06001386 pkg = Package(UNIT_TEST_PKG, CODE_SCAN_DIR)
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001387 if not pkg.build_system():
1388 print("No valid build system, exit")
1389 sys.exit(0)
1390
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301391 prev_umask = os.umask(000)
James Feist878df5c2018-07-26 14:54:28 -07001392
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301393 # Determine dependencies and add them
1394 dep_added = dict()
1395 dep_added[UNIT_TEST_PKG] = False
William A. Kennington III40d5c7c2018-12-13 14:37:59 -08001396
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301397 # Create dependency tree
1398 dep_tree = DepTree(UNIT_TEST_PKG)
Patrick Williamsb6535952020-12-15 06:40:10 -06001399 build_dep_tree(UNIT_TEST_PKG, CODE_SCAN_DIR, dep_added, dep_tree, BRANCH)
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001400
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301401 # Reorder Dependency Tree
Andrew Jeffery89b64b62020-03-13 12:15:48 +10301402 for pkg_name, regex_str in DEPENDENCIES_REGEX.items():
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301403 dep_tree.ReorderDeps(pkg_name, regex_str)
1404 if args.verbose:
1405 dep_tree.PrintTree()
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001406
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301407 install_list = dep_tree.GetInstallList()
Andrew Geissler9ced4ed2019-11-18 14:33:58 -06001408
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301409 # We don't want to treat our package as a dependency
1410 install_list.remove(UNIT_TEST_PKG)
James Feist878df5c2018-07-26 14:54:28 -07001411
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301412 # Install reordered dependencies
1413 for dep in install_list:
1414 build_and_install(dep, False)
James Feist878df5c2018-07-26 14:54:28 -07001415
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301416 # Run package unit tests
1417 build_and_install(UNIT_TEST_PKG, True)
James Feist878df5c2018-07-26 14:54:28 -07001418
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301419 os.umask(prev_umask)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001420
1421 # Run any custom CI scripts the repo has, of which there can be
1422 # multiple of and anywhere in the repository.
Patrick Williamse08ffba2022-12-05 10:33:46 -06001423 ci_scripts = find_file(["run-ci.sh", "run-ci"], CODE_SCAN_DIR)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001424 if ci_scripts:
Patrick Williamsb6535952020-12-15 06:40:10 -06001425 os.chdir(CODE_SCAN_DIR)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001426 for ci_script in ci_scripts:
Patrick Williams55448ad2020-12-14 14:28:28 -06001427 check_call_cmd(ci_script)