Re: ecpg doesn't compile (datetime.h/dtime_t)

From: Michael Meskes <meskes(at)postgresql(dot)org>
To: Patrick Welche <prlw1(at)newn(dot)cam(dot)ac(dot)uk>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: ecpg doesn't compile (datetime.h/dtime_t)
Date: 2003-09-29 18:07:34
Message-ID: 20030929180734.GA18591@feivel.fam-meskes.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Sep 29, 2003 at 06:41:48PM +0100, Patrick Welche wrote:
> Today's cvs doesn't compile. I think it is due to
> cvs diff -r1.7 -r1.8 src/interfaces/ecpg/include/datetime.h
> I have dtime_t defined in my sys/types.h. The old version of datetime.h used

Argh, didn't know that some systems have this datatype.

> #define dtime_t timestamp, the new one uses a typedef. Is there actually
> a reason to keep dtime_t, or would just performing the #define, i.e.,
> the following patch be sufficient? It allows me to compile and install
> postgres.. (I don't use ecpg, so I really can't say..)

For ecpg it does not matter which name you use. The typedef is there for
only one reason, namely that Informix uses dtime_t and to be compatible
we have to define that type as well.

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes(at)jabber(dot)org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Meskes 2003-09-29 18:08:01 Re: ecpg doesn't compile (datetime.h/dtime_t)
Previous Message Lamar Owen 2003-09-29 17:59:09 Re: [ADMIN] postgres 6.2 vacuum