tools/git-repo.git/git_command.py, branch v2.71

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

style: use `import repo_logging` everywhere

2026-10-23T16:49:36+00:00

Align with Google style guides and `import repo_logging` as a module rather than import symbols out of it. Change-Id: Ie13c1f13be00dbf6dd3814a8780baf6453972237 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/647823 Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger Reviewed-by: Brian Gan

style: use `import repo_trace` everywhere

2026-10-23T16:01:00+00:00

Align with Google style guides and `import repo_trace` as a module rather than import symbols out of it. Change-Id: I6bea5e7a68775e93b6b7fcd218db666350aff690 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/647803 Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger Reviewed-by: Gavin Mak

style: use `import error` everywhere

2026-10-23T00:44:11+00:00

Align with Google style guides and `import error` as a module rather than import classes out of it. Change-Id: I8686fbe7887bd6ceac7a07877af63934f0d0ddd2 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/647563 Commit-Queue: Mike Frysinger Reviewed-by: Gavin Mak Reviewed-by: Insaf Latypov Tested-by: Mike Frysinger

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

Fix all flake8 warnings from newer flake8-bugbear and flake8-comprehensions

2026-04-03T14:50:52+00:00

Address warnings introduced by flake8-bugbear 24.12.12 and flake8-comprehensions 3.16.0: - C408: Replace dict()/list() calls with literal {} and [] - C413: Remove unnecessary list() around sorted() - C414: Remove unnecessary list() inside sorted() - C419: Suppress intentional list comprehension in all() (noqa) - B001: Replace bare except with except Exception - B006: Replace mutable default arguments with None - B010: Replace setattr() with direct attribute assignment - B017: Use RuntimeError instead of Exception in tests - B019: Suppress lru_cache on methods for long-lived objects (noqa) - B033: Remove duplicate item in set literal Change-Id: If4693d3e946200bbc22f689f7b94da604addcb80 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/566321 Tested-by: Carlos Fernandez Commit-Queue: Carlos Fernandez Reviewed-by: Mike Frysinger Reviewed-by: Gavin Mak

git_refs: read refs via git plumbing for files/reftable

2026-03-17T21:28:39+00:00

Replace direct `packed-refs` file parsing with `git for-each-ref` plumbing to support both `files` and `reftable` backends. Bug: 476209856 Change-Id: I2ad8ff8f3382426600f15370c997f9bc17165485 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/550401 Commit-Queue: Gavin Mak Reviewed-by: Mike Frysinger Tested-by: Gavin Mak

black: update to v25

2025-03-25T18:20:35+00:00

Requires a little reformatting in the tree. Change-Id: Iaa40fe0dfca372c49c04cc26edccb5f7b0c2a8ad Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/462883 Commit-Queue: Mike Frysinger Reviewed-by: Gavin Mak Tested-by: Mike Frysinger

Disable git terminal prompt during fetch/clone

2024-09-26T22:10:36+00:00

git fetch operation may prompt user to enter username and password. This won't be visible to user when repo sync operation since stdout and stderr are redirected. If that happens, user may think repo is doing work and likely won't realize it's stuck on user's input. This patch disables prompt for clone and fetch operations, and repo will fail fast. R=gavinmak@google.com Bug: b/368644181 Change-Id: I2efa88ae66067587a00678eda155d861034b9127 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/438001 Reviewed-by: Nasser Grainawi Tested-by: Josip Sokcevic Commit-Queue: Josip Sokcevic Reviewed-by: Gavin Mak