[Patches] [ python-Patches-491107 ] Cygwin setup.py import workaround patch

noreply@sourceforge.net noreply@sourceforge.net
Wed, 12 Dec 2001 06:40:27 -0800


Patches item #491107, was opened at 2001-12-10 03:53
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=491107&group_id=5470

Category: Distutils and setup.py
Group: None
>Status: Closed
>Resolution: Rejected
Priority: 5
Submitted By: Jason Tishler (jlt63)
Assigned to: Michael Hudson (mwh)
Summary: Cygwin setup.py import workaround patch

Initial Comment:
Sorry for submitting this in the 11th hour, but this patch
re-enables clean building under Cygwin.  See the following
for details:

    http://cygwin.com/ml/cygwin/2001-12/msg00409.html

Unfortunately, this patch is only a build workaround and
does *not* solve the root cause which is Cygwin's problem
with DLL address clashes during fork().  Hopefully, a yet to
be instituted rebase tool will solve this problem for real.
See the following for details:

    http://sources.redhat.com/ml/cygwin/2001-12/msg00446.html

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

>Comment By: Michael Hudson (mwh)
Date: 2001-12-12 06:40

Message:
Logged In: YES 
user_id=6656

I'm rejecting this.  Linking _socket statically is a better
workaround until the issue actually gets sorted out at the
cygwin end.

Jason, feel free to complain if you think this isn't the
right thing to do.

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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-12-10 07:34

Message:
Logged In: YES 
user_id=6380

Sure.  While the release candidate is officially scheduled
for Wednesday this weel, I think it'll actually be Friday.

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

Comment By: Michael Hudson (mwh)
Date: 2001-12-10 07:21

Message:
Logged In: YES 
user_id=6656

Well, it lets Python build, but the resulting Python doesn't
work all that well.

I've just noticed that linking _socket statically seems to
cure the problem.

Can we have a few more days to fiddle with this?  I wouldn't
recommend applying this patch at this stage.

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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-12-10 07:08

Message:
Logged In: YES 
user_id=6380

Michael, can you review this ASAP? If not, please assign to
Tim.

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

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=491107&group_id=5470