From 8a2e6a7ae010a064744acd16ef26aa27a5df6e34 Mon Sep 17 00:00:00 2001
From: Takuto Ikuta <tikuta@chromium.org>
Date: Fri, 26 Feb 2021 04:24:49 +0000
Subject: [PATCH] ninjalog_uploader_wrapper: do not use binary read/write for
 json

This is for python3 compatibility.

Bug: b/177465438
Change-Id: I4770136ea702bcaf7a8b644ef842db2f1a7d1200
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2717272
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Chris Hall <chrishall@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
---
 ninjalog_uploader_wrapper.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ninjalog_uploader_wrapper.py b/ninjalog_uploader_wrapper.py
index 83ba9a44fb..e55fca49a2 100755
--- a/ninjalog_uploader_wrapper.py
+++ b/ninjalog_uploader_wrapper.py
@@ -22,7 +22,7 @@ VERSION = 2
 
 def LoadConfig():
   if os.path.isfile(CONFIG):
-    with open(CONFIG, 'rb') as f:
+    with open(CONFIG, 'r') as f:
       config = json.load(f)
       if config['version'] == VERSION:
         config['countdown'] = max(0, config['countdown'] - 1)
@@ -37,7 +37,7 @@ def LoadConfig():
 
 
 def SaveConfig(config):
-  with open(CONFIG, 'wb') as f:
+  with open(CONFIG, 'w') as f:
     json.dump(config, f)