Re: pgxs regression

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Joe Conway <mail(at)joeconway(dot)com>
Cc: Thomas Hallgren <thhal(at)mailblocks(dot)com>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, "Hackers (PostgreSQL)" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgxs regression
Date: 2004-11-20 18:59:18
Message-ID: 24136.1100977158@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Joe Conway <mail(at)joeconway(dot)com> writes:
> I'll go take a look at the thread (haven't yet) but as of the moment
> this is not fixed in cvs. Here are two examples from contrib (in each
> case I inserted "USE_PGXS = 1" into the Makefile):

> # cd contrib/dblink/
> # vi Makefile
> # make
> make: *** No rule to make target
> `/usr/local/pgsql-dev/lib/pgxs/src/makefiles/../../src/port/pg_config_paths.h',
> needed by `all-static-lib'. Stop.

My inclination is to get rid of the dependency of the Makefile.shlib
targets on pg_config_paths.h; Bruce has never satisfactorily explained
why he added that. There may be a need for a dependency on
pg_config_paths.h someplace, but surely this is the wrong place.

I don't understand what $(PTHREAD_H_WIN32) is doing there either --- the
only definition of it that I can see is
PTHREAD_H_WIN32 = yes
in libpq/Makefile, which does not look like a build target. (The
comment just above that saying "this doesn't work yet" doesn't increase
my faith any...)

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Stark 2004-11-20 19:10:34 Re: Error handling in plperl and pltcl
Previous Message Joe Conway 2004-11-20 18:57:44 cvs web errors