public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: Thomas Munro <[email protected]>
Cc: Heikki Linnakangas <[email protected]>
Cc: pgsql-hackers <[email protected]>
Subject: Re: Remove support for old realpath() API
Date: Mon, 12 Aug 2024 09:04:23 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+hUKGLpmrr-vQHVWtMe-O=_AvS_CWWRJ+Y=UnGHd3P31S1RbQ@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<CA+hUKGLpmrr-vQHVWtMe-O=_AvS_CWWRJ+Y=UnGHd3P31S1RbQ@mail.gmail.com>

On 12.08.24 08:47, Thomas Munro wrote:
> On Mon, Aug 12, 2024 at 6:18 PM Peter Eisentraut <[email protected]> wrote:
>> On 05.08.24 09:41, Heikki Linnakangas wrote:
>>> On 05/08/2024 09:12, Peter Eisentraut wrote:
>>>> The now preferred way to call realpath() is by passing NULL as the
>>>> second argument and get a malloc'ed result.  We still supported the
>>>> old way of providing our own buffer as a second argument, for some
>>>> platforms that didn't support the new way yet.  Those were only
>>>> Solaris less than version 11 and some older AIX versions (7.1 and
>>>> newer appear to support the new variant).  We don't support those
>>>> platforms versions anymore, so we can remove this extra code.
> 
> I checked this in the AIX 7.3 manual and the POSIX 2008 way does not
> appear to be mentioned there:
> 
> https://www.ibm.com/docs/en/aix/7.3?topic=r-realpath-subroutine
> 
> That's a bit confusing, or maybe there are just too many versioning
> systems to keep track of and I've made a mistake, because it looks
> like AIX 7.2.5+ has actual certification for Unix V7 AKA SUSv4 AKA
> POSIX 2008...  Or maybe the documentation is wrong and it does
> actually work.  I guess the IBM crew will be forced to look into this
> as they continue to work on their PostgreSQL/AIX patch, if it doesn't
> work...

Tom had tested this on and found that it does actually work on AIX 7.1 
and 7.3 but the documentation is wrong.

view thread (7+ messages)  latest in thread

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: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Remove support for old realpath() API
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox