Re: (forw) (forw) Re: [HACKERS] PHP 4.0.4pl1 BUILD: BUSTED WITH CURRENT CVS

[Larry Rosenman]

* Tom Lane <[EMAIL PROTECTED]> [010218 16:54]:
> Larry Rosenman <[EMAIL PROTECTED]> writes:
> > OK, I found it.  PHP was including postgres.h (which we no longer
> > install...., so we were picking up a Feb 7 version). 
> > Changing php's ext/pgsql/php_pgsql.h to #include <postgres_fe.h>
> > fixes it. 
> 
> Hm.  Should php be including either one?  I would have been in less
> hurry to invent a new file if I had thought that client apps were
> including postgres.h ...
> 
Interestingly, leaving out postgres_fe.h works as well. 

I'll update my bug report w/php to delete that line altogether. 

>                       regards, tom lane
-- 
Larry Rosenman                     http://www.lerctr.org/~ler
Phone: +1 972-414-9812                 E-Mail: [EMAIL PROTECTED]
US Mail: 1905 Steamboat Springs Drive, Garland, TX 75044-6749