#21 unable to get compiled under RH 7.2

open
nobody
None
5
2001-11-09
2001-11-09
Anonymous
No

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.

Discussion

  • 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

    Martin Purschke - 2002-02-18

    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

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks