8000 bpo-39930: Convert error to warning for more silent failure (GH-18921) · python/cpython@fde44ae · GitHub
[go: up one dir, main page]

Skip to content

Commit fde44ae

Browse files
authored
bpo-39930: Convert error to warning for more silent failure (GH-18921)
Makes it an error to create a layout without vcruntime DLL
1 parent 3c97e1e commit fde44ae

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

PC/layout/main.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,8 +173,12 @@ def in_build(f, dest="", new_name=None):
173173
if ns.include_stable:
174174
yield from in_build(PYTHON_STABLE_DLL_NAME)
175175

176+
found_any = False
176177
for dest, src in rglob(ns.build, "vcruntime*.dll"):
178+
found_any = True
177179
yield dest, src
180+
if not found_any:
181+
log_error("Failed to locate vcruntime DLL in the build.")
178182

179183
yield "LICENSE.txt", ns.build / "LICENSE.txt"
180184

PCbuild/pyproject.props

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ public override bool Execute() {
217217
<VCRuntimeDLL Include="$(VCRedistDir)\Microsoft.VC*.CRT\vcruntime*.dll" />
218218
</ItemGroup>
219219

220-
<Error Text="vcruntime14*.dll not found under $(VCInstallDir)" Condition="@(VCRuntimeDLL) == ''" />
220+
<Warning Text="vcruntime14*.dll not found under $(VCInstallDir)" Condition="@(VCRuntimeDLL) == ''" />
221221
<Message Text="VCRuntimeDLL: @(VCRuntimeDLL)" Importance="high" />
222222
</Target>
223223
</Project>

0 commit comments

Comments
 (0)
0