Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal - Mailing list pgsql-patches

From Reinhard Max
Subject Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal
Date
Msg-id Pine.LNX.4.33.0109061320080.21982-100000@wotan.suse.de
Whole thread Raw
In response to Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal  (Vsevolod Lobko <seva@sevasoft.kiev.ua>)
Responses Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal
List pgsql-patches
On Thu, 6 Sep 2001, Vsevolod Lobko wrote:

> On Thu, 6 Sep 2001, Reinhard Max wrote:
>
> > So this seems to be the most clean and consistent solution:
> >
> > - PostgreSQL requires to be compiled with --enable-multibyte
> >   and --enable-unicode-convertion if it ought to work correctly
> >   with Tcl/Tk >= 8.1 (client or server side).
> >
> > - PL/Tcl needs to be changed to use pg_do_encoding_conversion
> >   if it runs on a Tcl version >= 8.1 .
> >
> > - PgTcl needs to be changed to set it's cliend encoding to UNICODE
> >   if it runs on a Tcl version >= 8.1 .
> >
> > Can we agree on that?
>
> yes

Great!

> I'll do pl/tcl part in the next version of patch. Using this
> approach we can eliminate overhead for databases in UNICODE.
>
> Can somebody add tcl version checking to configure.in so we can do
> that all on configure stage?

Hmm, if we check the Tcl version at runtime instead, we could have
PL/Tcl and PgTcl libs that don't depend on the Tcl version they run
on. So one binary could possibly be used for all Tcl versions from 7.6
up to 8.4 .

The only useful change to configure.in I could think of is to issue a
warning if configure is called without --enable-multibyte and
--enable-unicode-convertion , but the available Tcl is >= 8.1 .

cu
    Reinhard


pgsql-patches by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: NLS Stuff
Next
From: Vsevolod Lobko
Date:
Subject: Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal