[Python-bugs-list] [ python-Bugs-231774 ] Architecture-specific config.h is badly named

[noreply@sourceforge.net]

Bugs item #231774, was updated on 2001-02-09 13:57
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=231774&group_id=5470

Category: Build
Group: Feature Request
Status: Open
Priority: 5
Submitted By: Gilbert Ramirez (gramirez)
Assigned to: Thomas Wouters (twouters)
Summary: Architecture-specific config.h is badly named

Initial Comment:
If Python is installed with architecture-dependent and architecture-independent files in separate locations (using --prefix and --exec-prefix), then config.h is placed in a directory different than Python.h (and pyport.h and pgenheaders.h, which also #include "config.h").

If I build a python module which has its own config.h (like pygtk), the build fails because Python.h #include's the *wrong* config.h. I cannot easily change the order of the
-I flags when compiling pygtk because autoconf places its flags (AM_CPPFLAGS) before any CPPFLAGS that I define in the CPPFLAGS environment variable.

It would be cleaner to follow the glib model which names its installed arch-specific file "glibconfig.h". We could install config.h as pythonconfig.h, and have Python.h #include "pythonconfig.h"



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

>Comment By: Martin v. Löwis (loewis)
Date: 2001-03-25 00:37

Message:
Logged In: YES 
user_id=21627

As for legitimately including config.h: it is normally
included indirectly, as a result of
AC_CHECK_HEADER(Python.h) or the like. configure will add it
the project's config.h, which might confuse the further
configuration process or have arbitrary other strange
effects.

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

Comment By: Martin v. Löwis (loewis)
Date: 2001-03-25 00:31

Message:
Logged In: YES 
user_id=21627

A fix for that problem has been submitted as
http://sourceforge.net/tracker/index.php?func=detail&aid=411138&group_id=5470&atid=305470

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

Comment By: Thomas Wouters (twouters)
Date: 2001-02-15 03:31

Message:
I'll do it, but I don't mind if anyone else does it before me (One free alpha is enough, anyway :-) My main question is: should anybody be (legitimately) including Python's config.h directly ? I was thinking 'py_acconfig.h' myself, for the new autoconf config.h name. 


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

Comment By: Tim Peters (tim_one)
Date: 2001-02-09 16:03

Message:
Assigned to Thomas.  Reassign to someone else if you don't want it!  Guido said he did want it, but didn't bother assigning it to anyone.  Given that we delayed "in dict", we have to find some other way to justify giving you a free alpha release <wink>.


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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-02-09 14:10

Message:
Good idea!  Patch anybody?

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

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