From 772038b8d852446b77c9ee857885874f2552e9a6 Mon Sep 17 00:00:00 2001 From: Scott Gasch Date: Wed, 17 Aug 2022 19:53:17 -0700 Subject: [PATCH] Ugh. Fix a bug in the new refactor of the env var config stuff. --- config.py | 9 ++++----- text_utils.py | 12 +++++++++--- 2 files changed, 13 insertions(+), 8 deletions(-) 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: -- 2.47.0