Solaris' term.h requires curses.h to be included first. This causes the configure script to emit lines about a bug in autoconf. From the autoconf mailing lists, their standard response is to fix the configure script, see e.g. http://mail.gnu.org/archive/html/bug-autoconf/2003-05/msg00118.html The following patch against 2.3 branch for configure and configure.in makes things a bit happier. Note that Include/py_curses.h already includes curses.h before term.h, this just fixes the breakage of configure.
Logged In: YES user_id=33168 Should the #if be an #ifdef ? Looks fine to me, but I don't know much about autoconf. :-) I think Martin is the expert. Martin do you have an opinion?
Logged In: YES user_id=21627 I find it confusing that the test for curses.h already refers to HAVE_CURSES_H; I think you should first check for curses.h, and then use HAVE_CURSES_H in the test for term.h I also agree that #ifdef is better than #if, even though it should not matter in an ISO C compiler (which replaces undefined symbols by 0 in an #if).