Skip to content

Commit d70dc45

Browse files
godlygeekpradyunsg
authored andcommitted
Improve the error for failed PEP 517 builds
The original message would result in errors like: Could not build wheels for xmlsec, which is required to install pyproject.toml-based projects Which could be misinterpreted to mean that xmlsec is required to install pyproject.toml-based projects, rather than that building wheels is. This commit aims to make the error message clearer while still working for a comma-separated list of package names. Signed-off-by: Matt Wozniski <[email protected]>
1 parent a33caa2 commit d70dc45

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

news/7f9639a2-df21-4e0c-9023-80f00fd71d20.trivial.rst

Whitespace-only changes.

src/pip/_internal/commands/install.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -427,8 +427,8 @@ def run(self, options: Values, args: List[str]) -> int:
427427

428428
if build_failures:
429429
raise InstallationError(
430-
"Could not build wheels for {}, which is required to "
431-
"install pyproject.toml-based projects".format(
430+
"ERROR: Failed to build installable wheels for some "
431+
"pyproject.toml based projects ({})".format(
432432
", ".join(r.name for r in build_failures) # type: ignore
433433
)
434434
)

0 commit comments

Comments
 (0)