[Patches] [ python-Patches-555929 ] Cygwin AH_BOTTOM cleanup patch

[noreply@sourceforge.net]

Patches item #555929, was opened at 2002-05-14 15:22
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=555929&group_id=5470

Category: Build
Group: Python 2.3
>Status: Open
Resolution: Accepted
Priority: 5
Submitted By: Jason Tishler (jlt63)
Assigned to: Jason Tishler (jlt63)
Summary: Cygwin AH_BOTTOM cleanup patch

Initial Comment:
This patch complies with the following request found
near the top of configure.in:

# This is for stuff that absolutely must end up in pyconfig.h.
# Please use pyport.h instead, if possible.

I tested this patch under Cygwin, Win32, and Red
Hat Linux.  Python built and ran successfully on
each of these platforms.

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

>Comment By: Martin v. Löwis (loewis)
Date: 2002-05-15 20:25

Message:
Logged In: YES 
user_id=21627

How did you test this patch? When I build Python on Linux, I
get tons of error messages

In file included from /usr/include/limits.h:152,
                 from
/usr/lib/gcc-lib/i486-suse-linux/2.95.3/include/limits.h:117,
                 from
/usr/lib/gcc-lib/i486-suse-linux/2.95.3/include/syslimits.h:7,
                 from
/usr/lib/gcc-lib/i486-suse-linux/2.95.3/include/limits.h:11,
                 from Include/Python.h:28,
                 from Python/frozenmain.c:4:
/usr/include/bits/xopen_lim.h:123: warning: `LONG_BIT' redefined

which come from pyport.h being included before limits.h.

Backing the patch out; try again.


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

Comment By: Jason Tishler (jlt63)
Date: 2002-05-15 13:55

Message:
Logged In: YES 
user_id=86216

Committed as:

configure 1.306
configure.in 1.316
pyconfig.h.in 1.34
Include/Python.h 2.47
Include/pyport.h 2.48

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

Comment By: Martin v. Löwis (loewis)
Date: 2002-05-15 00:03

Message:
Logged In: YES 
user_id=21627

Sounds good; please install this patch.

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

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