Applying this commit adds a file named "c/_2018/march/06/ovs/tests/testsuite.dir/at-groups/42/stderr" to the repo! I think that the failure needs to be indented a bit to un-confuse "git am".
When I fix that up, I get the following failure from flake8: ../tests/flowgen.py:17:1: F401 'os' imported but unused Assuming you fix up those two little things: Acked-by: Ben Pfaff <[email protected]> Thanks! On Tue, Mar 06, 2018 at 05:23:56PM +0200, Alin Gabriel Serdean wrote: > 42. library.at:3: testing flow extractor ... > ./library.at:4: $PYTHON $srcdir/flowgen.py >/dev/null 3>flows 4>pcap > --- /dev/null 2018-03-06 15:59:49 +0200 > +++ /c/_2018/march/06/ovs/tests/testsuite.dir/at-groups/42/stderr > 2018-03-06 15:59:49 +0200 > @@ -0,0 +1,4 @@ > +Traceback (most recent call last): > + File "../.././flowgen.py", line 204, in <module> > + flows = os.fdopen(3, 'wb') > +OSError: [Errno 9] Bad file descriptor > ./library.at:4: exit code was 1, expected 0 > 42. library.at:3: FAILED (library.at:4) > > Unfortunately 3/4/5/.. FDs don't work on Windows. Switch to a filename open > instead of opening the FDs. > > Signed-off-by: Alin Gabriel Serdean <[email protected]> > --- > tests/flowgen.py | 4 ++-- > tests/library.at | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tests/flowgen.py b/tests/flowgen.py > index 221a8f2bc..4d31b05a9 100755 > --- a/tests/flowgen.py > +++ b/tests/flowgen.py > @@ -201,8 +201,8 @@ def output(attrs): > packets.write(packet) > > > -flows = os.fdopen(3, 'wb') > -packets = os.fdopen(4, 'wb') > +flows = open('flows', 'wb') > +packets = open('pcap', 'wb') > > # Print pcap file header. > packets.write(struct.pack('>LHHLLLL', > diff --git a/tests/library.at b/tests/library.at > index 5efbfbb7c..5bfea2f69 100644 > --- a/tests/library.at > +++ b/tests/library.at > @@ -1,7 +1,7 @@ > AT_BANNER([library unit tests]) > > AT_SETUP([flow extractor]) > -AT_CHECK([$PYTHON $srcdir/flowgen.py >/dev/null 3>flows 4>pcap]) > +AT_CHECK([$PYTHON $srcdir/flowgen.py >/dev/null]) > AT_CHECK([ovstest test-flows flows pcap], [0], [checked 247 packets, 0 errors > ]) > AT_CLEANUP > -- > 2.16.1.windows.1 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
