Re: [RFA] Build failure fix: Python troubles on non-python targets...

Tom Tromey
2009-05-28T15:38:54Z

>>>>> "Pierre" == Pierre Muller <muller@ics.u-strasbg.fr> writes:

Pierre> On DJGPP,
Pierre> I now get this failure:
[...]

Sorry about that.

Pierre> The following two changes fixes compilation for me:
Pierre> Is this patch OK?

Nearly...

Pierre> +
Pierre> +#ifdef HAVE_PYTHON
Pierre> +
Pierre> #include "python.h"
Pierre> #include "python-internal.h"

This should unconditionally include python.h. So, the conditional
needs to be moved down a few lines. It is ok to include python.h even
if Python is not available; and in this case it is desirable so that
the declarations are picked up (and thus checked against the
definitions).

Ok with that change. Thanks.

Tom