]> git.ipfire.org Git - thirdparty/git.git/commit
pull: avoid running both merge and rebase
authorElijah Newren <newren@gmail.com>
Fri, 27 Mar 2020 21:51:40 +0000 (21:51 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 27 Mar 2020 22:54:54 +0000 (15:54 -0700)
commitfbae70ddc6da7df3f901d95da2b11b2cb58c8a29
tree773b411fae5b202d3d0436ed973d53de239109bc
parentb6d4d82bd5a49197d5d2f4f81c08da0d461cfcf1
pull: avoid running both merge and rebase

When opt_rebase is true, we still first check if we can fast-forward.
If the branch is fast-forwardable, then we can avoid the rebase and just
use merge to do the fast-forward logic.  However, when commit a6d7eb2c7a
("pull: optionally rebase submodules (remote submodule changes only)",
2017-06-23) added the ability to rebase submodules it accidentally
caused us to run BOTH a merge and a rebase.  Add a flag to avoid doing
both.

This was found when a user had both pull.rebase and rebase.autosquash
set to true.  In such a case, the running of both merge and rebase would
cause ORIG_HEAD to be updated twice (and match HEAD at the end instead
of the commit before the rebase started), against expectation.

Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/pull.c