Merge pull request #10 from BolunThompson/bash

Minor bash expansion bug fix
binpash · Jan 23, 2025 · 6e53105 · 6e53105
2 parents 08f52af + 5611608
commit 6e53105
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "sh-expand"
-version = "0.2.0"
+version = "0.2.1"
 authors = [
   { name="Michael Greenberg", email="[email protected]" },
   { name="Konstantinos Kallas", email="[email protected]" },

diff --git a/setup.py b/setup.py
@@ -4,7 +4,7 @@
 long_description = (Path(__file__).parent / "README.md").read_text()
 
 setup(name='sh-expand',
-      version='0.2.0',
+      version='0.2.1',
       packages=['sh_expand'],
       ## Necessary for the markdown to be properly rendered
       long_description=long_description,

diff --git a/sh_expand/bash_expand.py b/sh_expand/bash_expand.py
@@ -130,8 +130,6 @@ def should_expand_var(word: list[CArgChar]) -> bool:
             raise ImpureExpansion("Potential dangerous tilde expansion:", carg)
         if pair in {"<(", ">(", "$("}:
             raise ImpureExpansion("Potential process substitution", carg)
-        if char == "(":
-            raise ImpureExpansion("Potential array", carg)
         if char == "$":
             seen_dollar_sign = True
         if char in need_to_expand_chars: