Re: FYI: Debian Bug#162583: Automake causes Perl to overuse stack space

[Alexandre Duret-Lutz]

>>> "adl" == Alexandre Duret-Lutz <address@hidden> writes:

[...]

 adl> -   /^((?:\\\n|[^\n])*)(?:\n(\t.*))?$/som;
 adl> +   /^(.*?)(?:(?<!\\)\n(\t.*))?$/som;

*blush*, and these flags are wrong.  We certainly don't want
`m', because `$' would then match embedded new lines.  (And `o'
is superfluous without variables.)

2002-10-08  Alexandre Duret-Lutz  <address@hidden>

        * automake.in (file_contents_internal): Fix the relationship
        regex flags, don't use `m'.
 
Index: automake.in
===================================================================
RCS file: /cvs/automake/automake/automake.in,v
retrieving revision 1.1376
diff -u -r1.1376 automake.in
--- automake.in 8 Oct 2002 21:40:30 -0000       1.1376
+++ automake.in 8 Oct 2002 22:19:17 -0000
@@ -8164,7 +8164,7 @@
          # `new-line tab" not preceded by backslash (continuation
          # line).
          my $paragraph = $_;
-         /^(.*?)(?:(?<!\\)\n(\t.*))?$/som;
+         /^(.*?)(?:(?<!\\)\n(\t.*))?$/s;
          my ($relationship, $actions) = ($1, $2 || '');
 
          # Separate targets from dependencies: the first colon.

-- 
Alexandre Duret-Lutz