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: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, 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-10 23:44:35
Message-ID: 201105102344.p4ANiZT03175@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Bruce Momjian wrote:
> 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.

Attached patch applied to head only.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

Attachment Content-Type Size
/rtmp/diff text/x-diff 1.9 KB

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2011-05-11 00:57:05 Re: Changed behaviour of \'
Previous Message Skylar Hawk 2011-05-10 22:36:32 BUG #6020: Wrong data type returned after CAST in FROM

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Stark 2011-05-10 23:54:45 Re: the big picture for index-only scans
Previous Message Kevin Grittner 2011-05-10 23:14:40 Re: the big picture for index-only scans