From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | ljb <lbayuk(at)mindspring(dot)com> |
Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [GENERAL] Re: Re: copy to/from stdout using libpgtcl |
Date: | 2001-09-07 18:25:24 |
Message-ID: | 200109071825.f87IPPp00432@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-patches |
Patch is in the queue. Seems portability was already tested.
> tgl(at)sss(dot)pgh(dot)pa(dot)us wrote:
> >Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> >> Is this safe to put into the main code? It looks fine.
>
> Looks fine to me too, but I suppose you want to know if it works and
> doesn't break anything else...
> The original poster may be able to answer this with more confidence.
>
> >It looks to me like it'll break compatibility with all pre-8.3 Tcls.
> >Need some #ifdef's in there, no?
>
> No, I don't think so. It still uses the older "Tcl v1 channel type"
> structure, not the newer one introduced in Tcl 8.3.2. Should be
> OK back through 8.0, although I base this only on an eyeball-check
> of the Tcl 8.0.5 include files.
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/users-lounge/docs/faq.html
>
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2001-09-07 18:49:59 | Re: What Is The Firing Order? |
Previous Message | mike | 2001-09-07 18:24:58 | Undefined All PQ keywords |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2001-09-07 18:25:47 | Re: Makefile.PL for Pg.so |
Previous Message | Ian Lance Taylor | 2001-09-07 17:00:15 | Re: Support building in a different directory on Solaris |