Mesa (master): python: difflib prefers unicode strings

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Aug 22 15:42:34 UTC 2018


Module: Mesa
Branch: master
Commit: fc708069f74063428e90595f494de6c79515f64e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fc708069f74063428e90595f494de6c79515f64e

Author: Mathieu Bridon <bochecha at daitauha.fr>
Date:   Fri Aug 17 21:32:16 2018 +0200

python: difflib prefers unicode strings

Python 3 does not automatically convert from bytes to unicode strings
like Python 2 used to do.

This commit makes sure we pass unicode strings to difflib.unified_diff,
so that the script works on both Python 2 and 3.

Reviewed-by: Dylan Baker <dylan at pnwbakers.com>

---

 src/compiler/glsl/glcpp/tests/glcpp_test.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/compiler/glsl/glcpp/tests/glcpp_test.py b/src/compiler/glsl/glcpp/tests/glcpp_test.py
index 8ac5d7cb0a..e27391093c 100755
--- a/src/compiler/glsl/glcpp/tests/glcpp_test.py
+++ b/src/compiler/glsl/glcpp/tests/glcpp_test.py
@@ -64,8 +64,9 @@ def test_output(glcpp, filename, expfile, nl_format='\n'):
             stderr=subprocess.STDOUT,
             stdin=subprocess.PIPE)
         actual, _ = proc.communicate(f.read())
+        actual = actual.decode('utf-8')
 
-    with open(expfile, 'rb') as f:
+    with open(expfile, 'r') as f:
         expected = f.read()
 
     if actual == expected:




More information about the mesa-commit mailing list