Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | |
| 3 | # Copyright (C) 2023 The Android Open Source Project |
| 4 | # |
| 5 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 6 | # you may not use this file except in compliance with the License. |
| 7 | # You may obtain a copy of the License at |
| 8 | # |
| 9 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 10 | # |
| 11 | # Unless required by applicable law or agreed to in writing, software |
| 12 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 13 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 14 | # See the License for the specific language governing permissions and |
| 15 | # limitations under the License. |
| 16 | |
| 17 | import argparse |
| 18 | import collections |
| 19 | import json |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 20 | import os |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 21 | import subprocess |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 22 | import sys |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 23 | import tempfile |
| 24 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 25 | def get_top() -> str: |
| 26 | path = '.' |
| 27 | while not os.path.isfile(os.path.join(path, 'build/soong/tests/genrule_sandbox_test.py')): |
| 28 | if os.path.abspath(path) == '/': |
| 29 | sys.exit('Could not find android source tree root.') |
| 30 | path = os.path.join(path, '..') |
| 31 | return os.path.abspath(path) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 32 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 33 | def _build_with_soong(targets, target_product, *, keep_going = False, extra_env={}): |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 34 | env = { |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 35 | **os.environ, |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 36 | "TARGET_PRODUCT": target_product, |
| 37 | "TARGET_BUILD_VARIANT": "userdebug", |
| 38 | } |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 39 | env.update(extra_env) |
| 40 | args = [ |
| 41 | "build/soong/soong_ui.bash", |
| 42 | "--make-mode", |
| 43 | "--skip-soong-tests", |
| 44 | ] |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 45 | if keep_going: |
| 46 | args.append("-k") |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 47 | args.extend(targets) |
| 48 | try: |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 49 | subprocess.check_output( |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 50 | args, |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 51 | env=env, |
| 52 | ) |
| 53 | except subprocess.CalledProcessError as e: |
| 54 | print(e) |
| 55 | print(e.stdout) |
| 56 | print(e.stderr) |
| 57 | exit(1) |
| 58 | |
| 59 | |
| 60 | def _find_outputs_for_modules(modules, out_dir, target_product): |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 61 | module_path = os.path.join(out_dir, "soong", "module-actions.json") |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 62 | |
| 63 | if not os.path.exists(module_path): |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 64 | _build_with_soong(["json-module-graph"], target_product) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 65 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 66 | with open(module_path) as f: |
| 67 | action_graph = json.load(f) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 68 | |
| 69 | module_to_outs = collections.defaultdict(set) |
| 70 | for mod in action_graph: |
| 71 | name = mod["Name"] |
| 72 | if name in modules: |
| 73 | for act in mod["Module"]["Actions"]: |
Jason Macnak | 394f1b7 | 2023-06-15 09:28:26 -0700 | [diff] [blame] | 74 | if "}generate" in act["Desc"]: |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 75 | module_to_outs[name].update(act["Outputs"]) |
| 76 | return module_to_outs |
| 77 | |
| 78 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 79 | def _compare_outputs(module_to_outs, tempdir) -> dict[str, list[str]]: |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 80 | different_modules = collections.defaultdict(list) |
| 81 | for module, outs in module_to_outs.items(): |
| 82 | for out in outs: |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 83 | try: |
| 84 | subprocess.check_output(["diff", os.path.join(tempdir, out), out]) |
| 85 | except subprocess.CalledProcessError as e: |
| 86 | different_modules[module].append(e.stdout) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 87 | |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 88 | return different_modules |
| 89 | |
| 90 | |
| 91 | def main(): |
| 92 | parser = argparse.ArgumentParser() |
| 93 | parser.add_argument( |
| 94 | "--target_product", |
| 95 | "-t", |
| 96 | default="aosp_cf_arm64_phone", |
| 97 | help="optional, target product, always runs as eng", |
| 98 | ) |
| 99 | parser.add_argument( |
| 100 | "modules", |
| 101 | nargs="+", |
| 102 | help="modules to compare builds with genrule sandboxing enabled/not", |
| 103 | ) |
| 104 | parser.add_argument( |
| 105 | "--show-diff", |
| 106 | "-d", |
| 107 | action="store_true", |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 108 | help="whether to display differing files", |
| 109 | ) |
Liz Kammer | 2fb361c | 2023-06-09 11:29:48 -0400 | [diff] [blame] | 110 | parser.add_argument( |
| 111 | "--output-paths-only", |
| 112 | "-o", |
| 113 | action="store_true", |
Liz Kammer | 2fb361c | 2023-06-09 11:29:48 -0400 | [diff] [blame] | 114 | help="Whether to only return the output paths per module", |
| 115 | ) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 116 | args = parser.parse_args() |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 117 | os.chdir(get_top()) |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 118 | |
| 119 | out_dir = os.environ.get("OUT_DIR", "out") |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 120 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 121 | print("finding output files for the modules...") |
| 122 | module_to_outs = _find_outputs_for_modules(set(args.modules), out_dir, args.target_product) |
Jason Macnak | 394f1b7 | 2023-06-15 09:28:26 -0700 | [diff] [blame] | 123 | if not module_to_outs: |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 124 | sys.exit("No outputs found") |
Jason Macnak | 394f1b7 | 2023-06-15 09:28:26 -0700 | [diff] [blame] | 125 | |
Liz Kammer | 2fb361c | 2023-06-09 11:29:48 -0400 | [diff] [blame] | 126 | if args.output_paths_only: |
| 127 | for m, o in module_to_outs.items(): |
| 128 | print(f"{m} outputs: {o}") |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 129 | sys.exit(0) |
Liz Kammer | 2fb361c | 2023-06-09 11:29:48 -0400 | [diff] [blame] | 130 | |
Cole Faust | bbe2cc6 | 2023-07-12 17:55:28 -0700 | [diff] [blame^] | 131 | all_outs = list(set.union(*module_to_outs.values())) |
| 132 | |
| 133 | print("building without sandboxing...") |
| 134 | _build_with_soong(all_outs, args.target_product) |
| 135 | with tempfile.TemporaryDirectory() as tempdir: |
| 136 | for f in all_outs: |
| 137 | subprocess.check_call(["cp", "--parents", f, tempdir]) |
| 138 | |
| 139 | print("building with sandboxing...") |
| 140 | _build_with_soong( |
| 141 | all_outs, |
| 142 | args.target_product, |
| 143 | # We've verified these build without sandboxing already, so do the sandboxing build |
| 144 | # with keep_going = True so that we can find all the genrules that fail to build with |
| 145 | # sandboxing. |
| 146 | keep_going = True, |
| 147 | extra_env={"GENRULE_SANDBOXING": "true"}, |
| 148 | ) |
| 149 | |
| 150 | diffs = _compare_outputs(module_to_outs, tempdir) |
| 151 | if len(diffs) == 0: |
| 152 | print("All modules are correct") |
| 153 | elif args.show_diff: |
| 154 | for m, d in diffs.items(): |
| 155 | print(f"Module {m} has diffs {d}") |
| 156 | else: |
| 157 | print(f"Modules {list(diffs.keys())} have diffs") |
Liz Kammer | 767fad4 | 2023-06-09 11:23:15 -0400 | [diff] [blame] | 158 | |
| 159 | |
| 160 | if __name__ == "__main__": |
| 161 | main() |