repo: Use the worktree when checking the repo rev.

Avoids treating the operation as if it were acting on a bare repository, thereby triggering failures when the Git client is configured with `safe.bareRepository=explicit`. Repo doesn't actually use a bare repository, but pointing at the gitdir acts as if it had.

Bug: 307559774
Change-Id: I2c142275b2726a59526729c0b2c54faf728f125d
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/391554
Commit-Queue: Jason R. Coombs <jaraco@google.com>
Tested-by: Jason R. Coombs <jaraco@google.com>
Tested-by: Emily Shaffer <emilyshaffer@google.com>
Reviewed-by: Emily Shaffer <emilyshaffer@google.com>
Reviewed-by: Mike Frysinger <vapier@google.com>
This commit is contained in:
Jason R. Coombs 2023-11-01 10:35:30 -04:00 committed by LUCI
parent 8a6d1724d9
commit 560a79727f

View File

@ -353,7 +353,7 @@ to update the working directory files.
wrapper = Wrapper() wrapper = Wrapper()
try: try:
remote_ref, rev = wrapper.check_repo_rev( remote_ref, rev = wrapper.check_repo_rev(
rp.gitdir, rp.worktree,
opt.repo_rev, opt.repo_rev,
repo_verify=opt.repo_verify, repo_verify=opt.repo_verify,
quiet=opt.quiet, quiet=opt.quiet,