This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: test_distutils.py fails on VC6(Windows)
Type: Stage:
Components: Distutils Versions: Python 2.7
process
Status: closed Resolution:
Dependencies: Superseder:
Assigned To: tarek Nosy List: ocean-city, rpetrov, tarek
Priority: normal Keywords: patch

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 issue5977-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:49adminsetgithub: 50296
2009-05-18 08:12:55tareksetstatus: open -> closed

messages: + msg88019
2009-05-18 06:27:05ocean-citysetmessages: + msg88017
2009-05-18 00:17:18ocean-citysetmessages: + msg88010
2009-05-17 20:51:28rpetrovsetnosy: + rpetrov
messages: + msg88001
2009-05-17 11:00:55ocean-citysetfiles: + investigate.patch
keywords: + patch
messages: + msg87986
2009-05-17 10:35:26ocean-citysetmessages: + msg87981
2009-05-17 10:18:43tareksetmessages: + msg87979
2009-05-17 09:56:52tareksetmessages: + msg87977
2009-05-17 08:56:11ocean-citysetmessages: - msg87971
2009-05-17 08:56:00ocean-citysetmessages: + msg87973
2009-05-17 08:55:05ocean-citysetmessages: + msg87971
2009-05-17 08:50:06ocean-citycreate