Skip to content

Commit 28e5eb9

Browse files
committed
Avoid unnecessary indirection
Since there's not equivalent of get_isolated_environment_bin_path for the distutils backend, the additional declaretion in the sysconfig backend is unnecessary.
1 parent 77ef9f0 commit 28e5eb9

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

src/pip/_internal/locations/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -530,4 +530,4 @@ def get_isolated_environment_lib_paths(prefix: str) -> List[str]:
530530

531531

532532
def get_isolated_environment_bin_path(prefix: str) -> str:
533-
return _sysconfig.get_isolated_environment_bin_path(prefix)
533+
return _sysconfig.get_isolated_environment_paths(prefix)["scripts"]

src/pip/_internal/locations/_sysconfig.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -213,18 +213,13 @@ def get_platlib() -> str:
213213
return sysconfig.get_paths()["platlib"]
214214

215215

216-
def _get_isolated_environment_paths(prefix: str) -> typing.Dict[str, str]:
216+
def get_isolated_environment_paths(prefix: str) -> typing.Dict[str, str]:
217217
variables = {"base": prefix, "platbase": prefix}
218218
if "venv" in sysconfig.get_scheme_names():
219219
return sysconfig.get_paths(vars=variables, scheme="venv")
220220
return sysconfig.get_paths(vars=variables)
221221

222222

223223
def get_isolated_environment_lib_paths(prefix: str) -> typing.Tuple[str, str]:
224-
paths = _get_isolated_environment_paths(prefix)
224+
paths = get_isolated_environment_paths(prefix)
225225
return (paths["purelib"], paths["platlib"])
226-
227-
228-
def get_isolated_environment_bin_path(prefix: str) -> str:
229-
paths = _get_isolated_environment_paths(prefix)
230-
return paths["scripts"]

0 commit comments

Comments
 (0)