[Patches] [ python-Patches-411138 ] Fix for #231774: pyconfig.h

noreply@sourceforge.net noreply@sourceforge.net
Wed, 11 Apr 2001 05:40:53 -0700


Patches item #411138, was updated on 2001-03-25 00:30
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=411138&group_id=5470

Category: Build
Group: None
Status: Open
Priority: 5
Submitted By: Martin v. Löwis (loewis)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix for #231774: pyconfig.h

Initial Comment:
This patch renames config.h to pyconfig.h, to allow
autoconfiscation of python-based projects.

In addition to the changes collected in this patch, the
following actions are required when committing it:
- re-run autoheader and autoconf
- cvs add pyconfig.h (generated by autoheader), cvs
remove config.h
- rename  PC/config.h, PC/os2vacpp/config.h, and
RISCOS/config.h in CVS (probably using cvs add/remove)
- globally replace config.h in PC/os2vacpp/makefile and
PC/os2vacpp/makefile.omk; this change has been omitted
from the patch because it is quite large.



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

>Comment By: Thomas Wouters (twouters)
Date: 2001-04-11 05:40

Message:
Logged In: YES 
user_id=34209

I may be biased, but I don't see the difference between this
patch and my patch, except that there are some errors in
this patch :) It includes a change in fileobject.h that
isn't supposed to be there (I think), and it changes
distutils in the wrong way (distutils, for backward
compatibility, should check for both config.h and
pyconfig.h) -- though my patch doesn't address the latter
issue, it doesn't touch distutils at all.

Documentation should also mention both config.h and
pyconfig.h. Other than that, I don't care which patch gets
accepted, by the way :) But maybe Martin can explain why he
created this patch ?




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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-04-10 15:11

Message:
Logged In: YES 
user_id=6380

PS. Does this supersede patch #403978?

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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-04-10 15:09

Message:
Logged In: YES 
user_id=6380

Let's do this after 2.1.

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

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