Skip site navigation (1) Skip section navigation (2)

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 (view raw or flat)
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

pgsql-patches by date

Next:From: Tom LaneDate: 2007-09-28 23:13:32
Subject: Re: TCL fix in HEAD
Previous:From: Bruce MomjianDate: 2007-09-28 22:33:27
Subject: Re: TCL fix in HEAD

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group