commit a6c52f566acfbff5b0f37158c0d33adf05d250e5
author Joanna Wang <jojwang@google.com> Thu Nov 03 16:51:19 2022 -0400
committer Joanna Wang <jojwang@google.com> Thu Nov 03 21:07:07 2022 +0000
tree d79d55b872c3be39c54dcb6ef41749c40d39ccf2
parent 0d130d2da0754c546f654ede99a79aac2b8e6c5f

Set tracing to always on and save to .repo/TRACE_FILE.

- add `--trace_to_stderr` option so stderr will include trace outputs and any other errors that get sent to stderr
- while TRACE_FILE will only include trace outputs

piggy-backing on: https://gerrit-review.googlesource.com/c/git-repo/+/349154

Change-Id: I3895a84de4b2784f17fac4325521cd5e72e645e2
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/350114
Reviewed-by: LaMont Jones <lamontjones@google.com>
Tested-by: Joanna Wang <jojwang@google.com>

git_config.py

diff --git a/git_config.py b/git_config.py
index 6f80ae0..98cade3 100644
--- a/git_config.py
+++ b/git_config.py
@@ -349,9 +349,9 @@
     except OSError:
       return None
     try:
-      Trace(': parsing %s', self.file)
-      with open(self._json) as fd:
-        return json.load(fd)
+      with Trace(': parsing %s', self.file):
+        with open(self._json) as fd:
+          return json.load(fd)
     except (IOError, ValueError):
       platform_utils.remove(self._json, missing_ok=True)
       return None