Logged In: YES
user_id=35033
I checked the CVS version through the CVS browser. The
PACKAGE_ macro's are still there, but only the pyconfig.h.in
is available obviously and it features an #undef there. The
configure will likely convert that to a #define.
Anyway, the _real_ problem is not the PACKAGE_ macro's,
which happen to be identical between the Python and the
ImageMagick applications (where I submitted a similar bug
report). The _real_ problem is that the defines used in
pyconfig.h are very general, like HAVE_TERM or HAVE_THREAD.
It is likely that a package will define those macro's
itself, especially if it is a 'configure'-d package.
These macro's need to be prepended with PY_ for example.
Because most of these macro's are not needed for the include
files (if any at all ofcourse, in which case the entire file
need not be installed), I suggest that only the macro's
needed for install are duplicated in a separate file,
prefixed with PY_ and have the source code check for both
versions of the macro.
Or, alternatively, preprocess the pyconfig.h after creation
and before building (the root Makefile seems a logical
place) and convert it to a new pyconfig.h with the proper
PY_ prefix. And use these macro's exclusively in the python
code.
|