[Python-Dev] r62129 - in python/trunk: PCbuild/debug.vsprops PCbuild/kill_python.c PCbuild/kill_python.vcproj PCbuild/pcbuild.sln PCbuild/pythoncore.vcproj PCbuild/release.vsprops Tools/buildbot/Makefile Tools/buildbot/build-amd64.bat Tools/buildbot/bui... (original) (raw)

Trent Nelson tnelson at onresolve.com
Fri Apr 4 12:57:15 CEST 2008

I don't like the part where the solution kills the Python process during a rebuild. It's too surprising for the user.

Hmmm. When you say rebuild, I assume you mean the change I made to the pythoncore project's pre-link step to call kill_python.exe, and not to the post-build step of kill_python that runs itself? Personally, when I'm doing development, if I've got the pcbuild\python_d.exe console open, it's usually to test one liners, I'm not using it to do anything important. If I forget to close it before I kick off a build, it's annoying running into errors at the link stage, I'd certainly prefer the build system to kill off anything that'll inhibit a successful link before actually linking.

What do others that do Windows development think? I don't have a problem changing the build behaviour if the approach I've taken is generally disliked.

Trent.

From: Christian Heimes [lists at cheimes.de] Sent: 04 April 2008 09:25 To: python-checkins at python.org; Trent Nelson Subject: Re: r62129 - in python/trunk: PCbuild/debug.vsprops PCbuild/kill_python.c PCbuild/kill_python.vcproj PCbuild/pcbuild.sln PCbuild/pythoncore.vcproj PCbuild/release.vsprops Tools/buildbot/Makefile Tools/buildbot/build-amd64.bat Tools/buildbot/bui...

trent.nelson schrieb:

Author: trent.nelson Date: Thu Apr 3 20:27:06 2008 New Revision: 62129

Added: python/trunk/PCbuild/killpython.c (contents, props changed) python/trunk/PCbuild/killpython.vcproj Removed: python/trunk/Tools/buildbot/Makefile python/trunk/Tools/buildbot/killpython.bat python/trunk/Tools/buildbot/killpython.c python/trunk/Tools/buildbot/killpython.mak Modified: python/trunk/PCbuild/debug.vsprops python/trunk/PCbuild/pcbuild.sln python/trunk/PCbuild/pythoncore.vcproj python/trunk/PCbuild/release.vsprops python/trunk/Tools/buildbot/build-amd64.bat python/trunk/Tools/buildbot/build.bat python/trunk/Tools/buildbot/buildmsi.bat Log: Reimplement killpython. The existing version had a number of flaws, namely, it didn't work for x64 and it wasn't precise about which pythond.exe it was killing -- it just killed the first one it came across that happened to have 'pcbuild\pythond.exe' or 'build\pythond.exe' in it's path. The new version has been rewritten from the ground up and now lives in PCbuild, instead of Tools\buildbot, and it has also been incorporated into the Visual Studio solution (pcbuild.sln) as 'killpython'. The solution has also been altered such that killpython is called where necessary in the build process in order to prevent any linking errors due to open file locks. In lieu of this, all of the existing bits and pieces in Tools\buildbot that called out to killpython at various points have also be en removed as they are now obsolete. Tested on both Win32 and x64.

I don't like the part where the solution kills the Python process during a rebuild. It's too surprising for the user.

Christian

More information about the Python-Dev mailing list