admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove binary mode

Browse Source 
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 <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#36051}
This commit is contained in:
Christoffer Jansson 2022-02-22 14:17:00 +01:00 committed by WebRTC LUCI CQ
parent e20a5247f0
commit 36cf4787d0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools_webrtc/autoroller/roll_deps.py
View File

@ -449,8 +449,8 @@ def CalculateChangedClang(new_cr_rev):
return match.group(1) return match.group(1)
raise RollError('Could not parse Clang revision!') raise RollError('Could not parse Clang revision!')
with open(CLANG_UPDATE_SCRIPT_LOCAL_PATH, 'rb') as f: with open(CLANG_UPDATE_SCRIPT_LOCAL_PATH, 'r') as f:
current_lines = f.readlines().decode('utf-8') current_lines = f.readlines()
current_rev = GetClangRev(current_lines) current_rev = GetClangRev(current_lines)
new_clang_update_py = ReadRemoteCrFile(CLANG_UPDATE_SCRIPT_URL_PATH, new_clang_update_py = ReadRemoteCrFile(CLANG_UPDATE_SCRIPT_URL_PATH,