| From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> | 
| Subject: | Re: Quoting of psql \d output | 
| Date: | 2003-12-23 15:00:42 | 
| Message-ID: | 200312231500.hBNF0gW02709@candle.pha.pa.us | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches | 
Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > psql \d always double-quotes table names:
> > 	      Table "public.xx"
> 
> Yeah, that has bugged me for a while, because it is in fact *wrong* ...
> the above representation of a qualified name is incorrect.
> > This patch uses pg_dump fmtId() to double-quote only when necessary.
> 
> Seems good, but I think your Makefile patch is a brick or two shy of
> a load.  Don't you need something to make sure that parser/keywords.o
> is up to date?
OK, got it. I didn't notice the submake-backend rule before. Attached.
> More generally, maybe we ought to think about moving fmtId() and perhaps
> other parts of dumputils out of pg_dump and into some
> more-easily-accessible library.
I will return to that.  Right now we have been putting code shared by
the backend and /bin into pgport, but I don't think that is going to be
clean for things shared by just /bin, and we have the keywords.c
dependency, making this even more complex.
-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
| Attachment | Content-Type | Size | 
|---|---|---|
| unknown_filename | text/plain | 7.3 KB | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Christopher Kings-Lynne | 2003-12-23 15:47:42 | Re: [GENERAL] Temporary tables and miscellaneous schemas | 
| Previous Message | Tom Lane | 2003-12-23 15:00:40 | Re: Quoting of psql \d output | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Christopher Kings-Lynne | 2003-12-23 15:47:42 | Re: [GENERAL] Temporary tables and miscellaneous schemas | 
| Previous Message | Tom Lane | 2003-12-23 15:00:40 | Re: Quoting of psql \d output |