Re: Patch to change psql default banner v6 - Mailing list pgsql-patches

From Tom Lane
Subject Re: Patch to change psql default banner v6
Date
Msg-id 10838.1210880781@sss.pgh.pa.us
Whole thread Raw
In response to Re: Patch to change psql default banner v6  (Bruce Momjian <bruce@momjian.us>)
Responses Re: Patch to change psql default banner v6
List pgsql-patches
Bruce Momjian <bruce@momjian.us> writes:
> With major version mismatches it looks like this:

>     $ psql test
>     psql (8.4devel)
>             SSL connection (cipher: 2343, bits: 512)
>             WARNING: Console code page (323) differs from Windows code page (2323)
>                      8-bit characters might not work correctly. See psql reference
>                      page "Notes for Windows users" for details.
>         WARNING: psql version 8.4.0, server version 8.3.1.
>              Some psql features might not work.
>     Type "help" for help.

> By indenting those messages the 'help' message still stands out.

My advice: don't do that, it just looks weird.  Both of these look
fine to me:

$ psql test
psql (8.4devel)
SSL connection (cipher: 2343, bits: 512)
Type "help" for help.

$ psql test
psql (8.4devel)
SSL connection (cipher: 2343, bits: 512)
WARNING: Console code page (323) differs from Windows code page (2323)
         8-bit characters might not work correctly. See psql reference
         page "Notes for Windows users" for details.
WARNING: psql version 8.4.0, server version 8.3.1.
         Some psql features might not work.
Type "help" for help.

Also, maybe it's just me, but I think you have put the order of these
optional things exactly backwards.  I'd do

$ psql test
psql (8.4devel)
WARNING: psql version 8.4.0, server version 8.3.1.
         Some psql features might not work.
WARNING: Console code page (323) differs from Windows code page (2323)
         8-bit characters might not work correctly. See psql reference
         page "Notes for Windows users" for details.
SSL connection (cipher: 2343, bits: 512)
Type "help" for help.

Why?  Well, it's just more nearly the way it used to be.

            regards, tom lane

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Patch to change psql default banner v6
Next
From: Bruce Momjian
Date:
Subject: Re: Patch to change psql default banner v6