Mailing List Archive: Possible problem with timemodule.c [1.5.2c1]

[I decided not to bug Guido directly with this...]

My compile is completely clean except for Modules/timemodule.c:

./timemodule.c: In function `time_strptime':
./timemodule.c:429: warning: assignment makes pointer from integer without
a cast

This is in time_strptime(), naturally. The code immediately before this
is:

#ifdef HAVE_STRPTIME
/* extern char *strptime(); /* Enable this if it's not declared in <time.h> */

On Linux, strptime() IS declared in <time.h>. However, I find nothing in
timemodule.c that would cause <time.h> to be included for Linux. configure
does find strptime and does cause HAVE_STRPTIME to be defined in config.h.

This is unlikely to be a "showstopper", but I thought I would point it
out. This may simply be a Linux (RedHat 5.2) problem. The more I look at
<time.h>, the more I lean towards thie conclusion. The prototype for
strptime() is not defined unless __USE_XOPEN is defined. The solution,
however, is not obvious.

--
andy dustman | programmer/analyst | comstar communications corporation
telephone: 770.485.6025 / 706.549.7689 | icq: 32922760 | pgp: 0xc72f3f1d

[Andy Dustman wrote]

> [I decided not to bug Guido directly with this...]

Hehe, I scan the newsgroup digests for the string "1.5.2" so I found
your post anyway :-)

> My compile is completely clean except for Modules/timemodule.c:
>
> ./timemodule.c: In function `time_strptime':
> ./timemodule.c:429: warning: assignment makes pointer from integer without
> a cast
>
> This is in time_strptime(), naturally. The code immediately before this
> is:
>
> #ifdef HAVE_STRPTIME
> /* extern char *strptime(); /* Enable this if it's not declared in <time.h> */
>
> On Linux, strptime() IS declared in <time.h>. However, I find nothing in
> timemodule.c that would cause <time.h> to be included for Linux. configure
> does find strptime and does cause HAVE_STRPTIME to be defined in config.h.

Rest assured, <time.h> is included, indirectly, by mytime.h or myselect.h.

> This is unlikely to be a "showstopper", but I thought I would point it
> out. This may simply be a Linux (RedHat 5.2) problem. The more I look at
> <time.h>, the more I lean towards thie conclusion. The prototype for
> strptime() is not defined unless __USE_XOPEN is defined. The solution,
> however, is not obvious.

This analysis sounds right to me. (Can't test it -- the only Linux
box we have here on the network was powered down because it was
overheating. Too much press attention for Open Source I guess :-)

Perhaps __USE_XOPEN could be defined somewhere by the configure
script? Anybody suggest a good spot to do this?

--Guido van Rossum (home page: http://www.python.org/~guido/)