From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
cc: Thomas Munro <thomas.munro@enterprisedb.com>,
        Pg Hackers <pgsql-hackers@postgresql.org>,
        Noah Misch <noah@leadboat.com>
Subject: Re: Three animals fail test-decoding-check on REL_10_STABLE
In-reply-to: <e552d9b3-7ecf-adf5-e91e-a97dba332ca5@dunslane.net>
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>
Comments: In-reply-to Andrew Dunstan <andrew@dunslane.net>
	message dated "Sat, 12 Jan 2019 13:34:36 -0500"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <6532.1547319815.1@sss.pgh.pa.us>
Date: Sat, 12 Jan 2019 14:03:35 -0500
Message-ID: <6533.1547319815@sss.pgh.pa.us>
Precedence: bulk

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.

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

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.

			regards, tom lane