[Patches] [ python-Patches-614770 ] MSVC 7.0 compiler support
noreply@sourceforge.net
noreply@sourceforge.net
Tue, 08 Oct 2002 08:51:33 -0700
Patches item #614770, was opened at 2002-09-25 20:09
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=614770&group_id=5470
Category: Distutils and setup.py
Group: Python 2.2.x
Status: Open
Resolution: None
Priority: 5
Submitted By: John Anderson (djohnanderson)
Assigned to: Nobody/Anonymous (nobody)
Summary: MSVC 7.0 compiler support
Initial Comment:
Distutils doesn't work with the current shipping
version of the Microsoft compiler (7.0). I've got a
patch that fixes it (context diffs of msvccompiler.py
against the latest code in CVS).
----------------------------------------------------------------------
>Comment By: John Anderson (djohnanderson)
Date: 2002-10-08 08:51
Message:
Logged In: YES
user_id=618290
Thanks Martin for taking the time to review my proposed changes.
I think your comments about looking in
Software\Microsoft\VisualStudio instead of
Software\Microsoft\DevStudio is a good point. I decided to
two look only in VisualStudio because that works for both
the version 6 and 7 compiler I tested with. I don't know if
the version 5 compiler also stores the version in DevStudio.
Another alternative, which would of course complicate code,
would be two look in both places and choose the highest
version. Let me know if that's what you'd prefer and I'll
upload a new patch.
The problem I'm having with the version 6 compiler (latest
service Pack 5) is that
SOFTWARE\Microsoft\Devstudio\6.0\Build System\ doesn't
exist. Instead it looks like it's been moved to
SOFTWARE\Microsoft\Shared Tools\Build System\, but in that
new location, SOFTWARE\Microsoft\Shared Tools\Build
System\Components\Platforms\Win32 (x86)\Directories doesn't
exist. This has the effect of not getting the correct
include directories for builds.
This also points out a serious flaw in looking at
undocumented registry entries to find information for the
build -- there's no guarantee that the registry information
won't change even within the same version of the compiler. I
don't have a good solution for this problem, but I'd rather
distutils reported an error when it couldn't find the
registry entries it expected -- rather than silently
ignoring it as it does now. In a few places I added code to
report unexpected missing registry entries, but not all. I
could if you're interested add error code in all cases.
Fixing the problem I'm having with the version 6 compiler
seems difficult, since it seems to work for you and doesn't
work for me -- apparently are registries are different.
Personally I'm content with leaving the version 6 compiler
broken since it isn't obvious how to fix it and it
apparently works for some people and I only intend to use
the version 7 or newer compilers.
I added three new functions: convert_mbcs, read_key, and by
far the largest: expand_macros. The first two make the code
simpler, easier to read, avoid unnecessary duplications, and
minimize the risk that someone would forget to deal with
mbcs. It would be difficult to understand the bug fix
without these two functions. My hope was that these changes
would make it easier for the next person who needs to learn
the code. The last, expand_macros, is necessary because the
version 7 compiler introduces macros which didn't exist in
previous versions of the compiler. It would be awkward to
implement the macros without having adding a new function.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-10-08 02:21
Message:
Logged In: YES
user_id=21627
I'm asking because you are not looking into
Software\Microsoft\Devstudio anymore to find the most
recent version. Not supporting MSVC5 anymore is probably
acceptable.
I never noticed that support for MSVC6 is broken - it works
fine for me... However, if you think you can improve that
somehow, please do - please elaborate what changes solve
what problems, though. It seems that a number of changes are
not strictly necessary (e.g. creation of new functions), to
really evaluate the patch, I have to know why you propose
these changes.
----------------------------------------------------------------------
Comment By: John Anderson (djohnanderson)
Date: 2002-10-07 08:27
Message:
Logged In: YES
user_id=618290
It's been so long since I had a copy of MSVC 5 -- I think it
became obsolete about 6 or 7 years ago. None of my changes
should have any impact on the operation of MSVC 5, but of
course you never know unless you try it. Also, the MSVC 6
support in distutils is currently broken -- although it
finds the compiler, the code to find the include paths is
totally broken. I have MSVC 6 (latest service pack) and 7
and would be willing to make both those work. Anyone who's
using 5 is going to have lots of other problems to deal with
besides distutils. Wouldn't surprise me if the MSVC 6 code
for finding paths would differ in each service pack -- since
it depends upon unsupported registry entries.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-10-07 04:43
Message:
Logged In: YES
user_id=21627
Can you report whether this patch works with MSVC 5?
----------------------------------------------------------------------
Comment By: John Anderson (djohnanderson)
Date: 2002-09-26 14:38
Message:
Logged In: YES
user_id=618290
There's no uploaded file! You have to check the
checkbox labeled "Check to Upload & Attach File"
when you upload a file.
Please try again.
(This is a SourceForge annoyance that we can do
nothing about. :-( )
----------------------------------------------------------------------
Comment By: John Anderson (djohnanderson)
Date: 2002-09-26 14:38
Message:
Logged In: YES
user_id=618290
Opps, I guess I forgot to check that little box. Sorry about
that.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-09-25 21:30
Message:
Logged In: YES
user_id=21627
There's no uploaded file! You have to check the
checkbox labeled "Check to Upload & Attach File"
when you upload a file.
Please try again.
(This is a SourceForge annoyance that we can do
nothing about. :-( )
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=614770&group_id=5470