Re: FullTransactionId changes are causing portability issues

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: buildfarm-members(at)lists(dot)postgresql(dot)org, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: FullTransactionId changes are causing portability issues
Date: 2019-05-23 18:05:19
Message-ID: 18538.1558634719@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: buildfarm-members pgsql-hackers

Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2019-05-23 13:46:15 -0400, Tom Lane wrote:
>> I'm not really excited about adopting a position that PG will only
>> build on GCC and clones thereof.

> That's not what I said though? Not supporting one compiler, on an OS
> that's effectively not being developed anymore, with a pretty
> indefensible behaviour, requiring not insignificant work by everyone,
> isn't the same as standardizing on gcc. I mean, we obviously are going
> to continue at the absolute very least gcc, llvm/clang and msvc.

I think you're vastly overstating the case for refusing support for this.
Adding "#ifndef FRONTEND" to relevant headers isn't a huge amount of work
--- it's certainly far less of a problem than the Microsoft-droppings
we've had to put in in so many places. The only real issue in my mind
is the lack of buildfarm support for detecting that we need to do so.

Also relevant here is that you have no evidence for the assumption that
these old Solaris compilers are the only live platform with the problem.
Yeah, we wish our buildfarm covered everything of interest, but it does
not. Maybe, if we get to beta2 without any additional reports of build
failures on beta1, that would be a bit of evidence that nobody else cares
--- but we have no such evidence right now. We certainly can't assume
that any pre-v12 release provides evidence of that, because up till
I retired pademelon, it was forcing us to keep this case supported.

regards, tom lane

In response to

Responses

Browse buildfarm-members by date

  From Date Subject
Next Message Andres Freund 2019-05-23 18:49:08 Re: FullTransactionId changes are causing portability issues
Previous Message Andres Freund 2019-05-23 17:52:31 Re: FullTransactionId changes are causing portability issues

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2019-05-23 18:06:57 Re: fsync failure in durable_unlink ignored in xlog.c?
Previous Message Andres Freund 2019-05-23 17:55:19 Re: fsync failure in durable_unlink ignored in xlog.c?