Re: TCL fix in HEAD

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Marshall, Steve" <smarshall(at)wsi(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: TCL fix in HEAD
Date: 2007-10-01 16:45:14
Message-ID: 15246.1191257114@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

"Marshall, Steve" <smarshall(at)wsi(dot)com> writes:
> Bruce Momjian wrote:
>> Can you send in a patch against pltcl.sgml?
>>
> PL/TCL documentation patch is attached.

This seems a bit wordy, as well as wrongly placed --- the time to tell
people they need a non-multithreaded Tcl is in the installation
instructions. I added the following instead.

regards, tom lane

Index: installation.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/installation.sgml,v
retrieving revision 1.292
diff -c -r1.292 installation.sgml
*** installation.sgml 25 Aug 2007 20:29:25 -0000 1.292
--- installation.sgml 1 Oct 2007 16:42:50 -0000
***************
*** 228,234 ****
<listitem>
<para>
If you want to build the <application>PL/Tcl</application>
! procedural language, you of course need a Tcl installation.
</para>
</listitem>

--- 228,237 ----
<listitem>
<para>
If you want to build the <application>PL/Tcl</application>
! procedural language, you of course need a <productname>Tcl</>
! installation. If you are using a pre-8.4 release of
! <productname>Tcl</>, ensure that it was built without multithreading
! support.
</para>
</listitem>

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Marshall, Steve 2007-10-01 17:26:26 Re: TCL fix in HEAD
Previous Message Richard Rowell 2007-10-01 16:17:14 Patch to inline stable SQL set returning UDFs