diff options
author | Jeroen Dhollander <jeroendh@google.com> | 2025-10-15 17:27:09 +0200 |
---|---|---|
committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2025-10-15 08:32:00 -0700 |
commit | e4872ac8baec782c86222ca93efb4a7c9b9e0c7b (patch) | |
tree | 1dc658cef9a04937e4240eddec28b050d97ee094 /git_superproject.py | |
parent | 46232648091f5490ac31eba2ec54c8b9c20729bf (diff) | |
download | git-repo-e4872ac8baec782c86222ca93efb4a7c9b9e0c7b.tar.gz |
sync: Use 'git rebase' during 'repo sync --rebase'
'repo sync --rebase' should do a rebase if it encounters local commits
during a 'repo sync'.
This was broken by
https://gerrit-review.git.corp.google.com/c/git-repo/+/437421,
which caused this to execute the '_doff' hook (which stands for
'do fast forward'), which is implemented using 'git merge --no-stat'.
This caused *multiple* actual editor windows to pop up (*) during
'repo sync --rebase', asking the user to enter a commit message for the
merge.
In this CL I explicitly make that code path do a 'git rebase'.
(*) and if you use a terminal editor like 'vim', this means you have 2+ concurrent vim windows rendered in the same terminal, while 'repo sync' keeps on printing other output lines, again in the same terminal. The result is .... not pretty to say the least :(
Bug: b:434565811
Test: Used it myself for over a week.
Change-Id: I0bf3ff181f15b9d5b2e3f85f7f84e302139fdab7
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/518602
Reviewed-by: Mike Frysinger <vapier@google.com>
Tested-by: Jeroen Dhollander <jeroendh@google.com>
Commit-Queue: Jeroen Dhollander <jeroendh@google.com>
Diffstat (limited to 'git_superproject.py')
0 files changed, 0 insertions, 0 deletions