[Python-checkins] bpo-39930: Convert error to warning for more silent failure (GH-18921)
Miss Islington (bot)
webhook-mailer at python.org
Wed Mar 11 10:30:35 EDT 2020
https://github.com/python/cpython/commit/3136f6f287a56afe87c0fd3e70c15c6e53ae0814
commit: 3136f6f287a56afe87c0fd3e70c15c6e53ae0814
branch: 3.7
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: GitHub <noreply at github.com>
date: 2020-03-11T07:30:25-07:00
summary:
bpo-39930: Convert error to warning for more silent failure (GH-18921)
Makes it an error to create a layout without vcruntime DLL
(cherry picked from commit fde44ae6d08d3df79554155b1cf079e73a8fabdd)
Co-authored-by: Steve Dower <steve.dower at python.org>
files:
M PC/layout/main.py
M PCbuild/pyproject.props
diff --git a/PC/layout/main.py b/PC/layout/main.py
index 910085c01bb2e..4d47b2e78f3f2 100644
--- a/PC/layout/main.py
+++ b/PC/layout/main.py
@@ -153,8 +153,12 @@ def in_build(f, dest="", new_name=None):
if ns.include_stable:
yield from in_build(PYTHON_STABLE_DLL_NAME)
+ found_any = False
for dest, src in rglob(ns.build, "vcruntime*.dll"):
+ found_any = True
yield dest, src
+ if not found_any:
+ log_error("Failed to locate vcruntime DLL in the build.")
yield "LICENSE.txt", ns.source / "LICENSE"
diff --git a/PCbuild/pyproject.props b/PCbuild/pyproject.props
index f1eac3641cc60..d76dd74d8a80d 100644
--- a/PCbuild/pyproject.props
+++ b/PCbuild/pyproject.props
@@ -211,7 +211,7 @@ public override bool Execute() {
<VCRuntimeDLL Include="$(VCRedistDir)\Microsoft.VC*.CRT\vcruntime*.dll" />
</ItemGroup>
- <Error Text="vcruntime14*.dll not found under $(VCInstallDir)" Condition="@(VCRuntimeDLL) == ''" />
+ <Warning Text="vcruntime14*.dll not found under $(VCInstallDir)" Condition="@(VCRuntimeDLL) == ''" />
<Message Text="VCRuntimeDLL: @(VCRuntimeDLL)" Importance="high" />
</Target>
</Project>
More information about the Python-checkins
mailing list