Re: pgsql: Use better comment marker in Autoconf input

From: Thomas Munro <munro(at)ip9(dot)org>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Use better comment marker in Autoconf input
Date: 2019-02-11 07:41:03
Message-ID: CADLWmXXncVaDhBWJ0U16b+NtkPeYRR+Ntb=h3bkB+d3yZxUExA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Sun, 10 Feb 2019 at 19:22, Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2019-02-10 01:45:58 -0500, Tom Lane wrote:
> > Andres Freund <andres(at)anarazel(dot)de> writes:
> > > On 2019-02-09 14:57:27 +0000, Peter Eisentraut wrote:
> > > I'd for a minute just put this down
> > > to an independent upgrade on the animal, but the other branches still
> > > build fine.
> >
> > The error seems to be a library version mismatch, which sure looks
> > like a busted software upgrade. But you have a darn good point:
> > if that's the problem, why didn't v11 break too?
>
> After a few coffees it's clear why: In v11, we have jit=0. As LLVM is
> loaded on-demand, we'll not see errors in v11.

It was using CC="ccache cc" for C, but no explicit CXX. Somehow our
configure script flipped from finding c++ (the system C++ compiler,
which is Clang) to finding g++ (an old GCC port I had installed) after
that commit, and the resulting .so didn't work too well. I did not
upgrade or change anything at that time. I must admit that commit of
Peter's looks fairly innocent though. Is it possible that an earlier
change caused it, but this change caused the "accache" to be cleared?
That accache seems a bit flimsy, I've had to rm it before. Anyway,
now that I've explicitly set CXX="ccache c++" and nuked the accache,
elver is green again.

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2019-02-11 09:08:12 pgsql: Remove unused macro
Previous Message Tom Lane 2019-02-11 03:51:48 pgsql: Fix indexable-row-comparison logic to account for covering index