From: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Thomas Munro <thomas.munro@enterprisedb.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Cc: Noah Misch <noah@leadboat.com>
Subject: Re: Three animals fail test-decoding-check on REL_10_STABLE
Date: Sun, 13 Jan 2019 08:28:53 -0500
Message-ID: <859e7c91-7ef4-d4b4-2ca2-8046e0cbee09@2ndQuadrant.com> (raw)
In-Reply-To: <6533.1547319815@sss.pgh.pa.us>
References: <CAEepm=1pEJdwv6DSGmOfpX0EaX7L7sT28c1nXpqvQvmLfEWb1g@mail.gmail.com>
<1302.1547218082@sss.pgh.pa.us>
<7757.1547242261@sss.pgh.pa.us>
<25016.1547249634@sss.pgh.pa.us>
<e552d9b3-7ecf-adf5-e91e-a97dba332ca5@dunslane.net>
<6533.1547319815@sss.pgh.pa.us>
On 1/12/19 2:03 PM, Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
>> On 1/11/19 6:33 PM, Tom Lane wrote:
>>> While I think I've fixed this bug, I'm still quite confused about why
>>> only some buildfarm animals showed the problem.
>> ... Is there something weird about naming of library files on HP-UX?
> Doh! I looked right at this code last night, but it failed to click:
>
> # these files should be present if we've temp_installed everything,
> # and not if we haven't. The represent core, contrib and test_modules.
> return ( (-d $tmp_loc)
> && (-f "$bindir/postgres" || -f "$bindir/postgres.exe")
> && (-f "$libdir/hstore.so" || -f "$libdir/hstore.dll")
> && (-f "$libdir/test_parser.so" || -f "$libdir/test_parser.dll"));
>
> On HPUX (at least the version gaur is running), the extension for
> shared libraries is ".sl" not ".so".
>
> That doesn't explain the failures on damselfly and koreaceratops,
> but they're both running very old buildfarm clients, which most
> likely just don't have the optimization to share a temp-install.
Yes, they are on an older version that doesn't use the NO_TEMP_INSTALL
flag at all.
> I wonder if it's practical to scrape DLSUFFIX out of src/Makefile.port
> instead of listing all the possibilities here. But I'm not sure how
> you'd deal with this bit in Makefile.hpux:
>
> ifeq ($(host_cpu), ia64)
> DLSUFFIX = .so
> else
> DLSUFFIX = .sl
> endif
I'd rather get make to tell us directly, something like:
.PHONY: show_dl_suffix
show_dl_suffix:
@echo $(DLSUFFIX)
I can arrange something like that in the buildfarm code if we think the
use case is too narrow.
> Anyway, the bigger picture here is that the shared-temp-install
> optimization is masking bugs in local "make check" rules. Not
> sure how much we care about that, though. Any such bug is only
> of interest to developers, and it only matters if someone actually
> stumbles over it.
>
>
right.
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: andrew.dunstan@2ndquadrant.com, tgl@sss.pgh.pa.us, andrew@dunslane.net, thomas.munro@enterprisedb.com, noah@leadboat.com
Subject: Re: Three animals fail test-decoding-check on REL_10_STABLE
In-Reply-To: <859e7c91-7ef4-d4b4-2ca2-8046e0cbee09@2ndQuadrant.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox