diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2018-04-19 11:02:32 -0700 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2018-04-23 09:55:17 -0700 |
| commit | dbf5b772b3b5f962c3186dd0073146539ec0586b (patch) | |
| tree | 7be89f9884da1a89cd357fd5a57038709684a49a | |
| parent | 0e945fdf23bac5a62c15edfcbfd9d6ac4eee592f (diff) | |
compiler/glsl: close fd's in glcpp_test.py
I would have thought falling out of scope would allow the gc to collect
these, but apparently it doesn't, and this hits an fd limit on macos.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106133
Fixes: db8cd8e36771eed98eb638fd0593c978c3da52a9
("glcpp/tests: Convert shell scripts to a python script")
Signed-off-by: Dylan Baker <dylan.c.baker@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Tested-by: Vinson Lee <vlee@freedesktop.org>
| -rwxr-xr-x | src/compiler/glsl/glcpp/tests/glcpp_test.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/compiler/glsl/glcpp/tests/glcpp_test.py b/src/compiler/glsl/glcpp/tests/glcpp_test.py index f07c2414ea..cdc63b3ac2 100755 --- a/src/compiler/glsl/glcpp/tests/glcpp_test.py +++ b/src/compiler/glsl/glcpp/tests/glcpp_test.py @@ -78,7 +78,8 @@ def _valgrind(glcpp, filename): extra_args = parse_test_file(filename, nl_format='\n') try: - _, tmpfile = tempfile.mkstemp() + fd, tmpfile = tempfile.mkstemp() + os.close(fd) with open(filename, 'rb') as f: proc = subprocess.Popen( ['valgrind', '--error-exitcode=31', '--log-file', tmpfile, glcpp] + extra_args, @@ -135,7 +136,8 @@ def _replace_test(args, replace): total += 1 testfile = os.path.join(args.testdir, filename) try: - _, tmpfile = tempfile.mkstemp() + fd, tmpfile = tempfile.mkstemp() + os.close(fd) with io.open(testfile, 'rt') as f: contents = f.read() with io.open(tmpfile, 'wt') as f: |