On Wed, Oct 21, 2009 at 15:34, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
> Magnus Hagander wrote:
>> From a quick look, it looks fine to me. I don't have time to do a
>> complete check right now, but I'll do that as soon as I can and then
>> commit it - unless people feel it's more urgent than maybe a week
>> worst case, in which case someone else has to pick it up :-)
> I'd rather wait till you can check it.
Apologies for the delay.
I've finally looked it over, and it still looks fine. I've cleaned up
some of the whitespace issues in the patch and applied it, including a
backpatch to 8.3 and a forward-patch to HEAD.
Thanks for the patch!
In response to
pgsql-hackers by date
|Next:||From: Tom Lane||Date: 2009-11-14 16:25:53|
|Subject: Re: [patch] pg_ctl init extension |
|Previous:||From: Kevin Grittner||Date: 2009-11-14 15:12:23|
|Subject: Re: [patch] pg_ctl init extension|
pgsql-bugs by date
|Next:||From: Tom Lane||Date: 2009-11-14 16:26:50|
|Subject: Re: Re: BUG #5065: pg_ctl start fails as administrator, with "could not locate matching postgres executable" |
|Previous:||From: Craig Ringer||Date: 2009-11-14 05:20:35|
|Subject: Re: Crazy query plan.|