Re: Re: [BUGS] BUG #5957: createdb with description and md5 auth forces to provide password twice

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: Raw Message | Whole Thread | 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. +

In response to

Responses

Browse pgsql-bugs by date

  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

Browse pgsql-hackers by date

  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