Re: WIP: psql default banner patch

From: "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>
To: "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: WIP: psql default banner patch
Date: 2008-04-22 23:38:38
Message-ID: 20080422163838.37512649@commandprompt.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, 22 Apr 2008 16:28:20 -0700
"Joshua D. Drake" <jd(at)commandprompt(dot)com> wrote:

> Why do we care, if the version matches? Not that I am feeling like
> fighting about it but it seems just a waste of bytes. It makes sense
> if the version doesn't match.
>
> Joshua D. Drake

Actually in thinking about this, this is what the patch does now:

If version matches:

jd(at)scratch:~/pgsql/src/bin/psql$ ./psql -U jd -h localhost -p 9000
postgres Type: help for help.
postgres=#

If version doesn't match:

jd(at)scratch:~/pgsql/src/bin/psql$ ./psql -U jd -h localhost postgres

WARNING: Server 8.2, psql is version 8.4. Some psql features may not
work.

Type: help for help.
postgres=>

I understand Alvaro's point on the translation so no sweat, I will fix
that. What if I add the version to the help? With this patch if you
type help you get:

postgres=> help

You are using psql, the command-line interface to PostgreSQL.

For SQL help type \h or \help .
For help using psql type \? .
To quit psql type \q .

To view the copyright type \copyright .

postgres=>

I could change that to:

You are using psql 8.2.7, the command-line interface to PostgreSQL.

For SQL help type \h or \help .
For help using psql type \? .
To quit psql type \q .

To view the copyright type \copyright .

Does that make sense?

Sincerely,

Joshua D. Drake

--
The PostgreSQL Company since 1997: http://www.commandprompt.com/
PostgreSQL Community Conference: http://www.postgresqlconference.org/
United States PostgreSQL Association: http://www.postgresql.us/
Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2008-04-22 23:39:38 Re: Per-table random_page_cost for tables that we know are always cached
Previous Message Joshua D. Drake 2008-04-22 23:28:20 Re: WIP: psql default banner patch