Can “git pull” automatically stash and pop pending changes?

前端 未结 5 858
死守一世寂寞
死守一世寂寞 2020-11-21 23:41

I know how to solve this:

user@host$ git pull
Updating 9386059..6e3ffde
error: Your local changes to the following files would be overwritten by merge:
    f         


        
5条回答
  •  再見小時候
    2020-11-22 00:29

    For Git 2.6+ (released 28 Sept 2015)

    The only git config setting which would be of interest is:

    rebase.autoStash
    

    (with Git 2.27, Q2 2020, you now also have merge.autostash, see below)

    When set to true, automatically create a temporary stash before the operation begins, and apply it after the operation ends.
    This means that you can run rebase on a dirty worktree.

    However, use with care: the final stash application after a successful rebase might result in non-trivial conflicts. Defaults to false.

    combine that with:

    pull.rebase
    

    When true, rebase branches on top of the fetched branch, instead of merging the default branch from the default remote when "git pull" is run.

    git config pull.rebase true
    git config rebase.autoStash true
    

    That would be enough for a simple git pull to work even in a dirty tree.
    No alias needed in that case.


    See commit 53c76dc (04 Jul 2015) by Kevin Daudt (Ikke).
    (Merged by Junio C Hamano -- gitster -- in commit e69b408, 17 Aug 2015)

    pull: allow dirty tree when rebase.autostash enabled

    rebase learned to stash changes when it encounters a dirty work tree, but git pull --rebase does not.

    Only verify if the working tree is dirty when rebase.autostash is not enabled.


    Note: if you want to pull without autostash (even though rebase.autoStash true is set), you have since git 2.9 (June 2016):

     pull --rebase --no-autostash
    

    See commit 450dd1d, commit 1662297, commit 44a59ff, commit 5c82bcd, commit 6ddc97c, commit eff960b, commit efa195d (02 Apr 2016), and commit f66398e, commit c48d73b (21 Mar 2016) by Mehul Jain (mehul2029).
    (Merged by Junio C Hamano -- gitster -- in commit 7c137bb, 13 Apr 2016)

    Commit f66398e in particular includes:

    pull --rebase: add --[no-]autostash flag

    If rebase.autoStash configuration variable is set, there is no way to override it for "git pull --rebase" from the command line.

    Teach "git pull --rebase" the --[no-]autostash command line flag which overrides the current value of rebase.autoStash, if set. As "git rebase" understands the --[no-]autostash option, it's just a matter of passing the option to underlying "git rebase" when "git pull --rebase" is called.


    Warning: before Git 2.14 (Q3 2017), "git pull --rebase --autostash" didn't auto-stash when the local history fast-forwards to the upstream.

    See commit f15e7cf (01 Jun 2017) by Tyler Brazier (tylerbrazier).
    (Merged by Junio C Hamano -- gitster -- in commit 35898ea, 05 Jun 2017)

    pull: ff --rebase --autostash works in dirty repo

    When git pull --rebase --autostash in a dirty repository resulted in a fast-forward, nothing was being autostashed and the pull failed.
    This was due to a shortcut to avoid running rebase when we can fast-forward, but autostash is ignored on that codepath.


    Update: Mariusz Pawelski asks in the comments an interesting question:

    So everybody is writing about autostash when you do rebase (or pull --rebase).

    But no one is taking about autostashing when you do normal pull with merges.
    So there is no automatic switch for that? Or I am missing something? I prefer doing git pull --rebase but OP asked about "standard" git pull

    Answer:

    The original thread discussing this autostash feature, it was implemented originally both for git pull (merge) and git pull --rebase.

    But... Junio C Hamano (Git maintainer) noted that:

    If the pull-merge were something that would induce the "annoyance" that triggered this topic, by definition, the local change overlaps with the merge, and this internal "stash pop" will touch the paths the merge touched and it is likely not result in "Dropped" but leave further conflicts to be resolved.

    I suspect that pull.autostash configuration is not a good addition because it encourages a bad, pain-inducing workflow.
    In simple cases it may not hurt, but when local changes are complex, it would actively hurt than not having it, and the configuration robs the incentive to choose.

    The equation is somewhat different for "pull-rebase", as "rebase" insists you to start from a clean working tree, so "download and then stop" annoyance feels bigger. I have a suspicion that loosening that may be a more productive fix to the real problem.

    So, regarding a classic pull-merge, it is better to:

    encourage the user to think about the nature of WIP he has in the working tree before running "git pull".
    Is it a too complex beast that may interfere with what others are doing, or is it a trivial change that he can stash away and pop it back?

    If the former, he will be far better off doing "checkout -b", keep working until the local change gets into somewhat a better shape and "commit", before pulling into the original branch.

    If the latter, he is better off doing:

    • "git pull",
    • after finding it conflicts, run
      • git stash,
      • git merge FETCH_HEAD and
      • git stash pop

    That being said, with Git 2.27 (Q2 2020), "git pull" learned to warn when no pull.rebase configuration exists, and neither --[no-]rebase nor --ff-only is given (which would result a merge).

    See commit d18c950 (10 Mar 2020) by Alex Henrie (alexhenrie).
    (Merged by Junio C Hamano -- gitster -- in commit 1c56d6f, 27 Mar 2020)

    pull: warn if the user didn't say whether to rebase or to merge

    Signed-off-by: Alex Henrie

    Often novice Git users forget to say "pull --rebase" and end up with an unnecessary merge from upstream.

    What they usually want is either "pull --rebase" in the simpler cases, or "pull --ff-only" to update the copy of main integration branches, and rebase their work separately.
    The pull.rebase configuration variable exists to help them in the simpler cases, but there is no mechanism to make these users aware of it.

    Issue a warning message when no --[no-]rebase option from the command line and no pull.rebase configuration variable is given.
    This will inconvenience those who never want to "pull --rebase", who haven't had to do anything special, but the cost of the inconvenience is paid only once per user, which should be a reasonable cost to help a number of new users.


    With Git 2.27 (Q2 2020), "git merge" learns the "--autostash" option, and the new merge.autostash setting.

    See commit d9f15d3, commit f8a1785, commit a03b555, commit 804fe31, commit 12b6e13, commit 0dd562e, commit 0816f1d, commit 9bb3dea, commit 4d4bc15, commit b309a97, commit f213f06, commit 86ed00a, commit facca7f, commit be1bb60, commit efcf6cf, commit c20de8b, commit bfa50c2, commit 3442c3d, commit 5b2f6d9 (07 Apr 2020), commit 65c425a (04 Apr 2020), and commit fd6852c, commit 805d9ea (21 Mar 2020) by Denton Liu (Denton-L).
    (Merged by Junio C Hamano -- gitster -- in commit bf10200, 29 Apr 2020)

    pull: pass --autostash to merge

    Signed-off-by: Denton Liu

    Before, --autostash only worked with git pull --rebase.

    However, in the last patch, merge learned --autostash as well so there's no reason why we should have this restriction anymore.
    Teach pull to pass --autostash to merge, just like it did for rebase.

    And:

    rebase: use apply_autostash() from sequencer.c

    Signed-off-by: Denton Liu

    The apply_autostash() function in builtin/rebase.c is similar enough to the apply_autostash() function in sequencer.c that they are almost interchangeable, except for the type of arg they accept. Make the sequencer.c version extern and use it in rebase.

    The rebase version was introduced in 6defce2b02 ("builtin rebase: support --autostash option", 2018-09-04, Git v2.20.0-rc0 -- merge listed in batch #8) as part of the shell to C conversion.
    It opted to duplicate the function because, at the time, there was another in-progress project converting interactive rebase from shell to C as well and they did not want to clash with them by refactoring sequencer.c version of apply_autostash().
    Since both efforts are long done, we can freely combine them together now.

提交回复
热议问题