Path: ...!weretis.net!feeder9.news.weretis.net!feeder8.news.weretis.net!eternal-september.org!feeder3.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail
From: Alan Grunwald <nospam.nurdglaw@gmail.com>
Newsgroups: comp.lang.tcl
Subject: Re: tcltls v1.7.23 and Tcl 9
Date: Sun, 29 Sep 2024 21:47:05 +0100
Organization: A noiseless patient Spider
Lines: 63
Message-ID: <vdcedt$1sj7o$1@dont-email.me>
References: <vdbhhg$1og55$1@dont-email.me> <vdbkf8$1opca$1@dont-email.me>
 <vdbuo2$1q9sh$1@dont-email.me> <vdc2q7$1qvbs$1@dont-email.me>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Injection-Date: Sun, 29 Sep 2024 22:47:58 +0200 (CEST)
Injection-Info: dont-email.me; posting-host="5b71ffa0c102547d7108ccb32425cdac";
	logging-data="1985784"; mail-complaints-to="abuse@eternal-september.org";	posting-account="U2FsdGVkX1/8ChmUnz92YIReW80si1wRxaSROKUimkE="
User-Agent: Mozilla Thunderbird
Cancel-Lock: sha1:CiwVqNkIdRM07S7IMioS8bU60po=
Content-Language: en-US
In-Reply-To: <vdc2q7$1qvbs$1@dont-email.me>
Bytes: 3442

On 29/09/2024 18:29, Paul Obermeier wrote:
> Am 29.09.2024 um 18:19 schrieb Paul Obermeier:
>> Am 29.09.2024 um 15:24 schrieb Alan Grunwald:
>>> On 29/09/2024 13:33, Alan Grunwald wrote:
>>>> I have encountered a problem with the above extension under tcl9.0.0
>>>>
>>>> After downloading 
>>>> https://www.tcl3d.org/bawt/download/Preview/Tcl9/tcltls-1.7.23.7z, 
>>>> as advertised on the "Porting extensions to Tcl 9" page on the wiki, 
>>>> configuring using the line
>>>>
>>>> ./configure --prefix=/opt/tcl9.0.0 --with-tcl=/opt/tcl9.0.0/lib
>>>>
>>>> building and installing, I cannot load the extension:
>>>>
>>>>      % package require tls
>>>>      version conflict for package "tcl": have 9.0.0, need 8.4
>>>>
>>>> The same error also occurs if I do
>>>>
>>>>      % load /opt/tcl9.0.0/lib/tcltls1.7.23/tcltls.so Tls
>>>>
>>>> so I assume a problem in the .so file. I've had a quick look through 
>>>> the sources, but I have no confidence that I can find the 
>>>> appropriate change.
>>>>
>>>> Please can someone help?
>>>
>>> Maybe I should have more confidence...
>>>
>>> I changed line 1809 of tls.c from
>>>
>>>      Tcl_InitStubs(interp, "Tcl", "8.4", 0)
>>>
>>> to
>>>
>>>      Tcl_InitStubs(interp, "Tcl", "8.4", 0) || Tcl_InitStubs(interp, 
>>> "Tcl", "9", 0)
>>>
>>> and the package now loads ok. Once I've ploughed through several more 
>>> porting issues in my own code, I'll have an idea of whether the 
>>> modified tls works.
>>
>> As long as I was testing with Tcl9 beta versions, the "8.4" in 
>> Tcl_InitStubs or Tk_InitStubs worked.
>> Since using official Tcl9 this does not work anymore, so I switched to 
>> use "8.4-" as is done with "package require" calls.
>>
>> I forgot to update the preview page with the changed packages.
>> Will do today or tomorrow.
>>
>> Paul
> 
> Updated relevant packages at the BAWT preview page.
> Note, that version 1.7.23 is based upon 1.7.22, which is 4 years old.
> You might also check the current trunk of tcltls at 
> https://core.tcl-lang.org/tcltls
> 
> Paul

Thanks Paul. The package seems to work ok with the change I made above, 
your suggestion sounds tidier but on the basis of leaving well alone, 
I'll stick with what I've got at present.