Mailinglist Archive: opensuse-bugs (12871 mails)

< Previous Next >
[Bug 412895] Tkined (package: scotty) doesn't save file
  • From: bugzilla_noreply@xxxxxxxxxx
  • Date: Wed, 7 Jan 2009 03:24:39 -0700 (MST)
  • Message-id: <20090107102439.2BB4224538F@xxxxxxxxxxxxxxxxxxxxxx>
https://bugzilla.novell.com/show_bug.cgi?id=412895

User marko.veelma@xxxxxxx added comment
https://bugzilla.novell.com/show_bug.cgi?id=412895#c1


Marko Veelma <marko.veelma@xxxxxxx> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC|
|marko.veelma@xxxxxxx




--- Comment #1 from Marko Veelma <marko.veelma@xxxxxxx> 2009-01-07 03:24:38
MST ---
After trying different workarounds I finally found a problem which comes from
Tcl 8.5 changes already since 2004-11-23 (based their ChangeLog). Code from
tclResult.c:

void
Tcl_AppendResultVA(

/*
* Strictly we should call Tcl_GetStringResult(interp) here to make sure
* that interp->result is correct according to the old contract, but that
* makes the performance of much code (e.g. in Tk) absolutely awful. So we
* leave it out; code that really wants interp->result can just insert the
* calls to Tcl_GetStringResult() itself. [Patch 1041072 discussion]
*/

#ifdef USE_DIRECT_INTERP_RESULT_ACCESS
/*
* Ensure that the interp->result is legal so old Tcl 7.* code still
* works. There's still embarrasingly much of it about...
*/

(void) Tcl_GetStringResult(interp);
#endif /* USE_DIRECT_INTERP_RESULT_ACCESS */

It seems that Tcl in old SuSE releases defined
"USE_DIRECT_INTERP_RESULT_ACCESS" and then everything worked like before.

Anyway I don't think it is good idea to enable this workaround and I did update
Tkined package itself with one more patch. After that it does work again with
Tcl 8.5. Only caveat is missing ifdefs and therefore it probably doesn't
compile with old Tcl any more..

Opensuse 11.0 recompiled packages can be found here:
http://www.version6.net/apt/SuSE/11.0-i386/ , patch itself is here:
http://www.version6.net/patches/tkined-tcl85.patch

Cheers,

--
Cougar


--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
< Previous Next >