tools/git-repo.git/tests/test_git_command.py, branch v2.72

upload: suppress git push progress when quiet or non-TTY

2026-10-26T21:51:52+00:00

`Project.UploadForReview()` unconditionally passed `--progress` to `git push`, forcing client and remote progress meters (`Counting objects`, `remote: Resolving deltas: 3%..100%`) even on non-TTY stderr or when `-q`/`--quiet` was passed. Meanwhile, `repo upload` ignored `opt.quiet` completely. Move `--progress` injection for `push` into `GitCommand` alongside `fetch` and `clone`, enabling it only when `sys.stderr.isatty()` and none of `--progress`, `--no-progress`, `--quiet`, or `-q` are set. Forward `opt.quiet` from `repo upload` to `Project.UploadForReview()` and pass `--no-progress` to `git push` rather than `--quiet` so Gerrit `remote: https://...` review URLs are still emitted and parsed by `_FindGerritUrls()`. Also suppress the non-error `[OK]` summary banner when `-q`/`--quiet` is set. Bug: 578114287 Change-Id: I88b7b055dcca0db80dae6962886a693fe717c741 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/649663 Tested-by: Gavin Mak Commit-Queue: Gavin Mak Reviewed-by: Brian Gan

git_command: allow lossless capture of machine output

2026-09-21T20:36:47+00:00

NUL-delimited Git formats may contain path bytes that are not valid UTF-8. Add an opt-in byte capture mode so parsers can preserve those bytes instead of receiving backslash replacements, while continuing to decode stderr for diagnostics. Bug: 543851900 Bug: 553599402 Change-Id: Ia9dfcde065a0f6f72c785a4b96896f346e17510a Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/632141 Tested-by: Gavin Mak Commit-Queue: Gavin Mak Reviewed-by: Brian Gan

project: centralize safe branch and commit resolution

2026-08-31T19:34:53+00:00

Resolve commit-ish values through one typed helper using rev-parse --verify --quiet and, on Git 2.30+, --end-of-options. Reject option-like revisions on older clients, reuse the helper for project and manifest resolution, and validate user branch names with check-ref-format --branch. Bug: 553599402 Change-Id: I2dda49ff3e781cec14d3b84263a8dace06b4073f Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/623182 Tested-by: Gavin Mak Commit-Queue: Gavin Mak Reviewed-by: Brian Gan

standardize file header wrt licensing

2025-08-21T18:04:41+00:00

We've been slightly inconsistent in the license header in files. Standardize them so we can automate checking. Change-Id: I3cdf85c9485d33cac2bb05c8080dfada3e5a5e8d Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/500102 Reviewed-by: Gavin Mak Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger

run_tests: move test filtering to pytest markers

2025-03-25T18:19:49+00:00

Move the test disable logic even closer to the exact test that's disabled. This way people updating tests have a better chance of seeing they'll get reduced coverage in the CQ. Change-Id: I57c1a073a844019798b27e14d742fd32925d9ae8 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/462882 Tested-by: Mike Frysinger Reviewed-by: Gavin Mak Commit-Queue: Mike Frysinger

cleanup: Update codebase to expect Python 3.6

2023-10-31T16:03:54+00:00

- Bump minimum version to Python 3.6. - Use f-strings in a lot of places. Change-Id: I2aa70197230fcec2eff8e7c8eb754f20c08075bb Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/389034 Tested-by: Jason R. Coombs Reviewed-by: Mike Frysinger Commit-Queue: Jason R. Coombs

delete Python 2 (object) compat

2023-10-20T04:51:01+00:00

Bug: 302871152 Change-Id: I39636d73a6e1d69efa8ade74f75c5381651e6dc8 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390054 Commit-Queue: Mike Frysinger Reviewed-by: Aravind Vasudevan Tested-by: Mike Frysinger

git_command: Augment underlying git errors with suggestions

2023-10-06T18:21:45+00:00

This change appends suggestions to the underlying git error to make the error slightly more actionable. DD: go/improve-repo-error-reporting & go/tee-repo-stderr Bug: b/292704435 Change-Id: I2bf8bea5fca42c6a9acd2fadc70f58f22456e027 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/387774 Commit-Queue: Aravind Vasudevan Reviewed-by: Jason Chang Tested-by: Aravind Vasudevan Reviewed-by: Aravind Vasudevan

isort: format codebase

2023-08-22T18:32:22+00:00

Change-Id: I6f11d123b68fd077f558d3c21349c55c5f251019 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/383715 Reviewed-by: Gavin Mak Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger

Raise repo exit errors in place of sys.exit

2023-08-07T23:56:07+00:00

Bug: b/293344017 Change-Id: I92d81c78eba8ff31b5252415f4c9a515a6c76411 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/381774 Tested-by: Jason Chang Reviewed-by: Joanna Wang Commit-Queue: Jason Chang