[New-bugs-announce] [issue37396] [2.7] PCbuild/pythoncore.vcxproj kills the wrong program

[STINNER Victor]

New submission from STINNER Victor <vstinner at redhat.com>:

x86 Windows XP 2.7:
https://buildbot.python.org/all/#/builders/45/builds/429

Killing any running python.exe instances...
...
C:\Program Files\MSBuild\Microsoft.Cpp\v4.0\Microsoft.CppClean.targets(74,5): warning : Access to the path 'd:\cygwin\home\db3l\buildarea\2.7.bolen-windows\build\PCBuild\python_d.exe' is denied. [d:\cygwin\home\db3l\buildarea\2.7.bolen-windows\build\PCbuild\python.vcxproj]

It's a debug build: it should kill python_d.exe, not python.exe.

--

PCbuild/pythoncore.vcxproj uses:

  <PropertyGroup>
    <MakeVersionInfoBeforeTarget>ClCompile</MakeVersionInfoBeforeTarget>
    <KillPython>true</KillPython>
  </PropertyGroup>

which comes from PCbuild/pyproject.vcxproj:

  <Target Name="KillPython" BeforeTargets="PrepareForBuild" Condition="'$(KillPython)' == 'true'">
    <Message Text="Killing any running python.exe instances..." Importance="high" />
    <KillPython FileName="$(OutDir)python$(PyDebugExt).exe" />
  </Target>

Problem: it seems like $(PyDebugExt) is not set to "_d" in PCbuild/pyproject.vcxproj.

Should PCbuild/pyproject.vcxproj explicitly import python.props which defines it?

    <!-- Suffix for all binaries when building for debug -->
    <PyDebugExt Condition="'$(PyDebugExt)' == '' and $(Configuration) == 'Debug'">_d</PyDebugExt>

Note: this issue is about Python 2.7.

----------
components: Build
messages: 346471
nosy: steve.dower, vstinner, zach.ware
priority: normal
severity: normal
status: open
title: [2.7] PCbuild/pythoncore.vcxproj kills the wrong program
versions: Python 2.7

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue37396>
_______________________________________