| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 1 | #!/usr/bin/env python | 
 | 2 | # | 
 | 3 | # Copyright (C) 2018 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. | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 16 | """Verify that one set of hidden API flags is a subset of another.""" | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 17 |  | 
 | 18 | import argparse | 
 | 19 | import csv | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 20 | import sys | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 21 | from itertools import chain | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 22 |  | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 23 | from signature_trie import signature_trie | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 24 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 25 |  | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 26 | def dict_reader(csv_file): | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 27 |     return csv.DictReader( | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 28 |         csv_file, delimiter=",", quotechar="|", fieldnames=["signature"]) | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 29 |  | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 30 |  | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 31 | def read_flag_trie_from_file(file): | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 32 |     with open(file, "r", encoding="utf8") as stream: | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 33 |         return read_flag_trie_from_stream(stream) | 
 | 34 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 35 |  | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 36 | def read_flag_trie_from_stream(stream): | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 37 |     trie = signature_trie() | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 38 |     reader = dict_reader(stream) | 
 | 39 |     for row in reader: | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 40 |         signature = row["signature"] | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 41 |         trie.add(signature, row) | 
 | 42 |     return trie | 
 | 43 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 44 |  | 
 | 45 | def extract_subset_from_monolithic_flags_as_dict_from_file( | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 46 |         monolithic_trie, patterns_file): | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 47 |     """Extract a subset of flags from the dict of monolithic flags. | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 48 |  | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 49 |     :param monolithic_trie: the trie containing all the monolithic flags. | 
 | 50 |     :param patterns_file: a file containing a list of signature patterns that | 
| Paul Duffin | 67b9d61 | 2021-07-21 17:38:47 +0100 | [diff] [blame] | 51 |     define the subset. | 
 | 52 |     :return: the dict from signature to row. | 
 | 53 |     """ | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 54 |     with open(patterns_file, "r", encoding="utf8") as stream: | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 55 |         return extract_subset_from_monolithic_flags_as_dict_from_stream( | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 56 |             monolithic_trie, stream) | 
| Paul Duffin | 67b9d61 | 2021-07-21 17:38:47 +0100 | [diff] [blame] | 57 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 58 |  | 
 | 59 | def extract_subset_from_monolithic_flags_as_dict_from_stream( | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 60 |         monolithic_trie, stream): | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 61 |     """Extract a subset of flags from the trie of monolithic flags. | 
| Paul Duffin | 67b9d61 | 2021-07-21 17:38:47 +0100 | [diff] [blame] | 62 |  | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 63 |     :param monolithic_trie: the trie containing all the monolithic flags. | 
| Paul Duffin | 67b9d61 | 2021-07-21 17:38:47 +0100 | [diff] [blame] | 64 |     :param stream: a stream containing a list of signature patterns that define | 
 | 65 |     the subset. | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 66 |     :return: the dict from signature to row. | 
 | 67 |     """ | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 68 |     dict_signature_to_row = {} | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 69 |     for pattern in stream: | 
 | 70 |         pattern = pattern.rstrip() | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 71 |         rows = monolithic_trie.get_matching_rows(pattern) | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 72 |         for row in rows: | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 73 |             signature = row["signature"] | 
 | 74 |             dict_signature_to_row[signature] = row | 
 | 75 |     return dict_signature_to_row | 
 | 76 |  | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 77 |  | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 78 | def read_signature_csv_from_stream_as_dict(stream): | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 79 |     """Read the csv contents from the stream into a dict. | 
 | 80 |  | 
 | 81 |     The first column is assumed to be the signature and used as the | 
 | 82 |     key. | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 83 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 84 |     The whole row is stored as the value. | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 85 |     :param stream: the csv contents to read | 
 | 86 |     :return: the dict from signature to row. | 
 | 87 |     """ | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 88 |     dict_signature_to_row = {} | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 89 |     reader = dict_reader(stream) | 
 | 90 |     for row in reader: | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 91 |         signature = row["signature"] | 
 | 92 |         dict_signature_to_row[signature] = row | 
 | 93 |     return dict_signature_to_row | 
 | 94 |  | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 95 |  | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 96 | def read_signature_csv_from_file_as_dict(csv_file): | 
| Paul Duffin | b5cd522 | 2022-02-28 19:06:49 +0000 | [diff] [blame] | 97 |     """Read the csvFile into a dict. | 
 | 98 |  | 
 | 99 |     The first column is assumed to be the signature and used as the | 
 | 100 |     key. | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 101 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 102 |     The whole row is stored as the value. | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 103 |     :param csv_file: the csv file to read | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 104 |     :return: the dict from signature to row. | 
 | 105 |     """ | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 106 |     with open(csv_file, "r", encoding="utf8") as f: | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 107 |         return read_signature_csv_from_stream_as_dict(f) | 
 | 108 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 109 |  | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 110 | def compare_signature_flags(monolithic_flags_dict, modular_flags_dict, | 
 | 111 |                             implementation_flags): | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 112 |     """Compare the signature flags between the two dicts. | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 113 |  | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 114 |     :param monolithic_flags_dict: the dict containing the subset of the | 
 | 115 |     monolithic flags that should be equal to the modular flags. | 
 | 116 |     :param modular_flags_dict:the dict containing the flags produced by a single | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 117 |     bootclasspath_fragment module. | 
 | 118 |     :return: list of mismatches., each mismatch is a tuple where the first item | 
 | 119 |     is the signature, and the second and third items are lists of the flags from | 
 | 120 |     modular dict, and monolithic dict respectively. | 
 | 121 |     """ | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 122 |     mismatching_signatures = [] | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 123 |     # Create a sorted set of all the signatures from both the monolithic and | 
 | 124 |     # modular dicts. | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 125 |     all_signatures = sorted( | 
 | 126 |         set(chain(monolithic_flags_dict.keys(), modular_flags_dict.keys()))) | 
 | 127 |     for signature in all_signatures: | 
 | 128 |         monolithic_row = monolithic_flags_dict.get(signature, {}) | 
 | 129 |         monolithic_flags = monolithic_row.get(None, []) | 
 | 130 |         if signature in modular_flags_dict: | 
 | 131 |             modular_row = modular_flags_dict.get(signature, {}) | 
 | 132 |             modular_flags = modular_row.get(None, []) | 
| Paul Duffin | 280bae6 | 2021-07-20 18:03:53 +0100 | [diff] [blame] | 133 |         else: | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 134 |             modular_flags = implementation_flags | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 135 |         if monolithic_flags != modular_flags: | 
 | 136 |             mismatching_signatures.append( | 
 | 137 |                 (signature, modular_flags, monolithic_flags)) | 
 | 138 |     return mismatching_signatures | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 139 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 140 |  | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 141 | def main(argv): | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 142 |     args_parser = argparse.ArgumentParser( | 
 | 143 |         description="Verify that sets of hidden API flags are each a subset of " | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 144 |         "the monolithic flag file. For each module this uses the provided " | 
 | 145 |         "signature patterns to select a subset of the monolithic flags and " | 
 | 146 |         "then it compares that subset against the filtered flags provided by " | 
 | 147 |         "the module. If the module's filtered flags does not contain flags for " | 
 | 148 |         "a signature then it is assumed to have been filtered out because it " | 
 | 149 |         "was not part of an API and so is assumed to have the implementation " | 
 | 150 |         "flags.") | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 151 |     args_parser.add_argument( | 
| Paul Duffin | 0c12b78 | 2022-04-08 00:28:11 +0100 | [diff] [blame] | 152 |         "--monolithic-flags", help="The monolithic flag file") | 
 | 153 |     args_parser.add_argument( | 
 | 154 |         "--module-flags", | 
 | 155 |         action="append", | 
 | 156 |         help="A colon separated pair of paths. The first is a path to a " | 
 | 157 |         "filtered set of flags, and the second is a path to a set of " | 
 | 158 |         "signature patterns that identify the set of classes belonging to " | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 159 |         "a single bootclasspath_fragment module. Specify once for each module " | 
 | 160 |         "that needs to be checked.") | 
 | 161 |     args_parser.add_argument( | 
 | 162 |         "--implementation-flag", | 
 | 163 |         action="append", | 
 | 164 |         help="A flag in the set of flags that identifies a signature which is " | 
 | 165 |         "not part of an API, i.e. is the signature of a private implementation " | 
 | 166 |         "member. Specify as many times as necessary to define the " | 
 | 167 |         "implementation flag set. If this is not specified then the " | 
 | 168 |         "implementation flag set is empty.") | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 169 |     args = args_parser.parse_args(argv[1:]) | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 170 |  | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 171 |     # Read in all the flags into the trie | 
| Paul Duffin | 0c12b78 | 2022-04-08 00:28:11 +0100 | [diff] [blame] | 172 |     monolithic_flags_path = args.monolithic_flags | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 173 |     monolithic_trie = read_flag_trie_from_file(monolithic_flags_path) | 
| Paul Duffin | dfa1083 | 2021-05-13 17:31:51 +0100 | [diff] [blame] | 174 |  | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 175 |     implementation_flags = args.implementation_flag or [] | 
 | 176 |  | 
| Paul Duffin | 53a7607 | 2021-07-21 17:27:09 +0100 | [diff] [blame] | 177 |     # For each subset specified on the command line, create dicts for the flags | 
| Paul Duffin | c11f667 | 2021-07-20 00:04:21 +0100 | [diff] [blame] | 178 |     # provided by the subset and the corresponding flags from the complete set | 
 | 179 |     # of flags and compare them. | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 180 |     failed = False | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 181 |     module_pairs = args.module_flags or [] | 
 | 182 |     for modular_pair in module_pairs: | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 183 |         parts = modular_pair.split(":") | 
 | 184 |         modular_flags_path = parts[0] | 
 | 185 |         modular_patterns_path = parts[1] | 
 | 186 |         modular_flags_dict = read_signature_csv_from_file_as_dict( | 
 | 187 |             modular_flags_path) | 
 | 188 |         monolithic_flags_subset_dict = \ | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 189 |             extract_subset_from_monolithic_flags_as_dict_from_file( | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 190 |                 monolithic_trie, modular_patterns_path) | 
 | 191 |         mismatching_signatures = compare_signature_flags( | 
| Paul Duffin | bd88c88 | 2022-04-07 23:32:19 +0100 | [diff] [blame] | 192 |             monolithic_flags_subset_dict, modular_flags_dict, | 
 | 193 |             implementation_flags) | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 194 |         if mismatching_signatures: | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 195 |             failed = True | 
 | 196 |             print("ERROR: Hidden API flags are inconsistent:") | 
| Paul Duffin | 181b56c | 2022-04-08 00:03:32 +0100 | [diff] [blame] | 197 |             print("< " + modular_flags_path) | 
 | 198 |             print("> " + monolithic_flags_path) | 
 | 199 |             for mismatch in mismatching_signatures: | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 200 |                 signature = mismatch[0] | 
 | 201 |                 print() | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 202 |                 print("< " + ",".join([signature] + mismatch[1])) | 
 | 203 |                 print("> " + ",".join([signature] + mismatch[2])) | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 204 |  | 
 | 205 |     if failed: | 
 | 206 |         sys.exit(1) | 
 | 207 |  | 
| Spandan Das | 559132f | 2021-08-25 18:17:33 +0000 | [diff] [blame] | 208 |  | 
| Paul Duffin | 428c651 | 2021-07-21 15:33:22 +0100 | [diff] [blame] | 209 | if __name__ == "__main__": | 
 | 210 |     main(sys.argv) |