From: Gerhard Sittig <redacted>
Date: Sun, 20 Nov 2016 15:51:47 +0000 (+0100)
Subject: build support: sort list of installed items, factor out pretty printer
X-Git-Tag: libsigrokdecode-0.5.0~117
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=3ab22459a92d2e129362ded379f6b75c0f6d4cdf;p=libsigrokdecode.git

build support: sort list of installed items, factor out pretty printer

Alpha-sort the list of installed items, to support developers which scan
the list for newly added items. Factor out the logic which pretty prints
installed items while maintaining a maximum screen output line length.
---

diff --git a/tools/install-decoders b/tools/install-decoders
index 8445da5..3772bd5 100755
--- a/tools/install-decoders
+++ b/tools/install-decoders
@@ -24,6 +24,20 @@ from shutil import copy
 from getopt import getopt
 
 
+_inst_pp_col_max = 80
+_inst_pp_col = 0
+def _install_pretty_print(item):
+    """Pretty print an install item. Enforce maximum line width."""
+    global _inst_pp_col
+    if item is None:
+        _inst_pp_col = 0
+        return
+    _inst_pp_col += len(item)
+    if _inst_pp_col > _inst_pp_col_max:
+        print()
+        _inst_pp_col = len(item)
+    print(item, end = "")
+
 def install(srcdir, dstdir, s):
     worklist = []
     for pd in os.listdir(srcdir):
@@ -42,15 +56,10 @@ def install(srcdir, dstdir, s):
         if install_list:
             worklist.append((pd, pd_dir, install_list))
 
+    worklist.sort()
     print("Installing %d %s:" % (len(worklist), s))
-    col = 0
     for pd, pd_dir, install_list in worklist:
-        msg = pd + ' '
-        if (col + len(msg) > 80):
-            print()
-            col = 0
-        print(msg, end='')
-        col += len(msg)
+        _install_pretty_print("{} ".format(pd))
         pd_dst = os.path.join(dstdir, pd)
         try:
             os.mkdir(pd_dst)
@@ -62,6 +71,7 @@ def install(srcdir, dstdir, s):
         for f in install_list:
             copy(os.path.join(pd_dir, f), pd_dst)
     print()
+    _install_pretty_print(None)
 
 
 def config_get_extra_install(config_file):