On Mon, Jan 04, 2021 at 12:30:11 +0100, Tim Wiederhake wrote:
Using 'argparse' for argument handling simplifies merging this script with cpu-gather.py in a later patch.
Signed-off-by: Tim Wiederhake <twiederh@redhat.com> --- tests/cputestdata/cpu-cpuid.py | 78 +++++++++++++++++++--------------- 1 file changed, 43 insertions(+), 35 deletions(-)
diff --git a/tests/cputestdata/cpu-cpuid.py b/tests/cputestdata/cpu-cpuid.py index dac43debb6..6ca72d2262 100755 --- a/tests/cputestdata/cpu-cpuid.py +++ b/tests/cputestdata/cpu-cpuid.py ... +def main(): + parser = argparse.ArgumentParser(description="Diff cpuid results") + subparsers = parser.add_subparsers(dest="action", required=True) + diffparser = subparsers.add_parser( + "diff", + help="Diff json description of CPU model against known features.") + diffparser.add_argument( + "json_files", + nargs="+", + metavar="FILE", + type=os.path.realpath, + help="Path to one or more json CPU model descriptions.") + args = parser.parse_args() + + diff(args) + exit(0)
This exit(0) here is redundant, it's removed by the last patch in this series, and our syntax check doesn't like it. So I just removed this line before pushing. Jirka