From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Make hstore_plperl's build even more like plperl's |
Date: | 2015-05-02 16:15:27 |
Message-ID: | 5544F81F.8010709@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 05/02/2015 11:05 AM, Andrew Dunstan wrote:
>
> On 05/02/2015 10:40 AM, Peter Eisentraut wrote:
>> On 5/2/15 10:20 AM, Andrew Dunstan wrote:
>>> Putting the CORE directory at the end of PG_CPPFLAGS doesn't work.
>>> Using
>>> the override to put it at the end of CPPFLAGS does work.
>> It's strange that the include directory order should matter. What is
>> the explanation for that? I don't see this documented anywhere. I
>> don't see any evidence in the buildfarm that this makes a difference.
>>
>> The intent of my changes was to unbreak other platforms that had been
>> broken by these Windows-related changes (which did succeed).
>>
>>
>
> What failures on other platforms? There's not a single buildfarm
> failure I can see attributable to this. All the recent failures are
> listed here: <http://www.pgbuildfarm.org/cgi-bin/show_failures.pl>
>
> The evidence in favor of my changes is in my testing. When I do it
> your way I get errors, when I do it my way I don't.
>
> Note that the override is EXACTLY what is done unconditionally in the
> plperl GNUmakefile:
>
> override CPPFLAGS := -I. -I$(srcdir) $(CPPFLAGS)
> -I$(perl_archlibexp)/CORE
>
>
> So, contrary to your claims, you have not made this makefile more like
> plperl's - you've made it less like plperl's, and detrimentally so.
FYI, the attached patch allows jacana to build and test hstore_plperl
without error.
cheers
andrew
Attachment | Content-Type | Size |
---|---|---|
hstore_plperl-fix.patch | text/x-patch | 932 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2015-05-02 17:02:29 | Re: pgsql: Make hstore_plperl's build even more like plperl's |
Previous Message | Andrew Dunstan | 2015-05-02 15:09:05 | Re: pgsql: Move interpreter shared library detection to configure |