fix: rollback shell=True for both windows and linux compatibility (#53)

Davis-3450 · web-flow · commit b7abaa67cd82 · 2025-10-03T10:13:53.000-05:00
diff --git a/android_unpinner/vendor/platform_tools/__init__.py b/android_unpinner/vendor/platform_tools/__init__.py
@@ -23,7 +23,7 @@ def adb(cmd: str) -> subprocess.CompletedProcess[str]:
     full_cmd = f"{base} {cmd}"
     try:
         proc = subprocess.run(
-            full_cmd, shell=False, check=True, capture_output=True, text=True
+            full_cmd, shell=True, check=True, capture_output=True, text=True
         )
     except subprocess.CalledProcessError as e:
         logging.debug(f"cmd='{full_cmd}'\n" f"{e.stdout=}\n" f"{e.stderr=}")

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ def adb(cmd: str) -> subprocess.CompletedProcess[str]:`
`23`	`23`	`full_cmd = f"{base} {cmd}"`
`24`	`24`	`try:`
`25`	`25`	`proc = subprocess.run(`
`26`		`- full_cmd, shell=False, check=True, capture_output=True, text=True`
	`26`	`+ full_cmd, shell=True, check=True, capture_output=True, text=True`
`27`	`27`	`)`
`28`	`28`	`except subprocess.CalledProcessError as e:`
`29`	`29`	`logging.debug(f"cmd='{full_cmd}'\n" f"{e.stdout=}\n" f"{e.stderr=}")`