diff options
author | Simon McVittie <smcv@debian.org> | 2018-01-17 15:19:12 +0000 |
---|---|---|
committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-02-22 19:24:47 +0000 |
commit | 6cabeaed3f45c8f3e05fb5e1152bfa5b96a3bc79 (patch) | |
tree | e6355aaf9568e2898c95cc48964b792392197d2c /tests/bootloader-entries-crosscheck.py | |
parent | 98b597b465a556f3ad5b16478d7edee3037fbd4d (diff) | |
download | ostree-6cabeaed3f45c8f3e05fb5e1152bfa5b96a3bc79.tar.gz |
tests/bootloader-entries-crosscheck: Use Python 3-friendly sorting
This is a little clearer than a strcmp()-style negative/zero/positive
return, and also works in Python 2.
Signed-off-by: Simon McVittie <smcv@debian.org>
Closes: #1457
Approved by: cgwalters
Diffstat (limited to 'tests/bootloader-entries-crosscheck.py')
-rwxr-xr-x | tests/bootloader-entries-crosscheck.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/bootloader-entries-crosscheck.py b/tests/bootloader-entries-crosscheck.py index 38e8e451..5faa5487 100755 --- a/tests/bootloader-entries-crosscheck.py +++ b/tests/bootloader-entries-crosscheck.py @@ -38,8 +38,8 @@ def fatal(msg): sys.stderr.write('\n') sys.exit(1) -def compare_entries_descending(a, b): - return int(b['version']) - int(a['version']) +def entry_get_version(entry): + return int(entry['version']) def get_ostree_option(optionstring): for o in optionstring.split(): @@ -65,7 +65,7 @@ for fname in os.listdir(loaderpath): v = line[s+1:] entry[k] = v entries.append(entry) - entries.sort(compare_entries_descending) + entries.sort(key=entry_get_version, reverse=True) # Parse SYSLINUX config with open(syslinuxpath) as f: |