Issue24508
Created on 2015-06-25 04:26 by zach.ware, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| 2.7_project_files_backport.diff | zach.ware, 2015-06-25 04:26 | |||
| Messages (9) | |||
|---|---|---|---|
| msg245795 - (view) | Author: Zachary Ware (zach.ware) * ![]() |
Date: 2015-06-25 04:26 | |
Here's the (enormous) patch backporting 3.5's project files to 2.7. It also takes care of adjusting the old project files to their new home in PC/VS9.0, and makes a couple of small adjustments in distutils.command.build_ext, FixTk, regrtest, and the Tcl/Tk tests to keep the test suite running properly. I've run a couple of builds on the custom buildbots, and at this point there are no failures that I can see any reason to blame this patch for. I would greatly appreciate some review, or at least 'review by testing' to make sure this doesn't completely break anybody's workflow. Terry, I'm particularly interested in your feedback since I'm pretty sure you generally build from the VS GUI, which I usually avoid doing. PCbuild\pcbuild.sln *will not* work with the VS 2008 GUI, but it should work with any other VS, 2010 or newer. If you'd prefer to continue using the 2008 GUI, PC\VS9.0\pcbuild.sln should continue to work. When using the new project files, though, this should also take care of the headache of making Tcl/Tk play the game; you shouldn't have to do anything extra to be able to use Tkinter. Steve, I did not make any changes to Tools\msi, but I'd be happy to if it makes your job any easier. Looking forward to your feedback, Zach |
|||
| msg245797 - (view) | Author: Steve Dower (steve.dower) * ![]() |
Date: 2015-06-25 04:56 | |
Feel free to fix up msi.py as much as it needs. I don't particularly understand it - I just run it :) |
|||
| msg245798 - (view) | Author: Zachary Ware (zach.ware) * ![]() |
Date: 2015-06-25 05:06 | |
Do you plan to use the old project files or the new? For the new, there should be no change in Tools/msi, which would make fixing it very easy for me ;) |
|||
| msg245809 - (view) | Author: Steve Dower (steve.dower) * ![]() |
Date: 2015-06-25 13:03 | |
I'll try out the changes when I get a bit of time today or tomorrow and decide then what I'll do. msi.py doesn't actually build the projects AFAIK, so it's probably okay. It might try to read them to get paths though. |
|||
| msg245881 - (view) | Author: Steve Dower (steve.dower) * ![]() |
Date: 2015-06-27 15:45 | |
Looks like the required install configuration is: * VS 2008 + VS 2010, or * VS 2008 + VS 2013 or later + Windows SDK 7.1 The SDK is at http://www.microsoft.com/en-us/download/details.aspx?id=8279 and contains MSVC10 and platform toolset files for both VC9 and 10. Neither VS 2008 nor VS 2013 include the VC9 platform toolset. The VC for Python package is also not sufficient, as it doesn't include rc.exe (though I'd be surprised if the output of newer versions of rc.exe aren't compatible). I haven't checked thoroughly to see what else may be missing. |
|||
| msg245882 - (view) | Author: Steve Dower (steve.dower) * ![]() |
Date: 2015-06-27 15:48 | |
32-bit build was fine (VS 2013 + SDK 7.1 + VS 2008), but something has gone funny with the 64-bit. Need to go AFK for a bit now, but I'll try and figure that out later today. |
|||
| msg245893 - (view) | Author: Steve Dower (steve.dower) * ![]() |
Date: 2015-06-27 20:29 | |
64-bit build didn't work (couldn't find ws2_32.lib) until I ran the SDK Configuration utility and selected v6.0A as the default SDK. This may be because I forgot to explicitly select the 64-bit compilers when I initially installed VS 2008, but it's a worthwhile troubleshooting approach. Still building some of the big projects now, and I have to leave it again, but it looks like everything is okay once you get past setup. I'm +0 on this change now, and I really hope the buildbots will pick up any subtle changes in the build. We should also run as many tests as possible against existing extension modules. |
|||
| msg246041 - (view) | Author: Zachary Ware (zach.ware) * ![]() |
Date: 2015-07-01 17:08 | |
Thanks for testing this, Steve! It sounds like PCbuild/readme.txt will need a significant update in the setup portion to cover the different ways to get what is needed (and to mention PC/VS9.0 as a simpler setup). I use third-party extension modules rarely enough (and even more rarely on Windows) that I'm not sure what a good set of test extension modules would be. I'm happy to run some tests if somebody can suggest some packages for it, including ones that come with binaries and ones that require compilation. With no negative votes standing, I plan to fix the readmes and commit this before the end of next week. |
|||
| msg246790 - (view) | Author: Roundup Robot (python-dev) ![]() |
Date: 2015-07-16 05:42 | |
New changeset ca78b9449e04 by Zachary Ware in branch '2.7': Close #24508: Backport the 3.5 MSBuild project files. https://hg.python.org/cpython/rev/ca78b9449e04 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:18 | admin | set | github: 68696 |
| 2015-07-16 05:42:07 | python-dev | set | status: open -> closed nosy:
+ python-dev resolution: fixed |
| 2015-07-01 17:08:03 | zach.ware | set | messages: + msg246041 |
| 2015-06-27 20:29:16 | steve.dower | set | messages: + msg245893 |
| 2015-06-27 15:48:57 | steve.dower | set | messages: + msg245882 |
| 2015-06-27 15:45:04 | steve.dower | set | messages: + msg245881 |
| 2015-06-25 23:21:29 | eric.snow | set | nosy:
+ loewis |
| 2015-06-25 13:03:50 | steve.dower | set | messages: + msg245809 |
| 2015-06-25 05:06:49 | zach.ware | set | messages: + msg245798 |
| 2015-06-25 04:56:13 | steve.dower | set | messages: + msg245797 |
| 2015-06-25 04:26:48 | zach.ware | create | |

