Re: TCL fix in HEAD - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: TCL fix in HEAD
Date
Msg-id 200709282243.l8SMhk811818@momjian.us
Whole thread Raw
In response to Re: TCL fix in HEAD  ("Marshall, Steve" <smarshall@wsi.com>)
Responses Re: TCL fix in HEAD  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: TCL fix in HEAD  ("Marshall, Steve" <smarshall@wsi.com>)
List 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@momjian.us>        http://momjian.us
  EnterpriseDB                             http://postgres.enterprisedb.com

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

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: TCL fix in HEAD
Next
From: Tom Lane
Date:
Subject: Re: TCL fix in HEAD