commit ca227ecc2bfd7582675b735b8ff81477cce26cac
author Mike Frysinger <vapier@chromium.org> Tue Jul 02 17:59:21 2019 -0400
committer Commit Bot <commit-bot@chromium.org> Sat Jul 06 04:04:38 2019 +0000
tree c1680a2169c57da952e7599217f09378987ea63f
parent 28f766851ebb223e99ea2b45e8abe795923b1046

stop using cmp builtin

This doesn't exist in Python 3, and our use of it isn't necessary.
Switch to plain == operator instead.

BUG=chromium:980619
TEST=unittests pass

Change-Id: I05d93b935f51a4f29fe6d3cd05d28a14a12a62db
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1688540
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

scripts/upload_symbols.py

diff --git a/scripts/upload_symbols.py b/scripts/upload_symbols.py
index 5380ff0..f577132 100644
--- a/scripts/upload_symbols.py
+++ b/scripts/upload_symbols.py
@@ -390,7 +390,7 @@
       set_match = {'debugId': b.header.id.replace('-', ''),
                    'debugFile': b.header.name}
       for cs_result in result.get('pairs', []):
-        if cmp(set_match, cs_result.get('symbolId')) == 0:
+        if set_match == cs_result.get('symbolId'):
           if cs_result.get('status') == 'FOUND':
             logging.debug('Found duplicate: %s', b.display_name)
             b.status = SymbolFile.DUPLICATE