#21 unable to get compiled under RH 7.2

[Anonymous]

The long and short of it is that there is a problem
with the tm structure getting included. I fixed it
by adding the following lines of code:

queue.c

/* added by Chuck Emary to get RH 7.2 compile -
configure in config.h*/
#ifdef _RH72
#include <time.h>
#endif

config.h
/* Added by Chuck Emary to get a compilation on RH
7.2 */
#define _RH72 1

I have not tested this fix yet - so this is just an
FYI for anyone with the same problem.

As an additional note I imported the *.am file into
kDevelop and had no problems.

[Marcio Luis Teixeira]

Logged In: YES
user_id=272639

Do you mean you added those lines to "queued.c" (rather than
"queue.c")? I was having compilation problems (using RH 7.1)
and the error was in "queued.c". Added the lines you
suggested and it works now.
Thanks.

[Martin Purschke]

Logged In: YES
user_id=461830

There is an open bug report for this. The diagnosis is
right, but I'd like to suggest a different fix - remember
that config.h is a generated file, so it gets overwritten
each time you run configure.

I guess the author meant to use the define.h file as the
general include-it-all. Now there is autoconf support for
time.h and sys/time.h in the configure.in file, and we just
need to react properly.

configure defines TM_IN_SYS_TIME is the tm structure is
defined in sys/time.h (which isn't the case on RH7.2).

I changed the start of define.h to read

#ifdef HAVE_SYS_TIME_H

#ifdef TM_IN_SYS_TIME
#include <sys/time.h>
#else
#include <time.h>
#endif

#else
#include <time.h>
#endif

so we include sys/time.h only when configure determines that
we have that file AND that it actually defines the tm
structure, otherwise we include time.h instead.

Then it compiles on RH7.2 and 6.2 as well.

Martin

[Nobody/Anonymous]

Logged In: NO

this seems to be the year old solution to this problem the
"correct" way
http://sourceforge.net/mailarchive/forum.php?thread_id=185664&forum_id=4288