diff mbox

Increase expect match buffer size

Message ID AM5PR0701MB26579AA33D8F66D921BA88AEE4A00@AM5PR0701MB2657.eurprd07.prod.outlook.com
State New
Headers show

Commit Message

Bernd Edlinger July 17, 2017, 1:22 p.m. UTC 
Hi,


this is to mitigate a race condition in expect 5.45-7
(which will be fixed in 5.45-8) see this thread for details:

https://gcc.gnu.org/ml/gcc/2017-07/msg00081.html

By increasing the match buffer size from 2000 to 10000
bytes we can avoid the fall-out due to the bug in expect,
while we have to wait for the next stable debian version.


Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
Is it OK for trunk?


Thanks
Bernd.

Comments

Richard Biener July 17, 2017, 2:36 p.m. UTC | #1 
On Mon, 17 Jul 2017, Bernd Edlinger wrote:

> Hi,
> 
> 
> this is to mitigate a race condition in expect 5.45-7
> (which will be fixed in 5.45-8) see this thread for details:
> 
> https://gcc.gnu.org/ml/gcc/2017-07/msg00081.html
> 
> By increasing the match buffer size from 2000 to 10000
> bytes we can avoid the fall-out due to the bug in expect,
> while we have to wait for the next stable debian version.
> 
> 
> Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
> Is it OK for trunk?

Ok.

Richard.
diff mbox

Patch

2017-07-18  Bernd Edlinger  <bernd.edlinger@hotmail.de>

        * lib/gcc-dg.exp: Increase expect's match buffer size.

Index: gcc/testsuite/lib/gcc-dg.exp
===================================================================
--- gcc/testsuite/lib/gcc-dg.exp	(revision 250150)
+++ gcc/testsuite/lib/gcc-dg.exp	(working copy)
@@ -43,6 +43,9 @@ 
   setenv LANG C.ASCII
 }
 
+# Avoid sporadic data-losses with expect
+match_max -d 10000
+
 # Ensure GCC_COLORS is unset, for the rare testcases that verify
 # how output is colorized.
 if [info exists ::env(GCC_COLORS) ] {