Issue 6046: test_distutils.py fails on VC6(Windows) (original) (raw)

Created on 2009-05-17 08:50 by ocean-city, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
investigate.patch ocean-city,2009-05-17 11:00
Messages (10)
msg87970 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-17 08:50
test_disutils(test_get_outputs) fails on VC6. I cannot know if this happens on VC9 too because buildbot is down. :-( ///////////////////////////////////////////////////////////// test_get_outputs (distutils.tests.test_build_ext.BuildExtTestCase) ... foo.c ライブラリ c:\docume~1\whiter~1\locals~1\temp\tmpzdhkyv\tempt\docume~1\whiter~1\l ocals~1\temp\tmpkhvw2m\foo.lib とオブジェクト c:\docume~1\whiter~1\locals~1\temp\tmp zdhkyv\tempt\docume~1\whiter~1\locals~1\temp\tmpkhvw2m\foo.exp を作成中 CVPACK : 致命的なエラー CK1003: ファイル c:\docume~1\whiter~1\locals~1\temp\tmpb4w8fe\f oo.exe を開くことができません LINK : warning LNK4027: CVPACK error ///////////////////////////////////////////////////////////// I looked into the directory "tmpkhvw2m" before it would be deleted, but I could not find "foo.exe". There was only "foo".
msg87973 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-17 08:56
Here is translated version. test_get_outputs (distutils.tests.test_build_ext.BuildExtTestCase) ... foo.c Library c:\docume~1\whiter~1\locals~1\temp\tmpzdhkyv\tempt\docume~1\whiter~1\l ocals~1\temp\tmpkhvw2m\foo.lib and object c:\docume~1\whiter~1\locals~1\temp\tmp zdhkyv\tempt\docume~1\whiter~1\locals~1\temp\tmpkhvw2m\foo.exp is being created CVPACK : fatal error CK1003: File c:\docume~1\whiter~1\locals~1\temp\tmpb4w8fe\f oo.exe cannot be opened LINK : warning LNK4027: CVPACK error
msg87977 - (view) Author: Tarek Ziadé (tarek) * (Python committer) Date: 2009-05-17 09:56
When I made this change I checked it under VC9, so it should work on this version. I'll have to create a VM with VC6 to track it down. Until then I'll deactivate this test when it's VC6
msg87979 - (view) Author: Tarek Ziadé (tarek) * (Python committer) Date: 2009-05-17 10:18
Ok done. can you check on your side ? Don't we have a win32+MVCS6 buildslave somewhere ? That would be nice to have. I am running 3 differents VMs already when I work on these files. It would be great to have buildbot making regression tests over all compiler flavors.
msg87981 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-17 10:35
Ah, well, if this runs on VC9, I think you don't have to worry about VC6. (VC6 is too old) Of course, I'm happy if this runs on VC6 too. Anyway, there is msvc9compiler.py, so maybe does different code run on between VC9 and VC8(or elder)? I don't know about buildbot slave.
msg87986 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-17 11:00
It seems there is two workarounds. 1. remove /pdb:None from linker arguments. 2. Add extension to output filename. (like attached patch) Maybe, if there is /pdb:None in linker arguments and output filename doesn't have extension, link.exe may think it has default extension ".exe". I'm not sure.
msg88001 - (view) Author: Roumen Petrov (rpetrov) * Date: 2009-05-17 20:51
The patch -w32.patch from issue 5977 has to fix this. The patch just add suffix for target binary, i.e. on linux target will become foo.so and on w32 foo.pyd. Note .pyd is correct sugfxi for w32.
msg88010 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-18 00:17
Roumen, your patch worked fine. ---------------------------------------------------------------------- Ran 115 tests in 53.207s OK
msg88017 - (view) Author: Hirokazu Yamamoto (ocean-city) * (Python committer) Date: 2009-05-18 06:26
> It would be great to have buildbot making regression tests > over all compiler flavors. Yes, I hope snakebite.org will enable us to access to several MSVC compilers.
msg88019 - (view) Author: Tarek Ziadé (tarek) * (Python committer) Date: 2009-05-18 08:12
Thanks Roumen,that was it. I've commited the change and added a test to verify the produced file has the right extension. Thanks for the feedback Hirokazu
History
Date User Action Args
2022-04-11 14:56:49 admin set github: 50296
2009-05-18 08:12:55 tarek set status: open -> closedmessages: +
2009-05-18 06:27:05 ocean-city set messages: +
2009-05-18 00:17:18 ocean-city set messages: +
2009-05-17 20:51:28 rpetrov set nosy: + rpetrovmessages: +
2009-05-17 11:00:55 ocean-city set files: + investigate.patchkeywords: + patchmessages: +
2009-05-17 10:35:26 ocean-city set messages: +
2009-05-17 10🔞43 tarek set messages: +
2009-05-17 09:56:52 tarek set messages: +
2009-05-17 08:56:11 ocean-city set messages: -
2009-05-17 08:56:00 ocean-city set messages: +
2009-05-17 08:55:05 ocean-city set messages: +
2009-05-17 08:50:06 ocean-city create