From: Scott Gasch <scott@gasch.org>
Date: Thu, 18 Aug 2022 02:53:17 +0000 (-0700)
Subject: Ugh.  Fix a bug in the new refactor of the env var config stuff.
X-Git-Url: https://wannabe.guru.org/gitweb/?a=commitdiff_plain;h=772038b8d852446b77c9ee857885874f2552e9a6;p=python_utils.git

Ugh.  Fix a bug in the new refactor of the env var config stuff.
---

diff --git a/config.py b/config.py
index 0c40166..a83433c 100644
--- a/config.py
+++ b/config.py
@@ -303,7 +303,6 @@ class Config:
     @staticmethod
     def _parse_arg_into_env(arg: str) -> Optional[Tuple[str, str, List[str]]]:
         """Internal helper to parse commandling args into environment vars."""
-
         arg = arg.strip()
         if not arg.startswith('['):
             return None
@@ -314,7 +313,7 @@ class Config:
 
         chunks = arg.split()
         if len(chunks) > 1:
-            var = arg.split()[0]
+            var = chunks[0]
         else:
             var = arg
 
@@ -323,7 +322,7 @@ class Config:
         env = var.upper()
         while env[0] == '-':
             env = env[1:]
-        return arg, env, chunks
+        return var, env, chunks
 
     def _augment_sys_argv_from_environment_variables(self):
         """Internal.  Look at the system environment for variables that match
@@ -364,9 +363,9 @@ class Config:
                                 from string_utils import to_bool
 
                                 if len(chunks) == 1 and to_bool(value):
-                                    sys.argv.append(arg)
+                                    sys.argv.append(var)
                                 elif len(chunks) > 1:
-                                    sys.argv.append(arg)
+                                    sys.argv.append(var)
                                     sys.argv.append(value)
                     arg = ''
 
diff --git a/text_utils.py b/text_utils.py
index 6437e62..890bc63 100644
--- a/text_utils.py
+++ b/text_utils.py
@@ -44,31 +44,37 @@ def get_console_rows_columns() -> RowsColumns:
     rows: Optional[str] = os.environ.get('LINES', None)
     cols: Optional[str] = os.environ.get('COLUMNS', None)
     if not rows or not cols:
+        logger.debug('Rows: %s, cols: %s, trying stty.', rows, cols)
         try:
             rows, cols = cmd(
                 "stty size",
                 timeout_seconds=1.0,
             ).split()
-        except Exception:
+        except Exception as e:
+            logger.exception(e)
             rows = None
             cols = None
 
     if rows is None:
+        logger.debug('Rows: %s, cols: %s, tput rows.', rows, cols)
         try:
             rows = cmd(
                 "tput rows",
                 timeout_seconds=1.0,
             )
-        except Exception:
+        except Exception as e:
+            logger.exception(e)
             rows = None
 
     if cols is None:
+        logger.debug('Rows: %s, cols: %s, tput cols.', rows, cols)
         try:
             cols = cmd(
                 "tput cols",
                 timeout_seconds=1.0,
             )
-        except Exception:
+        except Exception as e:
+            logger.exception(e)
             cols = None
 
     if not rows or not cols: