Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice
Date
Msg-id 201105052212.p45MC4X12852@momjian.us
Whole thread Raw
In response to Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > Prior to PG 8.2, this was necessary to put the comment on the database,
> > but now that we have the shared comment/description table
> > pg_shdescription, this is not necessary.
> 
> > Do we need createdb to be able to create databases for pre-8.2 clusters?
> > If not, the attached patch fixes the double-prompting.
> 
> Well, if you're only going to change this in HEAD, that might be an
> acceptable limitation, but if you intend to back-patch I think not.
> Older versions of createdb are probably significantly more likely to
> be used with even-older servers.

This code has been that way since pre-8.2 so I see no need to backpatch;
this is the first such complaint I have seen.

> Seems like it wouldn't be that hard to test the server version and only
> reconnect if it's pre-8.2.

I am not excited about adding more code for this so I am thinking
head-only.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice
Next
From: Tom Lane
Date:
Subject: Re: Backpatching of "Teach the regular expression functions to do case-insensitive matching"