Date: Sat, 1 Jun 2013 18:01:45 +0100 From: Chris Rees <crees@FreeBSD.org> To: Dimitry Andric <dim@freebsd.org> Cc: tcltk@freebsd.org Subject: Re: patch: increase tcl86 regular expression depth limit to help dejagnu Message-ID: <CADLo839m8-xHgzVs1H2G%2BbZ=_G1RVLdebneCcaHExdpKyM52Fw@mail.gmail.com> In-Reply-To: <DF38771A-FFC5-4E92-B95A-24F13AB331B1@FreeBSD.org> References: <DF38771A-FFC5-4E92-B95A-24F13AB331B1@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 1 June 2013 17:35, Dimitry Andric <dim@freebsd.org> wrote:
> Hi,
>
> When running the libffi test suite using dejagnu, I ran into the
> following tcl86 error:
>
> ...
> Running target unix
> Using /usr/local/share/dejagnu/baseboards/unix.exp as board description file for target.
> Using /usr/local/share/dejagnu/config/unix.exp as generic interface file for target.
> Using ../../testsuite/config/default.exp as tool-and-target-specific interface file.
> Running ../../testsuite/libffi.call/call.exp ...
> ERROR: tcl error sourcing ../../testsuite/libffi.call/call.exp.
> ERROR: couldn't compile regular expression pattern: out of memory
> while executing
> "regexp $texttmp ${output}"
> (procedure "saved-dg-test" line 247)
> invoked from within
> "saved-dg-test ../../testsuite/libffi.call/huge_struct.c {-O0 -W -Wall} {}"
> ("eval" body line 1)
> invoked from within
> "eval saved-dg-test $args "
> (procedure "dg-test" line 6)
> invoked from within
> "dg-test $testcase $flags ${default-extra-flags}"
> (procedure "dg-runtest" line 10)
> invoked from within
> "dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*.\[cS\]]] "-O0 -W -Wall" """
> (file "../../testsuite/libffi.call/call.exp" line 24)
> invoked from within
> "source ../../testsuite/libffi.call/call.exp"
> ("uplevel" body line 1)
> invoked from within
> "uplevel #0 source ../../testsuite/libffi.call/call.exp"
> invoked from within
> "catch "uplevel #0 source $test_file_name""
> Running ../../testsuite/libffi.special/special.exp ...
>
> After some searching, I found this similar bug on Tcl's tracker:
> <http://sourceforge.net/tracker/?func=detail&aid=3603557&group_id=10894&atid=110894>;
>
> The change that introduced the limit is here:
> <http://core.tcl.tk/tcl/info/1ab85d8c7d>;
>
> To make tcl86 work for the libffi tests, I simply doubled the limit from
> 500 to 1000, as in the attached diff. Can you please consider adding
> this to the tcl86 port?
The attached diff that I missed, and made by hand >.<
Pietro, Mikhail, I see no reason not to commit this?
I'll do so tomorrow if neither of you object.
Chris
http://www.bayofrum.net/~crees/patches/tcl-increase-regex-limit.diff
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo839m8-xHgzVs1H2G%2BbZ=_G1RVLdebneCcaHExdpKyM52Fw>