Re: New tcl-8.5 and 8Kingdoms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Wart wrote:
Hans de Goede wrote:
Michael Thomas wrote:
Did you have to make any changes to 8Kingdoms to get it to build with
Tcl 8.5?  If so please pass along any patches.  I'll try building and
running it myself to see if I can track down the problem.

Nope,

No changes, I also did a diff between the buildlogs, no new compiler
warnings or anything like that.

I did build it with gcc-4.3, as I was working on it not building with
gcc-4.3 too.

I've just committed my latest 8Kingdoms work to cvs, so you can get it
from there, including a patch for the crash you reported (tested using
tcl 8.4).

It looks like it's a result of the stack checking code in the Tcl
library itself.  Turning off the stack checking code by compiling Tcl
with -DTCL_NO_STACK_CHECK=1 makes the problem in 8Kingdoms go away.

With the stack checking turned on, the Tcl scripts in 8Kingdoms fail
with the error "out of stack space (infinite loop?)".  This error is
generated by the stack checking code in tclBasic.c line 3439.

I don't understand enough of this stack checking stuff to debug it any
further, unfortunately.


And here is another program which is known to brake with stack checking:
http://wiki.tcl.tk/20153

Quoting the author:
"The stack checking is somewhat of a hack IMO anyway"
"Unfortunately as long as the broken stack checking remains in Tcl, this must require compiling Tcl with -DTCL_NO_STACK_CHECK."

So maybe it would be best to just disable it on Fedora?

Regards,

Hans

--
fedora-devel-list mailing list
fedora-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-devel-list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux