From 36cf4787d064635578cde107c9b993379b17fec6 Mon Sep 17 00:00:00 2001 From: Christoffer Jansson Date: Tue, 22 Feb 2022 14:17:00 +0100 Subject: [PATCH] Remove binary mode Realized I could actually test this fully locally using: vpython3 roll_deps.py --dry-run -i No-Try: True Bug: webrtc:13607 Change-Id: Ibd4d8c002f76b3524da22056a34645178c66687a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/252064 Reviewed-by: Mirko Bonadei Commit-Queue: Christoffer Jansson Cr-Commit-Position: refs/heads/main@{#36051} --- tools_webrtc/autoroller/roll_deps.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools_webrtc/autoroller/roll_deps.py b/tools_webrtc/autoroller/roll_deps.py index 85ba943e7d..2f374eb70c 100755 --- a/tools_webrtc/autoroller/roll_deps.py +++ b/tools_webrtc/autoroller/roll_deps.py @@ -449,8 +449,8 @@ def CalculateChangedClang(new_cr_rev): return match.group(1) raise RollError('Could not parse Clang revision!') - with open(CLANG_UPDATE_SCRIPT_LOCAL_PATH, 'rb') as f: - current_lines = f.readlines().decode('utf-8') + with open(CLANG_UPDATE_SCRIPT_LOCAL_PATH, 'r') as f: + current_lines = f.readlines() current_rev = GetClangRev(current_lines) new_clang_update_py = ReadRemoteCrFile(CLANG_UPDATE_SCRIPT_URL_PATH,