[Bug-gnulib] Re: new module: time_r; plus fixes for timegm, mktime, strftime

[Simon Josefsson]

Paul Eggert <address@hidden> writes:

> Simon Josefsson <address@hidden> writes:
>
>> I had some troubles applying the patch, but when it is installed I can
>> test it too on a few platforms.
>
> OK, I tested it a bit more, fixed some minor comments and dependency
> problems in the modules/* files, and installed it.  Please let me know
> if you run into problems.

On hppa2.0w-hp-hpux11.11 and ia64-hp-hpux11.22 I get the following.
Let me know if you want config.log output.  (It works fine on ~15
other platforms.)

/bin/sh ../libtool --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I..      -g -c 
-o time_r.lo `test -f 'time_r.c' || echo './'`time_r.c
 gcc -DHAVE_CONFIG_H -I. -I. -I.. -g -c time_r.c  -fPIC -DPIC -o .libs/time_r.o
time_r.c: In function `rpl_gmtime_r':
time_r.c:62: too few arguments to function `copy_tm_result'
time_r.c: In function `rpl_localtime_r':
time_r.c:68: too few arguments to function `copy_tm_result'
*** Error exit code 1

bash-2.04$ grep -i time config.h
/* Define to 1 if you have the declaration of `timegm', and to 0 if you don't.
#define HAVE_DECL_TIMEGM 0
/* Define to 1 if you have the `strftime' function. */
/* #undef HAVE_STRFTIME */
/* Define to 1 if you have the <sys/time.h> header file. */
#define HAVE_SYS_TIME_H 1
/* Define to 1 if you have the `timegm' function. */
/* #undef HAVE_TIMEGM */
/* Define to 1 if you have the <time.h> header file. */
#define HAVE_TIME_H 1
/* Define to 1 if localtime_r, etc. have the type signatures that POSIX
/* #undef HAVE_TIME_R_POSIX */
   automatically deduced at run-time.
/* Define to 1 if you can safely include both <sys/time.h> and <time.h>. */
#define TIME_WITH_SYS_TIME 1
/* Define to 1 if your <sys/time.h> declares `struct tm'. */
/* #undef TM_IN_SYS_TIME */
/* Define to rpl_mktime if the replacement function should be used. */
#define mktime rpl_mktime
bash-2.04$