Re: PostGIS vs. PGXS in 9.0beta3

Tom Lane wrote:
> Josh Berkus <josh(at)agliodbs(dot)com> writes:
>
>> A 9.0b3 tester reported this issue with our single most popular
>> PostgreSQL extension, PostGIS:
>>
>
>
>> ==================
>> Postgis makes use of 'PGXS' in postgresql > 8.2. Within postgresql-9,
>> datadir and many other variables are defined as multiple values with an
>> append operator, like this:
>>
>
>
>> $ grep -i datadir /usr/pgsql-9.0/lib/pgxs/src/Makefile.global
>> [snip]
>> datadir := /usr/pgsql-9.0/share
>>
>
> This analysis is nonsense on its face --- := is not an append operator
> and we do not have any multiple values for datadir.
>
> The referenced postgis-users thread seems to indicate that the postgis
> guys found and fixed a problem in their own makefiles. If not, we need
> a clearer description of what they think the problem is.
>

The real problem has nothing to do with any of the analysis, as you say.
It is this: they have an override file for PGXS and it uses
$(mkinstalldirs) which we got rid of about a year ago. So apparently
they haven't been testing much against any of our alphas or betas or
they would have seen this long ago. The correct fix is to do the
following in the PostGIS source root:

sed -i -e 's/mkinstalldirs/MKDIR_P/' postgis/Makefile.pgxs

cheers

andrew