Re: split postmaster's checkDataDir to src/common

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: split postmaster's checkDataDir to src/common
Date: 2013-08-28 02:38:29
Message-ID: 31712.1377657509@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> Here's a patch to move src/port/pgcheckdir.c to src/common/checkdir.c
> and add the shareable part of postmaster's current checkDataDir code
> into it, as pg_check_dir_permissions. This is in the spirit of using it
> in pgstat to check the directory defined by stats_temp_directory.

> (This is basically the same patch I posted earlier, except the file is
> now in src/common instead of src/port.)

Okay for HEAD, but ...

> Barring objections I intend to push this to 9.3 and HEAD shortly.

What exactly is the argument for pushing this into 9.3? Since we
are past rc1, we should treat that branch as released. If you wouldn't
back-patch into all supported branches, you shouldn't be patching 9.3
either.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2013-08-28 02:57:37 Re: Deprecating RULES
Previous Message Sawada Masahiko 2013-08-28 01:28:04 Re: Behaviour of take over the synchronous replication