| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Grzegorz Szpetkowski <gszpetkowski(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice |
| Date: | 2011-05-05 22:12:04 |
| Message-ID: | 201105052212.p45MC4X12852@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs pgsql-hackers |
Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)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(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Josh Kupershmidt | 2011-05-06 00:56:36 | Re: documentation bug - behave of NEW a OLD in plpgsql's triggers |
| Previous Message | Tom Lane | 2011-05-05 21:34:04 | Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2011-05-05 22:12:17 | Re: Backpatching of "Teach the regular expression functions to do case-insensitive matching" |
| Previous Message | Tom Lane | 2011-05-05 21:34:04 | Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice |