[pywin32-bugs] [ pywin32-Patches-1727357 ] new VC8.0 build patch for AMD64 branch (inc axdebug)

[SourceForge.net <no...@so...>]

Patches item #1727357, was opened at 2007-05-29 20:13
Message generated for change (Comment added) made by mhammond
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=551956&aid=1727357&group_id=78018

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Steve Yin (steveyin)
Assigned to: Nobody/Anonymous (nobody)
Summary: new VC8.0 build patch for AMD64 branch (inc axdebug)

Initial Comment:
hi mhammond, I have reviewed the patch you committed to cvs, there is still a little bug in cvs. In my previous patch, I totally ignored the axdebug extenstion, because I can not find activdbg.h header in vista sdk.

I found the activdbg.h header exists in the lastest platform sdk, which is build for Windows Server 2003 SP1. So, here comes completed patch for amd64 ;-)

PS. I have to remove the depend of dsound.h, because setup.py just can not find it. and, I also upload a patch for distutils. this patched version can handle the manifest file.

----------------------------------------------------------------------

>Comment By: Mark Hammond (mhammond)
Date: 2007-05-29 23:20

Message:
Logged In: YES 
user_id=14198
Originator: NO

Thanks for the update.  MSDN documents the dwSourceContext params as
DWORD_PTR (rather than DWORDLONG).  So we need to change both the
"interface" and "gateway" side of this to use
PyWinObject_As[/From]DWORD_PTR to handle this param (so it works as well as
compiles ;)  That should also make it possible to remove the #ifdef's.

The change involving IEnumDebugStackFrames64 will also be a problem -
nothing in pywin32 will implement that, so it will fail at runtime.

You included a patch to PyWinTypes.h and win32\* which reverse previous
changes I made.  Was this intentional?  If so, what was the problem you
faced?

If we can sort these out, I'd be happy to apply it!

Thanks - I'll check out the distutils patch tomorrow or the next day...

----------------------------------------------------------------------

Comment By: Steve Yin (steveyin)
Date: 2007-05-29 20:16

Message:
Logged In: YES 
user_id=1613955
Originator: YES

here comes the patch for distutils.

there are some of bugs in pythonwin, maybe it's amd64 related, but I don't
have time to debug it. Sorry.
File Added: distutils-vc8.patch

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=551956&aid=1727357&group_id=78018