Be paranoid about multiple inclusions (apparently some systems don't like

it for certain header files.)  Be a little bit more careful in setting
up the standard environment in config.h.  This is AIX portability work.

aclocal.m4

@@ -21,8 +21,10 @@ dnl for these particular functions.)
 dnl --------------------------------------------------------------------------
 AC_DEFUN(PA_SIGSETJMP,
 [AC_MSG_CHECKING([for sigsetjmp])
- AC_TRY_LINK(
- [#include <setjmp.h>],
+ AC_TRY_LINK([
+#ifdef HAVE_SETJMP_H
+#include <setjmp.h>
+#endif],
  [sigjmp_buf buf;
   sigsetjmp(buf,1);
   siglongjmp(buf,2);],

config.h

@@ -70,7 +70,24 @@
 #include <unistd.h>
 #endif
 
+#ifdef HAVE_SETJMP_H
 #include <setjmp.h>
+#endif
+
+#ifdef TIME_WITH_SYS_TIME
+#include <sys/time.h>
+#include <time.h>
+#else
+#if HAVE_SYS_TIME_H
+#include <sys/time.h>
+#else
+#include <time.h>
+#endif
+#endif
+
+#include <errno.h>
+#include <signal.h>
+
 #include <sys/socket.h>
 
 /* If we don't have intmax_t, try creating it */

configure.in

@@ -58,16 +58,20 @@ AC_CHECK_HEADERS(stdint.h)
 PA_CHECK_INTTYPES_H_SANE
 AC_CHECK_HEADERS(libgen.h)
 AC_CHECK_HEADERS(memory.h)
+AC_CHECK_HEADERS(setjmp.h)
 AC_CHECK_HEADERS(stddef.h)
 AC_CHECK_HEADERS(stdlib.h)
 AC_CHECK_HEADERS(string.h)
 AC_CHECK_HEADERS(strings.h)
 AC_CHECK_HEADERS(sysexits.h)
+AC_CHECK_HEADERS(time.h)
 AC_CHECK_HEADERS(unistd.h)
 AC_CHECK_HEADERS(sys/filio.h)
 AC_CHECK_HEADERS(sys/stat.h)
+AC_CHECK_HEADERS(sys/time.h)
 AC_CHECK_HEADERS(sys/types.h)
 AC_CHECK_HEADERS(arpa/inet.h)
+AC_HEADER_TIME
 dnl This is needed on some versions of FreeBSD...
 AC_CHECK_HEADERS(machine/param.h)
 

tftp/main.c

@@ -51,23 +51,9 @@ static const char *rcsid UNUSED =
 /*
  * TFTP User Program -- Command Interface.
  */
-#include <sys/types.h>
-#include <sys/socket.h>
 #include <sys/file.h>
-
-#include <netinet/in.h>
-
-#include <arpa/inet.h>
-
 #include <ctype.h>
-#include <errno.h>
 #include <netdb.h>
-#include <setjmp.h>
-#include <signal.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
 #include <fcntl.h>
 #ifdef WITH_READLINE
 #include <readline/readline.h>

tftp/tftp.c

@@ -50,21 +50,6 @@ static const char *rcsid UNUSED =
 /*
  * TFTP User Program -- Protocol Machines
  */
-#include <sys/types.h>
-#include <sys/socket.h>
-#include <sys/time.h>
-
-#include <netinet/in.h>
-
-#include <arpa/tftp.h>
-
-#include <errno.h>
-#include <setjmp.h>
-#include <signal.h>
-#include <stdio.h>
-#include <string.h>
-#include <unistd.h>
-
 #include "extern.h"
 
 extern  struct sockaddr_in peeraddr;	/* filled in by main */

tftp/tftpsubs.c

@@ -56,15 +56,7 @@ static const char *rcsid UNUSED =
 			Jim Guyton 10/85
  */
 
-#include <sys/types.h>
-#include <sys/socket.h>
 #include <sys/ioctl.h>
-#include <sys/time.h>
-#include <netinet/in.h>
-#include <arpa/tftp.h>
-#include <stdio.h>
-#include <string.h>
-#include <unistd.h>
 
 #define PKTSIZE MAX_SEGSIZE+4       /* should be moved to tftp.h */
 

tftpd/misc.c

@@ -17,7 +17,6 @@
 
 #include "config.h"		/* Must be included first! */
 #include <syslog.h>
-#include <signal.h>
 #include "tftpd.h"
 
 /*

tftpd/tftpd.c

@@ -54,28 +54,14 @@ static const char *rcsid UNUSED =
  * This version includes many modifications by Jim Guyton <guyton@rand-unix>
  */
 
-#include <sys/types.h>
 #include <sys/ioctl.h>
-#include <sys/stat.h>
-#include <sys/time.h>
 #include <signal.h>
 #include <fcntl.h>
-
-#include <sys/socket.h>
-#include <netinet/in.h>
-#include <arpa/tftp.h>
 #include <netdb.h>
-
-#include <setjmp.h>
-#include <syslog.h>
-#include <stdio.h>
-#include <errno.h>
 #include <ctype.h>
-#include <string.h>
-#include <stdlib.h>
 #include <pwd.h>
-#include <unistd.h>
 #include <limits.h>
+#include <syslog.h>
 
 #include "tftpsubs.h"
 #include "recvfrom.h"