lzy/sci-gui-agent-benchmark
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ver Jan31stv2

Browse Source 
added a warning comment about expanding tildes in command parameter of
subprocess.xxx
This commit is contained in:
David Chang
2024-01-31 10:56:24 +08:00
parent 2f1a9ba56c
commit 46b99147e9
1 changed files with 0 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
desktop_env/server/main.py
View File

@@ -56,7 +56,6 @@ def execute_command():
if isinstance(command, str) and not shell: if isinstance(command, str) and not shell:
command = shlex.split(command) command = shlex.split(command)
# WARNING: auto expansion should be completed by set `shell=True` and use str as `command`
# Expand user directory # Expand user directory
for i, arg in enumerate(command): for i, arg in enumerate(command):
if arg.startswith("~/"): if arg.startswith("~/"):
@@ -99,7 +98,6 @@ def launch_app():
if isinstance(command, str) and not shell: if isinstance(command, str) and not shell:
command = shlex.split(command) command = shlex.split(command)
# WARNING: auto expansion should be completed by set `shell=True` and use str as `command`
# Expand user directory # Expand user directory
for i, arg in enumerate(command): for i, arg in enumerate(command):
if arg.startswith("~/"): if arg.startswith("~/"):