Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 2 Jun 2013 16:37:57 +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:  <CADLo839YCuSkTdwEdsZ-K%2BriK5myNzgGXXqfKAQzC2qb=ECHBQ@mail.gmail.com>
In-Reply-To: <CADLo839m8-xHgzVs1H2G%2BbZ=_G1RVLdebneCcaHExdpKyM52Fw@mail.gmail.com>
References:  <DF38771A-FFC5-4E92-B95A-24F13AB331B1@FreeBSD.org> <CADLo839m8-xHgzVs1H2G%2BbZ=_G1RVLdebneCcaHExdpKyM52Fw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 1 June 2013 18:01, Chris Rees <crees@freebsd.org> wrote:
> 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.
>

Committed, thanks!

Chris



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo839YCuSkTdwEdsZ-K%2BriK5myNzgGXXqfKAQzC2qb=ECHBQ>