blob: c44d72bcf56977e441b056ea594e24c10f658a2a [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
Matthew Barthd1810372016-12-19 16:57:21 -060010from git import Repo
William A. Kennington III3992d102021-05-17 01:41:04 -070011# interpreter is not used directly but this resolves dependency ordering
12# that would be broken if we didn't include it.
13from mesonbuild import interpreter
William A. Kennington IIIfcd70772020-06-04 00:50:23 -070014from mesonbuild import coredata, optinterpreter
Patrick Williamse95626d2021-04-13 11:13:53 -050015from mesonbuild.mesonlib import OptionKey
Patrick Williams95095f12021-04-14 14:42:35 -050016from mesonbuild.mesonlib import version_compare as meson_version_compare
Andrew Jeffery89b64b62020-03-13 12:15:48 +103017from urllib.parse import urljoin
Andrew Jefferya4e31c62018-03-08 13:45:28 +103018from subprocess import check_call, call, CalledProcessError
Matthew Barthccb7f852016-11-23 17:43:02 -060019import os
20import sys
Matthew Barth33df8792016-12-19 14:30:17 -060021import argparse
William A. Kennington IIIa2156732018-06-30 18:38:09 -070022import multiprocessing
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -050023import re
William A. Kennington IIIe67f5fc2018-12-06 17:40:30 -080024import subprocess
William A. Kennington III3f1d1202018-12-06 18:02:07 -080025import shutil
William A. Kennington III4e1d0a12018-07-16 12:04:03 -070026import platform
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -050027
28
29class DepTree():
30 """
31 Represents package dependency tree, where each node is a DepTree with a
32 name and DepTree children.
33 """
34
35 def __init__(self, name):
36 """
37 Create new DepTree.
38
39 Parameter descriptions:
40 name Name of new tree node.
41 """
42 self.name = name
43 self.children = list()
44
45 def AddChild(self, name):
46 """
47 Add new child node to current node.
48
49 Parameter descriptions:
50 name Name of new child
51 """
52 new_child = DepTree(name)
53 self.children.append(new_child)
54 return new_child
55
56 def AddChildNode(self, node):
57 """
58 Add existing child node to current node.
59
60 Parameter descriptions:
61 node Tree node to add
62 """
63 self.children.append(node)
64
65 def RemoveChild(self, name):
66 """
67 Remove child node.
68
69 Parameter descriptions:
70 name Name of child to remove
71 """
72 for child in self.children:
73 if child.name == name:
74 self.children.remove(child)
75 return
76
77 def GetNode(self, name):
78 """
79 Return node with matching name. Return None if not found.
80
81 Parameter descriptions:
82 name Name of node to return
83 """
84 if self.name == name:
85 return self
86 for child in self.children:
87 node = child.GetNode(name)
88 if node:
89 return node
90 return None
91
92 def GetParentNode(self, name, parent_node=None):
93 """
94 Return parent of node with matching name. Return none if not found.
95
96 Parameter descriptions:
97 name Name of node to get parent of
98 parent_node Parent of current node
99 """
100 if self.name == name:
101 return parent_node
102 for child in self.children:
103 found_node = child.GetParentNode(name, self)
104 if found_node:
105 return found_node
106 return None
107
108 def GetPath(self, name, path=None):
109 """
110 Return list of node names from head to matching name.
111 Return None if not found.
112
113 Parameter descriptions:
114 name Name of node
115 path List of node names from head to current node
116 """
117 if not path:
118 path = []
119 if self.name == name:
120 path.append(self.name)
121 return path
122 for child in self.children:
123 match = child.GetPath(name, path + [self.name])
124 if match:
125 return match
126 return None
127
128 def GetPathRegex(self, name, regex_str, path=None):
129 """
130 Return list of node paths that end in name, or match regex_str.
131 Return empty list if not found.
132
133 Parameter descriptions:
134 name Name of node to search for
135 regex_str Regex string to match node names
136 path Path of node names from head to current node
137 """
138 new_paths = []
139 if not path:
140 path = []
141 match = re.match(regex_str, self.name)
142 if (self.name == name) or (match):
143 new_paths.append(path + [self.name])
144 for child in self.children:
145 return_paths = None
146 full_path = path + [self.name]
147 return_paths = child.GetPathRegex(name, regex_str, full_path)
148 for i in return_paths:
149 new_paths.append(i)
150 return new_paths
151
152 def MoveNode(self, from_name, to_name):
153 """
154 Mode existing from_name node to become child of to_name node.
155
156 Parameter descriptions:
157 from_name Name of node to make a child of to_name
158 to_name Name of node to make parent of from_name
159 """
160 parent_from_node = self.GetParentNode(from_name)
161 from_node = self.GetNode(from_name)
162 parent_from_node.RemoveChild(from_name)
163 to_node = self.GetNode(to_name)
164 to_node.AddChildNode(from_node)
165
166 def ReorderDeps(self, name, regex_str):
167 """
168 Reorder dependency tree. If tree contains nodes with names that
169 match 'name' and 'regex_str', move 'regex_str' nodes that are
170 to the right of 'name' node, so that they become children of the
171 'name' node.
172
173 Parameter descriptions:
174 name Name of node to look for
175 regex_str Regex string to match names to
176 """
177 name_path = self.GetPath(name)
178 if not name_path:
179 return
180 paths = self.GetPathRegex(name, regex_str)
181 is_name_in_paths = False
182 name_index = 0
183 for i in range(len(paths)):
184 path = paths[i]
185 if path[-1] == name:
186 is_name_in_paths = True
187 name_index = i
188 break
189 if not is_name_in_paths:
190 return
191 for i in range(name_index + 1, len(paths)):
192 path = paths[i]
193 if name in path:
194 continue
195 from_name = path[-1]
196 self.MoveNode(from_name, name)
197
198 def GetInstallList(self):
199 """
200 Return post-order list of node names.
201
202 Parameter descriptions:
203 """
204 install_list = []
205 for child in self.children:
206 child_install_list = child.GetInstallList()
207 install_list.extend(child_install_list)
208 install_list.append(self.name)
209 return install_list
210
211 def PrintTree(self, level=0):
212 """
213 Print pre-order node names with indentation denoting node depth level.
214
215 Parameter descriptions:
216 level Current depth level
217 """
218 INDENT_PER_LEVEL = 4
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030219 print(' ' * (level * INDENT_PER_LEVEL) + self.name)
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500220 for child in self.children:
221 child.PrintTree(level + 1)
Matthew Barth33df8792016-12-19 14:30:17 -0600222
223
William A. Kennington III1fddb972019-02-06 18:03:53 -0800224def check_call_cmd(*cmd):
Matthew Barth33df8792016-12-19 14:30:17 -0600225 """
226 Verbose prints the directory location the given command is called from and
227 the command, then executes the command using check_call.
228
229 Parameter descriptions:
230 dir Directory location command is to be called from
231 cmd List of parameters constructing the complete command
232 """
William A. Kennington III1fddb972019-02-06 18:03:53 -0800233 printline(os.getcwd(), ">", " ".join(cmd))
Matthew Barth33df8792016-12-19 14:30:17 -0600234 check_call(cmd)
Matthew Barthccb7f852016-11-23 17:43:02 -0600235
236
Andrew Geisslera61acb52019-01-03 16:32:44 -0600237def clone_pkg(pkg, branch):
Matthew Barth33df8792016-12-19 14:30:17 -0600238 """
239 Clone the given openbmc package's git repository from gerrit into
240 the WORKSPACE location
241
242 Parameter descriptions:
243 pkg Name of the package to clone
Andrew Geisslera61acb52019-01-03 16:32:44 -0600244 branch Branch to clone from pkg
Matthew Barth33df8792016-12-19 14:30:17 -0600245 """
Andrew Jeffery7be94ca2018-03-08 13:15:33 +1030246 pkg_dir = os.path.join(WORKSPACE, pkg)
247 if os.path.exists(os.path.join(pkg_dir, '.git')):
248 return pkg_dir
Matthew Barthccb7f852016-11-23 17:43:02 -0600249 pkg_repo = urljoin('https://gerrit.openbmc-project.xyz/openbmc/', pkg)
Andrew Jeffery7be94ca2018-03-08 13:15:33 +1030250 os.mkdir(pkg_dir)
Andrew Geisslera61acb52019-01-03 16:32:44 -0600251 printline(pkg_dir, "> git clone", pkg_repo, branch, "./")
252 try:
253 # first try the branch
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030254 clone = Repo.clone_from(pkg_repo, pkg_dir, branch=branch)
255 repo_inst = clone.working_dir
Andrew Geisslera61acb52019-01-03 16:32:44 -0600256 except:
257 printline("Input branch not found, default to master")
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030258 clone = Repo.clone_from(pkg_repo, pkg_dir, branch="master")
259 repo_inst = clone.working_dir
Andrew Geisslera61acb52019-01-03 16:32:44 -0600260 return repo_inst
Matthew Barth33df8792016-12-19 14:30:17 -0600261
262
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030263def make_target_exists(target):
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800264 """
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030265 Runs a check against the makefile in the current directory to determine
266 if the target exists so that it can be built.
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800267
268 Parameter descriptions:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030269 target The make target we are checking
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800270 """
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030271 try:
Andrew Jeffery881041f2020-03-13 11:46:30 +1030272 cmd = ['make', '-n', target]
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030273 with open(os.devnull, 'w') as devnull:
274 check_call(cmd, stdout=devnull, stderr=devnull)
275 return True
276 except CalledProcessError:
277 return False
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800278
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800279
William A. Kennington IIIa2156732018-06-30 18:38:09 -0700280make_parallel = [
281 'make',
282 # Run enough jobs to saturate all the cpus
283 '-j', str(multiprocessing.cpu_count()),
284 # Don't start more jobs if the load avg is too high
285 '-l', str(multiprocessing.cpu_count()),
286 # Synchronize the output so logs aren't intermixed in stdout / stderr
287 '-O',
288]
289
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800290
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030291def build_and_install(name, build_for_testing=False):
William A. Kennington III780ec092018-12-06 14:46:50 -0800292 """
293 Builds and installs the package in the environment. Optionally
294 builds the examples and test cases for package.
295
296 Parameter description:
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030297 name The name of the package we are building
William A. Kennington IIIa0454912018-12-06 14:47:16 -0800298 build_for_testing Enable options related to testing on the package?
William A. Kennington III780ec092018-12-06 14:46:50 -0800299 """
Andrew Jefferyff5c5d52020-03-13 10:12:14 +1030300 os.chdir(os.path.join(WORKSPACE, name))
William A. Kennington III54d4faf2018-12-06 17:46:24 -0800301
302 # Refresh dynamic linker run time bindings for dependencies
William A. Kennington III1fddb972019-02-06 18:03:53 -0800303 check_call_cmd('sudo', '-n', '--', 'ldconfig')
William A. Kennington III54d4faf2018-12-06 17:46:24 -0800304
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030305 pkg = Package()
306 if build_for_testing:
307 pkg.test()
William A. Kennington III3f1d1202018-12-06 18:02:07 -0800308 else:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030309 pkg.install()
310
William A. Kennington III780ec092018-12-06 14:46:50 -0800311
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030312def build_dep_tree(name, pkgdir, dep_added, head, branch, dep_tree=None):
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500313 """
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030314 For each package (name), starting with the package to be unit tested,
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030315 extract its dependencies. For each package dependency defined, recursively
316 apply the same strategy
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500317
318 Parameter descriptions:
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030319 name Name of the package
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500320 pkgdir Directory where package source is located
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800321 dep_added Current dict of dependencies and added status
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500322 head Head node of the dependency tree
Andrew Geisslera61acb52019-01-03 16:32:44 -0600323 branch Branch to clone from pkg
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500324 dep_tree Current dependency tree node
325 """
326 if not dep_tree:
327 dep_tree = head
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800328
William A. Kennington IIIbe6aab22018-12-06 15:01:54 -0800329 with open("/tmp/depcache", "r") as depcache:
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800330 cache = depcache.readline()
331
332 # Read out pkg dependencies
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030333 pkg = Package(name, pkgdir)
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800334
Patrick Williams38c6b7d2021-02-18 07:26:14 -0600335 build = pkg.build_system()
336 if build == None:
337 raise Exception(f"Unable to find build system for {name}.")
338
339 for dep in set(build.dependencies()):
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800340 if dep in cache:
341 continue
342 # Dependency package not already known
343 if dep_added.get(dep) is None:
Patrick Williams38c6b7d2021-02-18 07:26:14 -0600344 print(f"Adding {dep} dependency to {name}.")
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800345 # Dependency package not added
346 new_child = dep_tree.AddChild(dep)
347 dep_added[dep] = False
Andrew Jeffery3b92fdd2020-03-13 11:49:18 +1030348 dep_pkgdir = clone_pkg(dep, branch)
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800349 # Determine this dependency package's
350 # dependencies and add them before
351 # returning to add this package
352 dep_added = build_dep_tree(dep,
353 dep_pkgdir,
354 dep_added,
355 head,
Andrew Geisslera61acb52019-01-03 16:32:44 -0600356 branch,
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800357 new_child)
358 else:
359 # Dependency package known and added
360 if dep_added[dep]:
Andrew Jeffery2cb0c7a2018-03-08 13:19:08 +1030361 continue
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500362 else:
William A. Kennington IIIc048cc02018-12-06 15:39:18 -0800363 # Cyclic dependency failure
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030364 raise Exception("Cyclic dependencies found in "+name)
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500365
Andrew Jefferyccf85d62020-03-13 10:25:42 +1030366 if not dep_added[name]:
367 dep_added[name] = True
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -0500368
369 return dep_added
Matthew Barthccb7f852016-11-23 17:43:02 -0600370
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700371
William A. Kennington III90b106a2019-02-06 18:08:24 -0800372def run_cppcheck():
Andrew Jefferybf735102020-03-13 12:13:21 +1030373 match_re = re.compile(r'((?!\.mako\.).)*\.[ch](?:pp)?$', re.I)
Brad Bishop48424d42020-01-07 13:01:31 -0500374 cppcheck_files = []
375 stdout = subprocess.check_output(['git', 'ls-files'])
Patrick Venturead4354e2018-10-12 16:59:54 -0700376
Brad Bishop48424d42020-01-07 13:01:31 -0500377 for f in stdout.decode('utf-8').split():
378 if match_re.match(f):
379 cppcheck_files.append(f)
380
381 if not cppcheck_files:
382 # skip cppcheck if there arent' any c or cpp sources.
383 print("no files")
384 return None
385
386 # http://cppcheck.sourceforge.net/manual.pdf
387 params = ['cppcheck', '-j', str(multiprocessing.cpu_count()),
Manojkiran Eda89a46cc2020-08-22 17:08:11 +0530388 '--enable=all', '--library=googletest', '--file-list=-']
Brad Bishop48424d42020-01-07 13:01:31 -0500389
390 cppcheck_process = subprocess.Popen(
391 params,
392 stdout=subprocess.PIPE,
393 stderr=subprocess.PIPE,
394 stdin=subprocess.PIPE)
395 (stdout, stderr) = cppcheck_process.communicate(
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030396 input='\n'.join(cppcheck_files).encode('utf-8'))
Brad Bishop48424d42020-01-07 13:01:31 -0500397
398 if cppcheck_process.wait():
Patrick Venturead4354e2018-10-12 16:59:54 -0700399 raise Exception('Cppcheck failed')
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030400 print(stdout.decode('utf-8'))
401 print(stderr.decode('utf-8'))
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700402
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030403
William A. Kennington III37a89a22018-12-13 14:32:02 -0800404def is_valgrind_safe():
405 """
406 Returns whether it is safe to run valgrind on our platform
407 """
William A. Kennington III0326ded2019-02-07 00:33:28 -0800408 src = 'unit-test-vg.c'
409 exe = './unit-test-vg'
410 with open(src, 'w') as h:
William A. Kennington IIIafb0f982019-04-26 17:30:28 -0700411 h.write('#include <errno.h>\n')
412 h.write('#include <stdio.h>\n')
William A. Kennington III0326ded2019-02-07 00:33:28 -0800413 h.write('#include <stdlib.h>\n')
414 h.write('#include <string.h>\n')
415 h.write('int main() {\n')
416 h.write('char *heap_str = malloc(16);\n')
417 h.write('strcpy(heap_str, "RandString");\n')
418 h.write('int res = strcmp("RandString", heap_str);\n')
419 h.write('free(heap_str);\n')
William A. Kennington IIIafb0f982019-04-26 17:30:28 -0700420 h.write('char errstr[64];\n')
421 h.write('strerror_r(EINVAL, errstr, sizeof(errstr));\n')
422 h.write('printf("%s\\n", errstr);\n')
William A. Kennington III0326ded2019-02-07 00:33:28 -0800423 h.write('return res;\n')
424 h.write('}\n')
425 try:
426 with open(os.devnull, 'w') as devnull:
427 check_call(['gcc', '-O2', '-o', exe, src],
428 stdout=devnull, stderr=devnull)
429 check_call(['valgrind', '--error-exitcode=99', exe],
430 stdout=devnull, stderr=devnull)
431 return True
432 except:
433 sys.stderr.write("###### Platform is not valgrind safe ######\n")
434 return False
435 finally:
436 os.remove(src)
437 os.remove(exe)
William A. Kennington III37a89a22018-12-13 14:32:02 -0800438
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030439
William A. Kennington III282e3302019-02-04 16:55:05 -0800440def is_sanitize_safe():
441 """
442 Returns whether it is safe to run sanitizers on our platform
443 """
William A. Kennington III0b7fb2b2019-02-07 00:33:42 -0800444 src = 'unit-test-sanitize.c'
445 exe = './unit-test-sanitize'
446 with open(src, 'w') as h:
447 h.write('int main() { return 0; }\n')
448 try:
449 with open(os.devnull, 'w') as devnull:
450 check_call(['gcc', '-O2', '-fsanitize=address',
451 '-fsanitize=undefined', '-o', exe, src],
452 stdout=devnull, stderr=devnull)
453 check_call([exe], stdout=devnull, stderr=devnull)
Andrew Geisslercd9578b2021-05-04 10:18:01 -0500454
455 # TODO - Sanitizer not working on ppc64le
456 # https://github.com/openbmc/openbmc-build-scripts/issues/31
457 if (platform.processor() == 'ppc64le'):
458 sys.stderr.write("###### ppc64le is not sanitize safe ######\n")
459 return False
460 else:
461 return True
William A. Kennington III0b7fb2b2019-02-07 00:33:42 -0800462 except:
463 sys.stderr.write("###### Platform is not sanitize safe ######\n")
464 return False
465 finally:
466 os.remove(src)
467 os.remove(exe)
William A. Kennington III282e3302019-02-04 16:55:05 -0800468
William A. Kennington III49d4e592019-02-06 17:59:27 -0800469
William A. Kennington IIIeaff24a2019-02-06 16:57:42 -0800470def maybe_make_valgrind():
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700471 """
472 Potentially runs the unit tests through valgrind for the package
473 via `make check-valgrind`. If the package does not have valgrind testing
474 then it just skips over this.
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700475 """
William A. Kennington III4e1d0a12018-07-16 12:04:03 -0700476 # Valgrind testing is currently broken by an aggressive strcmp optimization
477 # that is inlined into optimized code for POWER by gcc 7+. Until we find
478 # a workaround, just don't run valgrind tests on POWER.
479 # https://github.com/openbmc/openbmc/issues/3315
William A. Kennington III37a89a22018-12-13 14:32:02 -0800480 if not is_valgrind_safe():
William A. Kennington III75130192019-02-07 00:34:14 -0800481 sys.stderr.write("###### Skipping valgrind ######\n")
William A. Kennington III4e1d0a12018-07-16 12:04:03 -0700482 return
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700483 if not make_target_exists('check-valgrind'):
484 return
485
486 try:
Andrew Jeffery881041f2020-03-13 11:46:30 +1030487 cmd = make_parallel + ['check-valgrind']
William A. Kennington III1fddb972019-02-06 18:03:53 -0800488 check_call_cmd(*cmd)
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700489 except CalledProcessError:
William A. Kennington III90b106a2019-02-06 18:08:24 -0800490 for root, _, files in os.walk(os.getcwd()):
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700491 for f in files:
492 if re.search('test-suite-[a-z]+.log', f) is None:
493 continue
William A. Kennington III1fddb972019-02-06 18:03:53 -0800494 check_call_cmd('cat', os.path.join(root, f))
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700495 raise Exception('Valgrind tests failed')
496
Andrew Jefferye5fffa02020-03-13 11:51:57 +1030497
William A. Kennington IIIeaff24a2019-02-06 16:57:42 -0800498def maybe_make_coverage():
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700499 """
500 Potentially runs the unit tests through code coverage for the package
501 via `make check-code-coverage`. If the package does not have code coverage
502 testing then it just skips over this.
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700503 """
504 if not make_target_exists('check-code-coverage'):
505 return
506
507 # Actually run code coverage
508 try:
Andrew Jeffery881041f2020-03-13 11:46:30 +1030509 cmd = make_parallel + ['check-code-coverage']
William A. Kennington III1fddb972019-02-06 18:03:53 -0800510 check_call_cmd(*cmd)
William A. Kennington III0f0a6802018-07-16 11:52:33 -0700511 except CalledProcessError:
512 raise Exception('Code coverage failed')
Matthew Barthccb7f852016-11-23 17:43:02 -0600513
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030514
515class BuildSystem(object):
516 """
517 Build systems generally provide the means to configure, build, install and
518 test software. The BuildSystem class defines a set of interfaces on top of
519 which Autotools, Meson, CMake and possibly other build system drivers can
520 be implemented, separating out the phases to control whether a package
521 should merely be installed or also tested and analyzed.
522 """
Lei YU08d2b922022-04-25 11:21:36 +0800523
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030524 def __init__(self, package, path):
525 """Initialise the driver with properties independent of the build system
526
527 Keyword arguments:
528 package: The name of the package. Derived from the path if None
529 path: The path to the package. Set to the working directory if None
530 """
531 self.path = "." if not path else path
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030532 realpath = os.path.realpath(self.path)
533 self.package = package if package else os.path.basename(realpath)
Andrew Jeffery3f8e5a72020-03-13 11:47:56 +1030534 self.build_for_testing = False
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030535
536 def probe(self):
537 """Test if the build system driver can be applied to the package
538
539 Return True if the driver can drive the package's build system,
540 otherwise False.
541
542 Generally probe() is implemented by testing for the presence of the
543 build system's configuration file(s).
544 """
545 raise NotImplemented
546
547 def dependencies(self):
548 """Provide the package's dependencies
549
550 Returns a list of dependencies. If no dependencies are required then an
551 empty list must be returned.
552
553 Generally dependencies() is implemented by analysing and extracting the
554 data from the build system configuration.
555 """
556 raise NotImplemented
557
558 def configure(self, build_for_testing):
559 """Configure the source ready for building
560
561 Should raise an exception if configuration failed.
562
563 Keyword arguments:
564 build_for_testing: Mark the package as being built for testing rather
565 than for installation as a dependency for the
566 package under test. Setting to True generally
567 implies that the package will be configured to build
568 with debug information, at a low level of
569 optimisation and possibly with sanitizers enabled.
570
571 Generally configure() is implemented by invoking the build system
572 tooling to generate Makefiles or equivalent.
573 """
574 raise NotImplemented
575
576 def build(self):
577 """Build the software ready for installation and/or testing
578
579 Should raise an exception if the build fails
580
581 Generally build() is implemented by invoking `make` or `ninja`.
582 """
583 raise NotImplemented
584
585 def install(self):
586 """Install the software ready for use
587
588 Should raise an exception if installation fails
589
590 Like build(), install() is generally implemented by invoking `make` or
591 `ninja`.
592 """
593 raise NotImplemented
594
595 def test(self):
596 """Build and run the test suite associated with the package
597
598 Should raise an exception if the build or testing fails.
599
600 Like install(), test() is generally implemented by invoking `make` or
601 `ninja`.
602 """
603 raise NotImplemented
604
605 def analyze(self):
606 """Run any supported analysis tools over the codebase
607
608 Should raise an exception if analysis fails.
609
610 Some analysis tools such as scan-build need injection into the build
611 system. analyze() provides the necessary hook to implement such
612 behaviour. Analyzers independent of the build system can also be
613 specified here but at the cost of possible duplication of code between
614 the build system driver implementations.
615 """
616 raise NotImplemented
617
618
619class Autotools(BuildSystem):
620 def __init__(self, package=None, path=None):
621 super(Autotools, self).__init__(package, path)
622
623 def probe(self):
624 return os.path.isfile(os.path.join(self.path, 'configure.ac'))
625
626 def dependencies(self):
627 configure_ac = os.path.join(self.path, 'configure.ac')
628
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030629 contents = ''
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030630 # Prepend some special function overrides so we can parse out
631 # dependencies
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030632 for macro in DEPENDENCIES.keys():
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030633 contents += ('m4_define([' + macro + '], [' + macro + '_START$' +
634 str(DEPENDENCIES_OFFSET[macro] + 1) +
635 macro + '_END])\n')
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030636 with open(configure_ac, "rt") as f:
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030637 contents += f.read()
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030638
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030639 autoconf_cmdline = ['autoconf', '-Wno-undefined', '-']
640 autoconf_process = subprocess.Popen(autoconf_cmdline,
Andrew Jeffery7d4a26f2020-03-13 11:42:34 +1030641 stdin=subprocess.PIPE,
642 stdout=subprocess.PIPE,
643 stderr=subprocess.PIPE)
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030644 document = contents.encode('utf-8')
645 (stdout, stderr) = autoconf_process.communicate(input=document)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030646 if not stdout:
647 print(stderr)
648 raise Exception("Failed to run autoconf for parsing dependencies")
649
650 # Parse out all of the dependency text
651 matches = []
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030652 for macro in DEPENDENCIES.keys():
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030653 pattern = '(' + macro + ')_START(.*?)' + macro + '_END'
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030654 for match in re.compile(pattern).finditer(stdout.decode('utf-8')):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030655 matches.append((match.group(1), match.group(2)))
656
657 # Look up dependencies from the text
658 found_deps = []
659 for macro, deptext in matches:
660 for potential_dep in deptext.split(' '):
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030661 for known_dep in DEPENDENCIES[macro].keys():
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030662 if potential_dep.startswith(known_dep):
663 found_deps.append(DEPENDENCIES[macro][known_dep])
664
665 return found_deps
666
667 def _configure_feature(self, flag, enabled):
668 """
669 Returns an configure flag as a string
670
671 Parameters:
672 flag The name of the flag
673 enabled Whether the flag is enabled or disabled
674 """
675 return '--' + ('enable' if enabled else 'disable') + '-' + flag
676
677 def configure(self, build_for_testing):
678 self.build_for_testing = build_for_testing
679 conf_flags = [
680 self._configure_feature('silent-rules', False),
681 self._configure_feature('examples', build_for_testing),
682 self._configure_feature('tests', build_for_testing),
Ramin Izadpanah298d56b2020-11-17 16:55:28 +0000683 self._configure_feature('itests', INTEGRATION_TEST),
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030684 ]
Andrew Jefferyd0809692021-05-14 16:23:57 +0930685 conf_flags.extend([
686 self._configure_feature('code-coverage', build_for_testing),
687 self._configure_feature('valgrind', build_for_testing),
688 ])
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030689 # Add any necessary configure flags for package
690 if CONFIGURE_FLAGS.get(self.package) is not None:
691 conf_flags.extend(CONFIGURE_FLAGS.get(self.package))
692 for bootstrap in ['bootstrap.sh', 'bootstrap', 'autogen.sh']:
693 if os.path.exists(bootstrap):
694 check_call_cmd('./' + bootstrap)
695 break
696 check_call_cmd('./configure', *conf_flags)
697
698 def build(self):
699 check_call_cmd(*make_parallel)
700
701 def install(self):
Andrew Jeffery881041f2020-03-13 11:46:30 +1030702 check_call_cmd('sudo', '-n', '--', *(make_parallel + ['install']))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030703
704 def test(self):
705 try:
Andrew Jeffery881041f2020-03-13 11:46:30 +1030706 cmd = make_parallel + ['check']
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030707 for i in range(0, args.repeat):
708 check_call_cmd(*cmd)
Andrew Jefferyd0809692021-05-14 16:23:57 +0930709
710 maybe_make_valgrind()
711 maybe_make_coverage()
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030712 except CalledProcessError:
713 for root, _, files in os.walk(os.getcwd()):
714 if 'test-suite.log' not in files:
715 continue
716 check_call_cmd('cat', os.path.join(root, 'test-suite.log'))
717 raise Exception('Unit tests failed')
718
719 def analyze(self):
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030720 run_cppcheck()
721
722
723class CMake(BuildSystem):
724 def __init__(self, package=None, path=None):
725 super(CMake, self).__init__(package, path)
726
727 def probe(self):
728 return os.path.isfile(os.path.join(self.path, 'CMakeLists.txt'))
729
730 def dependencies(self):
731 return []
732
733 def configure(self, build_for_testing):
734 self.build_for_testing = build_for_testing
Ramin Izadpanah298d56b2020-11-17 16:55:28 +0000735 if INTEGRATION_TEST:
736 check_call_cmd('cmake', '-DCMAKE_EXPORT_COMPILE_COMMANDS=ON',
737 '-DITESTS=ON', '.')
738 else:
739 check_call_cmd('cmake', '-DCMAKE_EXPORT_COMPILE_COMMANDS=ON', '.')
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030740
741 def build(self):
742 check_call_cmd('cmake', '--build', '.', '--', '-j',
743 str(multiprocessing.cpu_count()))
744
745 def install(self):
746 pass
747
748 def test(self):
749 if make_target_exists('test'):
750 check_call_cmd('ctest', '.')
751
752 def analyze(self):
753 if os.path.isfile('.clang-tidy'):
Ed Tanousbc5c1c42020-10-01 17:49:20 -0700754 try:
755 os.mkdir("tidy-build")
756 except FileExistsError as e:
757 pass
758 # clang-tidy needs to run on a clang-specific build
759 check_call_cmd('cmake', '-DCMAKE_C_COMPILER=clang',
760 '-DCMAKE_CXX_COMPILER=clang++',
761 '-DCMAKE_EXPORT_COMPILE_COMMANDS=ON',
762 '-H.',
763 '-Btidy-build')
764 # we need to cd here because otherwise clang-tidy doesn't find the
765 # .clang-tidy file in the roots of repos. Its arguably a "bug"
766 # with run-clang-tidy at a minimum it's "weird" that it requires
767 # the .clang-tidy to be up a dir
Ed Tanous662890f2020-09-28 17:35:20 -0700768 os.chdir("tidy-build")
769 try:
Patrick Williamsf89c8502021-10-01 11:50:09 -0500770 check_call_cmd('run-clang-tidy', "-header-filter=.*", '-p',
Ed Tanousbc5c1c42020-10-01 17:49:20 -0700771 '.')
Ed Tanous662890f2020-09-28 17:35:20 -0700772 finally:
773 os.chdir("..")
774
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030775 maybe_make_valgrind()
776 maybe_make_coverage()
777 run_cppcheck()
778
779
780class Meson(BuildSystem):
781 def __init__(self, package=None, path=None):
782 super(Meson, self).__init__(package, path)
783
784 def probe(self):
785 return os.path.isfile(os.path.join(self.path, 'meson.build'))
786
787 def dependencies(self):
788 meson_build = os.path.join(self.path, 'meson.build')
789 if not os.path.exists(meson_build):
790 return []
791
792 found_deps = []
793 for root, dirs, files in os.walk(self.path):
794 if 'meson.build' not in files:
795 continue
796 with open(os.path.join(root, 'meson.build'), 'rt') as f:
797 build_contents = f.read()
Nan Zhouef13d532020-07-07 09:52:02 -0700798 pattern = r"dependency\('([^']*)'.*?\),?\n"
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030799 for match in re.finditer(pattern, build_contents):
800 group = match.group(1)
801 maybe_dep = DEPENDENCIES['PKG_CHECK_MODULES'].get(group)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030802 if maybe_dep is not None:
803 found_deps.append(maybe_dep)
804
805 return found_deps
806
807 def _parse_options(self, options_file):
808 """
809 Returns a set of options defined in the provides meson_options.txt file
810
811 Parameters:
812 options_file The file containing options
813 """
William A. Kennington III0ecd0bc2020-06-04 00:44:22 -0700814 oi = optinterpreter.OptionInterpreter('')
815 oi.process(options_file)
816 return oi.options
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030817
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700818 def _configure_boolean(self, val):
819 """
820 Returns the meson flag which signifies the value
821
822 True is true which requires the boolean.
823 False is false which disables the boolean.
824
825 Parameters:
826 val The value being converted
827 """
828 if val is True:
829 return 'true'
830 elif val is False:
831 return 'false'
832 else:
833 raise Exception("Bad meson boolean value")
834
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030835 def _configure_feature(self, val):
836 """
837 Returns the meson flag which signifies the value
838
839 True is enabled which requires the feature.
840 False is disabled which disables the feature.
841 None is auto which autodetects the feature.
842
843 Parameters:
844 val The value being converted
845 """
846 if val is True:
847 return "enabled"
848 elif val is False:
849 return "disabled"
850 elif val is None:
851 return "auto"
852 else:
853 raise Exception("Bad meson feature value")
854
William A. Kennington IIIfcd70772020-06-04 00:50:23 -0700855 def _configure_option(self, opts, key, val):
856 """
857 Returns the meson flag which signifies the value
858 based on the type of the opt
859
860 Parameters:
861 opt The meson option which we are setting
862 val The value being converted
863 """
864 if isinstance(opts[key], coredata.UserBooleanOption):
865 str_val = self._configure_boolean(val)
866 elif isinstance(opts[key], coredata.UserFeatureOption):
867 str_val = self._configure_feature(val)
868 else:
869 raise Exception('Unknown meson option type')
870 return "-D{}={}".format(key, str_val)
871
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030872 def configure(self, build_for_testing):
873 self.build_for_testing = build_for_testing
William A. Kennington III0ecd0bc2020-06-04 00:44:22 -0700874 meson_options = {}
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030875 if os.path.exists("meson_options.txt"):
876 meson_options = self._parse_options("meson_options.txt")
877 meson_flags = [
878 '-Db_colorout=never',
879 '-Dwerror=true',
880 '-Dwarning_level=3',
881 ]
882 if build_for_testing:
883 meson_flags.append('--buildtype=debug')
884 else:
885 meson_flags.append('--buildtype=debugoptimized')
Patrick Williamse95626d2021-04-13 11:13:53 -0500886 if OptionKey('tests') in meson_options:
Lei YU08d2b922022-04-25 11:21:36 +0800887 meson_flags.append(self._configure_option(
888 meson_options, OptionKey('tests'), build_for_testing))
Patrick Williamse95626d2021-04-13 11:13:53 -0500889 if OptionKey('examples') in meson_options:
Lei YU08d2b922022-04-25 11:21:36 +0800890 meson_flags.append(self._configure_option(
891 meson_options, OptionKey('examples'), build_for_testing))
Patrick Williamse95626d2021-04-13 11:13:53 -0500892 if OptionKey('itests') in meson_options:
Lei YU08d2b922022-04-25 11:21:36 +0800893 meson_flags.append(self._configure_option(
894 meson_options, OptionKey('itests'), INTEGRATION_TEST))
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030895 if MESON_FLAGS.get(self.package) is not None:
896 meson_flags.extend(MESON_FLAGS.get(self.package))
897 try:
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030898 check_call_cmd('meson', 'setup', '--reconfigure', 'build',
899 *meson_flags)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030900 except:
901 shutil.rmtree('build')
902 check_call_cmd('meson', 'setup', 'build', *meson_flags)
903
904 def build(self):
905 check_call_cmd('ninja', '-C', 'build')
906
907 def install(self):
908 check_call_cmd('sudo', '-n', '--', 'ninja', '-C', 'build', 'install')
909
910 def test(self):
Patrick Williams95095f12021-04-14 14:42:35 -0500911 # It is useful to check various settings of the meson.build file
912 # for compatibility, such as meson_version checks. We shouldn't
913 # do this in the configure path though because it affects subprojects
914 # and dependencies as well, but we only want this applied to the
915 # project-under-test (otherwise an upstream dependency could fail
916 # this check without our control).
917 self._extra_meson_checks()
918
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030919 try:
Brad Bishop7b8cef22020-08-26 15:58:09 -0400920 test_args = ('--repeat', str(args.repeat), '-C', 'build')
William A. Kennington IIId9062c42022-05-04 23:24:44 -0700921 check_call_cmd('meson', 'test', '--print-errorlogs', *test_args)
Brad Bishop7b8cef22020-08-26 15:58:09 -0400922
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030923 except CalledProcessError:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030924 raise Exception('Unit tests failed')
925
926 def _setup_exists(self, setup):
927 """
928 Returns whether the meson build supports the named test setup.
929
930 Parameter descriptions:
931 setup The setup target to check
932 """
933 try:
934 with open(os.devnull, 'w') as devnull:
935 output = subprocess.check_output(
Lei YU08d2b922022-04-25 11:21:36 +0800936 ['meson', 'test', '-C', 'build',
937 '--setup', setup, '-t', '0'],
938 stderr=subprocess.STDOUT)
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030939 except CalledProcessError as e:
940 output = e.output
Andrew Jeffery89b64b62020-03-13 12:15:48 +1030941 output = output.decode('utf-8')
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030942 return not re.search('Test setup .* not found from project', output)
943
944 def _maybe_valgrind(self):
945 """
946 Potentially runs the unit tests through valgrind for the package
Andrew Jeffery47fbfa52020-03-13 12:05:09 +1030947 via `meson test`. The package can specify custom valgrind
948 configurations by utilizing add_test_setup() in a meson.build
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030949 """
950 if not is_valgrind_safe():
951 sys.stderr.write("###### Skipping valgrind ######\n")
952 return
953 try:
954 if self._setup_exists('valgrind'):
Lei YU08d2b922022-04-25 11:21:36 +0800955 check_call_cmd('meson', 'test', '-t', '10', '-C', 'build',
William A. Kennington IIId9062c42022-05-04 23:24:44 -0700956 '--print-errorlogs', '--setup', 'valgrind')
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030957 else:
Lei YU08d2b922022-04-25 11:21:36 +0800958 check_call_cmd('meson', 'test', '-t', '10', '-C', 'build',
William A. Kennington IIId9062c42022-05-04 23:24:44 -0700959 '--print-errorlogs', '--wrapper', 'valgrind')
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030960 except CalledProcessError:
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030961 raise Exception('Valgrind tests failed')
962
963 def analyze(self):
964 self._maybe_valgrind()
965
966 # Run clang-tidy only if the project has a configuration
967 if os.path.isfile('.clang-tidy'):
Manojkiran Eda1aa91992020-10-02 14:11:53 +0530968 os.environ["CXX"] = "clang++"
969 check_call_cmd('meson', 'setup', 'build-clang')
Ed Tanous88db4c22022-01-25 15:15:59 -0800970 os.chdir("build-clang")
971 try:
972 check_call_cmd('run-clang-tidy', '-fix', '-format', '-p', '.')
973 except subprocess.CalledProcessError:
974 check_call_cmd("git", "-C", CODE_SCAN_DIR,
Lei YU08d2b922022-04-25 11:21:36 +0800975 "--no-pager", "diff")
Ed Tanous88db4c22022-01-25 15:15:59 -0800976 raise
977 finally:
978 os.chdir("..")
979
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030980 # Run the basic clang static analyzer otherwise
981 else:
982 check_call_cmd('ninja', '-C', 'build',
983 'scan-build')
984
985 # Run tests through sanitizers
986 # b_lundef is needed if clang++ is CXX since it resolves the
987 # asan symbols at runtime only. We don't want to set it earlier
988 # in the build process to ensure we don't have undefined
989 # runtime code.
990 if is_sanitize_safe():
991 check_call_cmd('meson', 'configure', 'build',
992 '-Db_sanitize=address,undefined',
993 '-Db_lundef=false')
William A. Kennington IIId9062c42022-05-04 23:24:44 -0700994 check_call_cmd('meson', 'test', '-C', 'build', '--print-errorlogs',
Andrew Jeffery15e423e2020-03-11 16:51:28 +1030995 '--logbase', 'testlog-ubasan')
996 # TODO: Fix memory sanitizer
Andrew Jeffery6015ca12020-03-13 11:50:10 +1030997 # check_call_cmd('meson', 'configure', 'build',
998 # '-Db_sanitize=memory')
999 # check_call_cmd('meson', 'test', '-C', 'build'
1000 # '--logbase', 'testlog-msan')
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301001 check_call_cmd('meson', 'configure', 'build',
Lei YUf684ad62020-09-21 17:02:26 +08001002 '-Db_sanitize=none')
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301003 else:
1004 sys.stderr.write("###### Skipping sanitizers ######\n")
1005
1006 # Run coverage checks
1007 check_call_cmd('meson', 'configure', 'build',
1008 '-Db_coverage=true')
1009 self.test()
1010 # Only build coverage HTML if coverage files were produced
1011 for root, dirs, files in os.walk('build'):
1012 if any([f.endswith('.gcda') for f in files]):
1013 check_call_cmd('ninja', '-C', 'build',
1014 'coverage-html')
1015 break
1016 check_call_cmd('meson', 'configure', 'build',
1017 '-Db_coverage=false')
1018 run_cppcheck()
1019
Patrick Williams95095f12021-04-14 14:42:35 -05001020 def _extra_meson_checks(self):
1021 with open(os.path.join(self.path, 'meson.build'), 'rt') as f:
1022 build_contents = f.read()
1023
1024 # Find project's specified meson_version.
1025 meson_version = None
1026 pattern = r"meson_version:[^']*'([^']*)'"
1027 for match in re.finditer(pattern, build_contents):
1028 group = match.group(1)
1029 meson_version = group
1030
1031 # C++20 requires at least Meson 0.57 but Meson itself doesn't
1032 # identify this. Add to our unit-test checks so that we don't
1033 # get a meson.build missing this.
1034 pattern = r"'cpp_std=c\+\+20'"
1035 for match in re.finditer(pattern, build_contents):
1036 if not meson_version or \
1037 not meson_version_compare(meson_version, ">=0.57"):
1038 raise Exception(
1039 "C++20 support requires specifying in meson.build: "
1040 + "meson_version: '>=0.57'"
1041 )
1042
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301043
1044class Package(object):
1045 def __init__(self, name=None, path=None):
Manojkiran Eda29d28252020-06-15 18:58:09 +05301046 self.supported = [Meson, Autotools, CMake]
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301047 self.name = name
1048 self.path = path
1049 self.test_only = False
1050
1051 def build_systems(self):
Andrew Jeffery881041f2020-03-13 11:46:30 +10301052 instances = (system(self.name, self.path) for system in self.supported)
1053 return (instance for instance in instances if instance.probe())
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301054
1055 def build_system(self, preferred=None):
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001056 systems = list(self.build_systems())
1057
1058 if not systems:
1059 return None
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301060
1061 if preferred:
Andrew Jeffery881041f2020-03-13 11:46:30 +10301062 return {type(system): system for system in systems}[preferred]
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301063
1064 return next(iter(systems))
1065
1066 def install(self, system=None):
1067 if not system:
1068 system = self.build_system()
1069
1070 system.configure(False)
1071 system.build()
1072 system.install()
1073
Andrew Jeffery19d75672020-03-13 10:42:08 +10301074 def _test_one(self, system):
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301075 system.configure(True)
1076 system.build()
1077 system.install()
1078 system.test()
Andrew Jefferyd0809692021-05-14 16:23:57 +09301079 if not TEST_ONLY:
1080 system.analyze()
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301081
Andrew Jeffery19d75672020-03-13 10:42:08 +10301082 def test(self):
1083 for system in self.build_systems():
1084 self._test_one(system)
1085
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301086
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001087def find_file(filename, basedir):
1088 """
Patrick Williams55448ad2020-12-14 14:28:28 -06001089 Finds all occurrences of a file (or list of files) in the base
1090 directory and passes them back with their relative paths.
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001091
1092 Parameter descriptions:
Patrick Williams55448ad2020-12-14 14:28:28 -06001093 filename The name of the file (or list of files) to
1094 find
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001095 basedir The base directory search in
1096 """
1097
Patrick Williams55448ad2020-12-14 14:28:28 -06001098 if not isinstance(filename, list):
Lei YU08d2b922022-04-25 11:21:36 +08001099 filename = [filename]
Patrick Williams55448ad2020-12-14 14:28:28 -06001100
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001101 filepaths = []
1102 for root, dirs, files in os.walk(basedir):
Brad Bishopeb667262021-08-06 13:54:58 -04001103 if os.path.split(root)[-1] == 'subprojects':
1104 for f in files:
1105 subproject = '.'.join(f.split('.')[0:-1])
1106 if f.endswith('.wrap') and subproject in dirs:
1107 # don't find files in meson subprojects with wraps
1108 dirs.remove(subproject)
Patrick Williams55448ad2020-12-14 14:28:28 -06001109 for f in filename:
1110 if f in files:
1111 filepaths.append(os.path.join(root, f))
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001112 return filepaths
1113
Andrew Jeffery9cc97d82020-03-13 11:57:44 +10301114
Matthew Barthccb7f852016-11-23 17:43:02 -06001115if __name__ == '__main__':
1116 # CONFIGURE_FLAGS = [GIT REPO]:[CONFIGURE FLAGS]
1117 CONFIGURE_FLAGS = {
Matthew Barth1d1c6732017-03-24 10:00:28 -05001118 'phosphor-logging':
Matt Spinler0744bb82020-01-16 08:23:35 -06001119 ['--enable-metadata-processing', '--enable-openpower-pel-extension',
Deepak Kodihalli3a4e1b42017-06-08 09:52:35 -05001120 'YAML_DIR=/usr/local/share/phosphor-dbus-yaml/yaml']
Matthew Barthccb7f852016-11-23 17:43:02 -06001121 }
1122
William A. Kennington III3f1d1202018-12-06 18:02:07 -08001123 # MESON_FLAGS = [GIT REPO]:[MESON FLAGS]
1124 MESON_FLAGS = {
Matt Spinler8448cfd2020-09-10 17:01:48 -05001125 'phosphor-dbus-interfaces':
Patrick Williams13af98a2021-04-21 15:17:00 -05001126 ['-Ddata_com_ibm=true', '-Ddata_org_open_power=true'],
1127 'phosphor-logging':
1128 ['-Dopenpower-pel-extension=enabled']
William A. Kennington III3f1d1202018-12-06 18:02:07 -08001129 }
1130
Matthew Barthccb7f852016-11-23 17:43:02 -06001131 # DEPENDENCIES = [MACRO]:[library/header]:[GIT REPO]
1132 DEPENDENCIES = {
1133 'AC_CHECK_LIB': {'mapper': 'phosphor-objmgr'},
Matthew Barth710f3f02017-01-18 15:20:19 -06001134 'AC_CHECK_HEADER': {
1135 'host-ipmid': 'phosphor-host-ipmid',
Patrick Ventureb41a4462018-10-03 17:27:38 -07001136 'blobs-ipmid': 'phosphor-ipmi-blobs',
Matthew Barth710f3f02017-01-18 15:20:19 -06001137 'sdbusplus': 'sdbusplus',
William A. Kennington IIIb4f730a2018-09-12 11:21:20 -07001138 'sdeventplus': 'sdeventplus',
William A. Kennington III23705242019-01-15 18:17:25 -08001139 'stdplus': 'stdplus',
Patrick Venture22329962018-09-14 10:23:04 -07001140 'gpioplus': 'gpioplus',
Saqib Khan66145052017-02-14 12:02:07 -06001141 'phosphor-logging/log.hpp': 'phosphor-logging',
Patrick Williamseab8a372017-01-30 11:21:32 -06001142 },
Brad Bishopebb49112017-02-13 11:07:26 -05001143 'AC_PATH_PROG': {'sdbus++': 'sdbusplus'},
Patrick Williamseab8a372017-01-30 11:21:32 -06001144 'PKG_CHECK_MODULES': {
Matthew Barth19e261e2017-02-01 12:55:22 -06001145 'phosphor-dbus-interfaces': 'phosphor-dbus-interfaces',
William A. Kennington III993ab332019-02-07 02:12:31 -08001146 'libipmid': 'phosphor-host-ipmid',
1147 'libipmid-host': 'phosphor-host-ipmid',
Brad Bishopebb49112017-02-13 11:07:26 -05001148 'sdbusplus': 'sdbusplus',
William A. Kennington IIIb4f730a2018-09-12 11:21:20 -07001149 'sdeventplus': 'sdeventplus',
William A. Kennington III23705242019-01-15 18:17:25 -08001150 'stdplus': 'stdplus',
Patrick Venture22329962018-09-14 10:23:04 -07001151 'gpioplus': 'gpioplus',
Brad Bishopebb49112017-02-13 11:07:26 -05001152 'phosphor-logging': 'phosphor-logging',
Marri Devender Raoa3eee8a2018-08-13 05:34:27 -05001153 'phosphor-snmp': 'phosphor-snmp',
Patrick Venturee584c3b2019-03-07 09:44:16 -08001154 'ipmiblob': 'ipmi-blob-tool',
Ben Tyner300ed3c2020-05-14 15:30:37 -05001155 'hei': 'openpower-libhei',
Sui Chen40cf5212020-11-24 12:12:23 -08001156 'phosphor-ipmi-blobs': 'phosphor-ipmi-blobs',
Willy Tu01534ec2021-10-14 10:16:45 -07001157 'libcr51sign': 'google-misc',
Brad Bishopebb49112017-02-13 11:07:26 -05001158 },
Matthew Barthccb7f852016-11-23 17:43:02 -06001159 }
1160
William A. Kennington IIIe67f5fc2018-12-06 17:40:30 -08001161 # Offset into array of macro parameters MACRO(0, 1, ...N)
1162 DEPENDENCIES_OFFSET = {
1163 'AC_CHECK_LIB': 0,
1164 'AC_CHECK_HEADER': 0,
1165 'AC_PATH_PROG': 1,
1166 'PKG_CHECK_MODULES': 1,
1167 }
1168
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -05001169 # DEPENDENCIES_REGEX = [GIT REPO]:[REGEX STRING]
1170 DEPENDENCIES_REGEX = {
Patrick Ventured37b8052018-10-16 16:03:03 -07001171 'phosphor-logging': r'\S+-dbus-interfaces$'
Leonel Gonzaleza62a1a12017-03-24 11:03:47 -05001172 }
1173
Matthew Barth33df8792016-12-19 14:30:17 -06001174 # Set command line arguments
1175 parser = argparse.ArgumentParser()
1176 parser.add_argument("-w", "--workspace", dest="WORKSPACE", required=True,
1177 help="Workspace directory location(i.e. /home)")
1178 parser.add_argument("-p", "--package", dest="PACKAGE", required=True,
1179 help="OpenBMC package to be unit tested")
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001180 parser.add_argument("-t", "--test-only", dest="TEST_ONLY",
1181 action="store_true", required=False, default=False,
1182 help="Only run test cases, no other validation")
Ramin Izadpanah298d56b2020-11-17 16:55:28 +00001183 arg_inttests = parser.add_mutually_exclusive_group()
1184 arg_inttests.add_argument("--integration-tests", dest="INTEGRATION_TEST",
Lei YU08d2b922022-04-25 11:21:36 +08001185 action="store_true", required=False, default=True,
1186 help="Enable integration tests [default].")
Ramin Izadpanah298d56b2020-11-17 16:55:28 +00001187 arg_inttests.add_argument("--no-integration-tests", dest="INTEGRATION_TEST",
Lei YU08d2b922022-04-25 11:21:36 +08001188 action="store_false", required=False,
1189 help="Disable integration tests.")
Matthew Barth33df8792016-12-19 14:30:17 -06001190 parser.add_argument("-v", "--verbose", action="store_true",
1191 help="Print additional package status messages")
Andrew Jeffery468309d2018-03-08 13:46:33 +10301192 parser.add_argument("-r", "--repeat", help="Repeat tests N times",
1193 type=int, default=1)
Andrew Geisslera61acb52019-01-03 16:32:44 -06001194 parser.add_argument("-b", "--branch", dest="BRANCH", required=False,
1195 help="Branch to target for dependent repositories",
1196 default="master")
Lei YU7ef93302019-11-06 13:53:21 +08001197 parser.add_argument("-n", "--noformat", dest="FORMAT",
1198 action="store_false", required=False,
1199 help="Whether or not to run format code")
Matthew Barth33df8792016-12-19 14:30:17 -06001200 args = parser.parse_args(sys.argv[1:])
1201 WORKSPACE = args.WORKSPACE
1202 UNIT_TEST_PKG = args.PACKAGE
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001203 TEST_ONLY = args.TEST_ONLY
Ramin Izadpanah298d56b2020-11-17 16:55:28 +00001204 INTEGRATION_TEST = args.INTEGRATION_TEST
Andrew Geisslera61acb52019-01-03 16:32:44 -06001205 BRANCH = args.BRANCH
Lei YU7ef93302019-11-06 13:53:21 +08001206 FORMAT_CODE = args.FORMAT
Matthew Barth33df8792016-12-19 14:30:17 -06001207 if args.verbose:
1208 def printline(*line):
1209 for arg in line:
Andrew Jeffery89b64b62020-03-13 12:15:48 +10301210 print(arg, end=' ')
1211 print()
Matthew Barth33df8792016-12-19 14:30:17 -06001212 else:
Andrew Jefferyfce557f2020-03-13 12:12:19 +10301213 def printline(*line):
1214 pass
Matthew Barthccb7f852016-11-23 17:43:02 -06001215
Patrick Williamsb6535952020-12-15 06:40:10 -06001216 CODE_SCAN_DIR = os.path.join(WORKSPACE, UNIT_TEST_PKG)
Lei YU7ef93302019-11-06 13:53:21 +08001217
James Feist878df5c2018-07-26 14:54:28 -07001218 # First validate code formatting if repo has style formatting files.
Adriana Kobylakbcee22b2018-01-10 16:58:27 -06001219 # The format-code.sh checks for these files.
Lei YU7ef93302019-11-06 13:53:21 +08001220 if FORMAT_CODE:
Patrick Williams5b4a1e82020-12-15 06:46:31 -06001221 format_scripts = find_file(['format-code.sh', 'format-code'],
1222 CODE_SCAN_DIR)
Andrew Geissler31502dd2021-01-07 14:04:02 -06001223
1224 # use default format-code.sh if no other found
1225 if not format_scripts:
1226 format_scripts.append(os.path.join(WORKSPACE, "format-code.sh"))
1227
Patrick Williams5b4a1e82020-12-15 06:46:31 -06001228 for f in format_scripts:
1229 check_call_cmd(f, CODE_SCAN_DIR)
Andrew Geisslera28286d2018-01-10 11:00:00 -08001230
Ed Tanous32768b82022-01-05 14:14:06 -08001231 # Check to see if any files changed
1232 check_call_cmd("git", "-C", CODE_SCAN_DIR,
1233 "--no-pager", "diff", "--exit-code")
1234
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001235 # Check if this repo has a supported make infrastructure
Patrick Williamsb6535952020-12-15 06:40:10 -06001236 pkg = Package(UNIT_TEST_PKG, CODE_SCAN_DIR)
Andrew Geissler8cb74fc2020-03-19 14:48:05 -05001237 if not pkg.build_system():
1238 print("No valid build system, exit")
1239 sys.exit(0)
1240
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301241 prev_umask = os.umask(000)
James Feist878df5c2018-07-26 14:54:28 -07001242
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301243 # Determine dependencies and add them
1244 dep_added = dict()
1245 dep_added[UNIT_TEST_PKG] = False
William A. Kennington III40d5c7c2018-12-13 14:37:59 -08001246
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301247 # Create dependency tree
1248 dep_tree = DepTree(UNIT_TEST_PKG)
Patrick Williamsb6535952020-12-15 06:40:10 -06001249 build_dep_tree(UNIT_TEST_PKG, CODE_SCAN_DIR, dep_added, dep_tree, BRANCH)
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001250
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301251 # Reorder Dependency Tree
Andrew Jeffery89b64b62020-03-13 12:15:48 +10301252 for pkg_name, regex_str in DEPENDENCIES_REGEX.items():
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301253 dep_tree.ReorderDeps(pkg_name, regex_str)
1254 if args.verbose:
1255 dep_tree.PrintTree()
William A. Kennington III65b37fa2019-01-31 15:15:17 -08001256
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301257 install_list = dep_tree.GetInstallList()
Andrew Geissler9ced4ed2019-11-18 14:33:58 -06001258
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301259 # We don't want to treat our package as a dependency
1260 install_list.remove(UNIT_TEST_PKG)
James Feist878df5c2018-07-26 14:54:28 -07001261
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301262 # Install reordered dependencies
1263 for dep in install_list:
1264 build_and_install(dep, False)
James Feist878df5c2018-07-26 14:54:28 -07001265
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301266 # Run package unit tests
1267 build_and_install(UNIT_TEST_PKG, True)
James Feist878df5c2018-07-26 14:54:28 -07001268
Andrew Jeffery15e423e2020-03-11 16:51:28 +10301269 os.umask(prev_umask)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001270
1271 # Run any custom CI scripts the repo has, of which there can be
1272 # multiple of and anywhere in the repository.
Patrick Williamsb6535952020-12-15 06:40:10 -06001273 ci_scripts = find_file(['run-ci.sh', 'run-ci'], CODE_SCAN_DIR)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001274 if ci_scripts:
Patrick Williamsb6535952020-12-15 06:40:10 -06001275 os.chdir(CODE_SCAN_DIR)
Matt Spinler9bfaaad2019-10-25 09:51:50 -05001276 for ci_script in ci_scripts:
Patrick Williams55448ad2020-12-14 14:28:28 -06001277 check_call_cmd(ci_script)