fix(available_interfaces): strip only trailing colon from entries

Leave other punctuation alone, at least for now. Closes #1133
scop · May 7, 2024 · a452aee · a452aee
1 parent 53fdce4
commit a452aee
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/bash_completion b/bash_completion
@@ -1737,7 +1737,7 @@ _comp_compgen_available_interfaces()
         fi
     } 2>/dev/null | _comp_awk \
         '/^[^ \t]/ { if ($1 ~ /^[0-9]+:/) { print $2 } else { print $1 } }')" &&
-        _comp_compgen -U generated set "${generated[@]%%[[:punct:]]*}"
+        _comp_compgen -U generated set "${generated[@]%:}"
 }
 
 # Echo number of CPUs, falling back to 1 on failure.

diff --git a/test/t/unit/test_unit_compgen_available_interfaces.py b/test/t/unit/test_unit_compgen_available_interfaces.py
@@ -22,4 +22,4 @@ def test_1_trailing_colons(self, bash, functions):
             "_comp__test_compgen available_interfaces",
             want_output=True,
         )
-        assert ":" not in output.strip()
+        assert ":>" not in output.strip()