diff options
author | Aravind Vasudevan <aravindvasudev@google.com> | 2023-09-14 08:46:44 +0000 |
---|---|---|
committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-09-14 17:13:37 +0000 |
commit | c993c5068e0f7e22124b1bfb17ad0425fe2b8c83 (patch) | |
tree | 7e1dadd9b04a42ebfd438e41f35c176f0f0578da /subcmds/manifest.py | |
parent | c3d7c8536c408a54c7af1486f0beec7a1b0eb0ad (diff) | |
download | git-repo-c993c5068e0f7e22124b1bfb17ad0425fe2b8c83.tar.gz |
subcmds: Use repo logger
Bug: b/292704435
Change-Id: Ia3a45d87fc0bf0d4a1ba53050d9c3cd2dba20e55
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/386236
Reviewed-by: Jason Chang <jasonnc@google.com>
Commit-Queue: Aravind Vasudevan <aravindvasudev@google.com>
Tested-by: Aravind Vasudevan <aravindvasudev@google.com>
Diffstat (limited to 'subcmds/manifest.py')
-rw-r--r-- | subcmds/manifest.py | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/subcmds/manifest.py b/subcmds/manifest.py index f72df348..101240d1 100644 --- a/subcmds/manifest.py +++ b/subcmds/manifest.py | |||
@@ -17,6 +17,10 @@ import os | |||
17 | import sys | 17 | import sys |
18 | 18 | ||
19 | from command import PagedCommand | 19 | from command import PagedCommand |
20 | from repo_logging import RepoLogger | ||
21 | |||
22 | |||
23 | logger = RepoLogger(__file__) | ||
20 | 24 | ||
21 | 25 | ||
22 | class Manifest(PagedCommand): | 26 | class Manifest(PagedCommand): |
@@ -132,7 +136,7 @@ to indicate the remote ref to push changes to via 'repo upload'. | |||
132 | manifest.SetUseLocalManifests(not opt.ignore_local_manifests) | 136 | manifest.SetUseLocalManifests(not opt.ignore_local_manifests) |
133 | 137 | ||
134 | if opt.json: | 138 | if opt.json: |
135 | print("warning: --json is experimental!", file=sys.stderr) | 139 | logger.warn("warning: --json is experimental!") |
136 | doc = manifest.ToDict( | 140 | doc = manifest.ToDict( |
137 | peg_rev=opt.peg_rev, | 141 | peg_rev=opt.peg_rev, |
138 | peg_rev_upstream=opt.peg_rev_upstream, | 142 | peg_rev_upstream=opt.peg_rev_upstream, |
@@ -159,13 +163,13 @@ to indicate the remote ref to push changes to via 'repo upload'. | |||
159 | if output_file != "-": | 163 | if output_file != "-": |
160 | fd.close() | 164 | fd.close() |
161 | if manifest.path_prefix: | 165 | if manifest.path_prefix: |
162 | print( | 166 | logger.warn( |
163 | f"Saved {manifest.path_prefix} submanifest to " | 167 | "Saved %s submanifest to %s", |
164 | f"{output_file}", | 168 | manifest.path_prefix, |
165 | file=sys.stderr, | 169 | output_file, |
166 | ) | 170 | ) |
167 | else: | 171 | else: |
168 | print(f"Saved manifest to {output_file}", file=sys.stderr) | 172 | logger.warn("Saved manifest to %s", output_file) |
169 | 173 | ||
170 | def ValidateOptions(self, opt, args): | 174 | def ValidateOptions(self, opt, args): |
171 | if args: | 175 | if args: |