[PATCH 1/3] Fix windres.exp test on Cygwin
Jon Turney
jon.turney@dronecode.org.uk
Mon Jun 6 16:17:00 GMT 2016
I'm guessing that Cygwin's Tcl is built with the default FD_SETSIZE of 64 or
otherwise has some issue with fds greater than 64. windres.exp's main test
leaks 2 fds per iteration, for 31 files currently. This seems to cause the
following objdump -p test to fail. Workaround this by closing fds which are
finished with.
I haven't audited the rest of the tests to see if there are any other
instances of this problem.
before:
Running /wip/binutils-gdb/binutils/testsuite/binutils-all/windres/windres.exp ...
FAIL: objdump -p
=== binutils Summary ===
# of expected passes 104
# of unexpected failures 1
# of expected failures 1
# of unsupported tests 4
after:
=== binutils Summary ===
# of expected passes 105
# of expected failures 1
# of unsupported tests 4
binutils/ChangeLog:
2016-03-10 Jon Turney <jon.turney@dronecode.org.uk>
* testsuite/binutils-all/windres/windres.exp : Work around a
Cygwin tcl issue with more than 64 fds open by closing fds when
finished with them.
---
binutils/ChangeLog | 6 ++++++
binutils/testsuite/binutils-all/windres/windres.exp | 2 ++
2 files changed, 8 insertions(+)
diff --git a/binutils/testsuite/binutils-all/windres/windres.exp b/binutils/testsuite/binutils-all/windres/windres.exp
index fc29dfe..e37fb33 100644
--- a/binutils/testsuite/binutils-all/windres/windres.exp
+++ b/binutils/testsuite/binutils-all/windres/windres.exp
@@ -89,6 +89,7 @@ foreach res $res_list {
}
continue
}
+ close $rc
pass "windres/$broot (parse)"
set rc [open $res]
@@ -141,6 +142,7 @@ foreach res $res_list {
file delete "tmpdir/$broot.res"
file delete "tmpdir/$broot.dump"
}
+ close $rc
}
# Test objdump -p
--
2.8.3
More information about the Binutils
mailing list