fix: make it work under python3.

plus some ruff cleanups.

Author: rouilj

scripts/contributors.py

@@ -8,6 +8,7 @@
 """
 
 from __future__ import print_function
+
 from subprocess import check_output
 
 # --- output settings
@@ -29,11 +30,10 @@
   'Stefan Seefeld <[email protected]>':
       ['Stefan Seefeld <[email protected]>'],
   'John Rouillard <[email protected]>':
-      ['rouilj@uland', 'rouilj']
-
+      ['rouilj@uland', 'rouilj'],
 }
 ROBOTS = ['No Author <[email protected]>']
-# /-- 
+# /--
 
 
 def compress(years):
@@ -85,8 +85,12 @@ def compress(years):
   if verbose:
     print("Getting HG log...")
     print("Using: ", command)
-  
+
   authorship = check_output(command, shell=True)
+
+  if not isinstance(authorship, str):
+    authorship = authorship.decode('utf-8')
+
   # authorship are strings like
   # 2003,Richard Jones <[email protected]>
   # ...
@@ -111,11 +115,11 @@ def compress(years):
     author = author.replace('<', '(')
     author = author.replace('>', ')')
     # years
-    if not year in years:
+    if year not in years:
       years[year] = set()
     years[year].add(author)
     # names
-    if not author in names:
+    if author not in names:
       names[author] = set()
     names[author].add(int(year))
 
@@ -134,14 +138,14 @@ def compress(years):
     if verbose:
       print("Years for each contributor...")
     print('')
-    
+
     def last_year(name):
       """Return year of the latest contribution for a given name"""
-      return sorted(list(names[name]))[-1]
+      return sorted(names[name])[-1]
 
     def first_year(name):
       """Return year of the first contribution"""
-      return sorted(list(names[name]))[0]
+      return sorted(names[name])[0]
 
     def year_key(name):
       """
@@ -150,9 +154,9 @@ def year_key(name):
       the most recent and long-term contributors are at the top.
       """
       return (last_year(name), -first_year(name))
-    
+
     print("Copyright (c)")
-    for author in sorted(list(names), key=year_key, reverse=True):
+    for author in sorted(names, key=year_key, reverse=True):
       years = list(names[author])
       yearstr = compress(years)