Re: TCL fix in HEAD

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: "Marshall, Steve" <smarshall(at)wsi(dot)com>
Cc: PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: TCL fix in HEAD
Date: 2007-09-28 22:43:46
Message-ID: 200709282243.l8SMhk811818@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Marshall, Steve wrote:
> >The recent TCL patch assumed Tcl_NotifierProcs.initNotifierProc
> >was added in TCL 8.2:
> >In fact it was added in 8.4 so I have modified the CVS with
> >the following patch.
>
> I confirmed this against the 8.2.5 release. Sorry I did not notice that
> when I initially created the patch.
>
> While the patch will not work for the earlier releases, it should be
> noted that the multithhreading problem still exists when linking
> postgresql against TCL 8.2 and 8.3 libraries that are compiled with the
> preprocessor symbol TCL_THREADS defined. Unfortunately, we cannot
> override the initNotifier behavior in those releases, so we don't have a
> workable solution.
>
> This is probably not a big problem, since TCL was not commonly compiled
> with multithreading enabled prior to 8.4. However, perhaps there should
> be a warning in the documentation on PL/TCL directing users to avoid
> linking postgresql against TCL libraries earlier than 8.4 that have
> multithreading enabled?

Can you send in a patch against pltcl.sgml?

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2007-09-28 23:13:32 Re: TCL fix in HEAD
Previous Message Bruce Momjian 2007-09-28 22:33:27 Re: TCL fix in HEAD