Re: HEAD: inclusion order wrong for input.c

[Ralf Wildenhues]

Hi Eric,

* Eric Blake wrote on Tue, Apr 10, 2007 at 03:24:15PM CEST:
> According to Ralf Wildenhues on 4/6/2007 5:07 AM:
> > ../../m4/tests/others.at:478: (sed -ne 1q; cat) < in.m4
> > + sed -ne 1q
> > + cat
> > stdout:
> > + set +x
> > ../../m4/tests/others.at:479: test "x`cat stdout`" = "x0)trailing data" || \
> >   { echo "skipping: sed is too greedy on seekable stdin"; exit 77; }
> > Not enabling shell tracing (command contains a `...` command substitution)
> > --- /dev/null       2006-05-22 18:44:12.000000000 +0200
> > +++ /tmp/build/tests/testsuite.dir/at-stdout        2007-04-04 
> > 18:36:33.000000000 +0200
> > @@ -0,0 +1 @@
> > +skipping: sed is too greedy on seekable stdin
> 
> Is this GNU sed?

Yes:
| GNU sed version 4.1.4

> If so, you may want to check if this will be solved
> before the next sed release.

I tried CVS sed now.  It does not have the above problem.  With it, the
test fails, though:

#                             -*- compilation -*-
68. others.at:458: testing ...
+ cat
+ set +x
../../m4/tests/others.at:466: m4 -b -d  < in.m4
+ m4 -b -d
+ cat
+ set +x
../../m4/tests/others.at:473: (m4; cat) < in.m4
+ m4
+ cat
+ set +x
../../m4/tests/others.at:478: (sed -ne 1q; cat) < in.m4
+ sed -ne 1q
+ cat
stdout:
0)trailing data
+ set +x
../../m4/tests/others.at:479: test "x`cat stdout`" = "x0)trailing data" || \
  { echo "skipping: sed is too greedy on seekable stdin"; exit 77; }
Not enabling shell tracing (command contains a `...` command substitution)
+ cat
+ set +x
../../m4/tests/others.at:487: (m4 -EE; cat) < in.m4
+ m4 -EE
+ cat
stderr:
/tmp/build/src/.libs/lt-m4:stdin:1: Warning: dnl: extra arguments ignored: 1 > 0
--- -   2007-04-10 20:48:38.229645000 +0200
+++ /tmp/build/tests/testsuite.dir/at-stdout    2007-04-10 20:48:38.000000000 
+0200
@@ -1,2 +1 @@
-trailing data
 
68. others.at:458: 68. stdin seekable (others.at:458): FAILED (others.at:487)


> Here's another iteration (and it should go faster, by showing me ALL of
> the stderr, rather than giving up at the first mismatch - fun with cheap
> m4 tricks).

Here's the output with that patch applied instead:

#                             -*- compilation -*-
70. others.at:606: testing ...
../../m4/tests/others.at:607: test -w /dev/full && test -c /dev/full || {
  echo "Skipping: no /dev/full support";
  exit 77
}
../../m4/tests/others.at:613: m4 -b -d --help >/dev/full < /dev/null
stderr:
/tmp/build/src/.libs/lt-m4: write error
../../m4/tests/others.at:613: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4: write error
../../m4/tests/others.at:619: m4 -b -d --version >/dev/full < /dev/null
stderr:
/tmp/build/src/.libs/lt-m4: write error: No space left on device
../../m4/tests/others.at:619: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4: write error: No space left on device
../../m4/tests/others.at:627: m4 -b -d in.m4 >/dev/full < /dev/null
stderr:
/tmp/build/src/.libs/lt-m4: write error: No space left on device
../../m4/tests/others.at:627: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4: write error: No space left on device
../../m4/tests/others.at:635: m4 -b -d in.m4 2>/dev/full < /dev/null
../../m4/tests/others.at:640: m4 -b -d -tdnl in.m4 2>/dev/full < /dev/null
../../m4/tests/others.at:645: m4 -b -d --debugfile=/dev/full -tdnl in.m4 < 
/dev/null
stderr:
/tmp/build/src/.libs/lt-m4: error writing to debug stream: No space left on 
device
../../m4/tests/others.at:645: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4: error writing to debug stream: No space left on device
../../m4/tests/others.at:656: m4 -b -d in.m4 >/dev/full < /dev/null
stderr:
/tmp/build/src/.libs/lt-m4:in.m4:1: write error: No space left on device
../../m4/tests/others.at:656: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4:in.m4:1: write error: No space left on device
../../m4/tests/others.at:664: m4 -b -d in.m4 >/dev/full < /dev/null
stderr:
/tmp/build/src/.libs/lt-m4:in.m4:1: write error: No space left on device
../../m4/tests/others.at:664: sed 's/^[^:]*[lt-]*m4[.ex]*:/m4:/
        /^m4debug: module/s/opening file.*/opening file/
        s/\(cannot open module [^:]*\):.*/\1/
    ' stderr >&2
stderr:
m4:in.m4:1: write error: No space left on device
../../m4/tests/others.at:669: false
../../m4/tests/others.at:669: exit code was 1, expected 0
70. others.at:606: 70. stdout full (others.at:606): FAILED (others.at:669)

Cheers,
Ralf