Re: [GENERAL] Re: Re: copy to/from stdout using libpgtcl

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

In response to

Browse pgsql-general by date

  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

Browse pgsql-patches by date

  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