postgres.h missing

Started by PostgreSQL Bugs Listalmost 25 years ago3 messagesbugs
Jump to latest
#1PostgreSQL Bugs List
pgsql-bugs@postgresql.org

Darren Cook (darren@flyingcolor.com) reports a bug with a severity of 3
The lower the number the more severe it is.

Short Description
postgres.h missing

Long Description
include/postgres.h is no longer there. php4 tries to include it, so now fails to compile.

postgres_fe.h says:
* This should be the first file included by PostgreSQL client libraries and
* application programs --- but not by backend modules, which should include
* postgres.h.

...which sounds like postgres.h should be there. I created a dummy postgres.h which just includes postgres_fe.h and php4 now compiles.

(BTW php4 is patched in CVS to include postgres_fe.h instead of postgres.h).

Sample Code

No file was uploaded with this report

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: PostgreSQL Bugs List (#1)
Re: postgres.h missing

pgsql-bugs@postgresql.org writes:

(BTW php4 is patched in CVS to include postgres_fe.h instead of postgres.h).

PHP *should* have been patched to not include either one. If they
didn't get it right yet, would you rattle their cage a little more?

regards, tom lane

#3Darren Cook
darren@flyingcolor.com
In reply to: Tom Lane (#2)
Re: postgres.h missing

pgsql-bugs@postgresql.org writes:

(BTW php4 is patched in CVS to include postgres_fe.h instead of

postgres.h).

PHP *should* have been patched to not include either one. If they
didn't get it right yet, would you rattle their cage a little more?

Sorry to confuse, I got that from the mailing list archives, not the actual
CVS; further on in the thread it says that neither are needed:
http://www.phpbuilder.com/mail/php-developer-list/2001022/0136.php

(the latest php4 release does not have this patch, so brace yourself as
more people upgrade to 7.1).

Darren