pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
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





view thread (10+ messages)  latest in thread

Message-ID: <859e7c91-7ef4-d4b4-2ca2-8046e0cbee09@2ndQuadrant.com>
Permalink:  ../859e7c91-7ef4-d4b4-2ca2-8046e0cbee09@2ndQuadrant.com/
Also on:    postgresql.org/message-id/859e7c91-7ef4-d4b4-2ca2-8046e0cbee09@2ndQuadrant.com

 · 

reply

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