Re: pgAgent: C++ Port - Patch Review - Mailing list pgadmin-hackers

From Linreg
Subject Re: pgAgent: C++ Port - Patch Review
Date
Msg-id 1566641.1Qb8IbrB6c@wolfclan.ang.de
Whole thread Raw
In response to Re: pgAgent: C++ Port - Patch Review  (Neel Patel <neel.patel@enterprisedb.com>)
List pgadmin-hackers

Hi,

 

I'll change it. (without prefix!)

 

Thomas

 

> > > 1.)

> > >

> > > it is system dependent. On my SUSE system pibpg-header to be installed

> >

> > under

> >

> > > the pgsql directory.

> > >

> > > Example: /usr/include/pgsql/libpq-fe.h

> >

> > What did the previous code do? That worked. Typically we would not

> > expect to see a pgsql/ prefix on a header, but would ensure the

> > compiler is passed the include directory with that directory in it.

>

> Previous code is without pgsql/ prefix on a header. So i think we should

> have only "#include <libpg-fe.h>".

>

> > --

> > Dave Page

> > Blog: http://pgsnake.blogspot.com

> > Twitter: @pgsnake

> >

> > EnterpriseDB UK: http://www.enterprisedb.com

> > The Enterprise PostgreSQL Company

 

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: PATCH: Improvement in the debugger support for EnterpriseDB <= 9.2
Next
From: Neel Patel
Date:
Subject: Re: pgAgent: C++ Port - Patch Review