Hongru/cibuildwheel
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Try removing shell

Browse Source
This commit is contained in:
Henry Schreiner
2020-07-06 10:49:24 -04:00
parent c68be58e78
commit 8466194576
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cibuildwheel/windows.py
+2 -3
View File
@@ -22,7 +22,7 @@ def shell(args: Sequence[Union[str, PathLike]], env: Optional[Dict[str, str]] =
cwd: Optional[str] = None) -> int: cwd: Optional[str] = None) -> int:
command = ' '.join(str(a) for a in args) command = ' '.join(str(a) for a in args)
print(f'+ {command}') print(f'+ {command}')
return subprocess.check_call(command, env=env, cwd=cwd, shell=True) return subprocess.check_call(list(str(a) for a in args), env=env, cwd=cwd)
def get_nuget_args(version: str, arch: str) -> List[str]: def get_nuget_args(version: str, arch: str) -> List[str]:
@@ -172,8 +172,7 @@ def pep_518_cp35_workaround(package_dir: Path, env: Dict[str, str]) -> None:
else [] else []
) )
if requirements: if requirements:
escaped_requirements = [f'"{s}"' for s in requirements] shell(['pip', 'install'] + requirements, env=env)
shell(['pip', 'install'] + escaped_requirements, env=env)
def build(options: BuildOptions) -> None: def build(options: BuildOptions) -> None: