Currently, executing a 'git pull' on a dirty worktree gives the
following annoying message:

    # User doesn't notice dirty worktree
    $ git pull

    ... # fetch operation

    error: Your local changes to the following files would be overwritten by 
merge:
        quux
    Please, commit your changes or stash them before you can merge.
    Aborting

At which point the user will stash her changes, re-execute git pull,
and pop the stash.  This process can easily be automated out for a
smooth:

    $ git pull

    ... # fetch operation

    Saved working directory and index state WIP on master: 8ea73ed baz
    HEAD is now at 8ea73ed baz

    ... # The merge/rebase process

    Dropped refs/stash@{0} (83f47fbfb07a741817633f9191dc4a1530f79249)

If there is a conflict during the merge/rebase process, the user has
to pop the stash by hand after committing the conflict resolution:

    $ git pull

    ... # fetch operation

    Saved working directory and index state WIP on master: 8ea73ed baz
    HEAD is now at 8ea73ed baz

    ... # The merge/rebase process

    Automatic merge failed; fix conflicts and then commit the result.
    Please run 'git stash pop' after commiting the conflict resolution.

Introduce a new configuration variable pull.autostash that does
exactly this.

Signed-off-by: Ramkumar Ramachandra <artag...@gmail.com>
---
 Documentation/config.txt   |  5 ++++
 Documentation/git-pull.txt |  7 ++++++
 git-pull.sh                | 33 ++++++++++++++++++++++--
 t/t5521-pull-options.sh    | 63 ++++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 106 insertions(+), 2 deletions(-)

diff --git a/Documentation/config.txt b/Documentation/config.txt
index 3d750e0..6c5cd8e 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -1804,6 +1804,11 @@ pull.rebase::
 it unless you understand the implications (see linkgit:git-rebase[1]
 for details).
 
+pull.autostash::
+       When true, automatically stash all changes before attempting
+       to merge/rebase, and pop the stash after a successful
+       merge/rebase.
+
 pull.octopus::
        The default merge strategy to use when pulling multiple branches
        at once.
diff --git a/Documentation/git-pull.txt b/Documentation/git-pull.txt
index 24ab07a..1c5384b 100644
--- a/Documentation/git-pull.txt
+++ b/Documentation/git-pull.txt
@@ -94,6 +94,13 @@ must be given before the options meant for 'git fetch'.
        has to be called afterwards to bring the work tree up to date with the
        merge result.
 
+--[no-]autostash::
+       When turned on, automatically stash all changes before
+       attempting to merge/rebase, and pop the stash after a
+       successful merge/rebase.  Useful for people who want to pull
+       with a dirty worktree.  This option can also be set via the
+       `pull.autostash` configuration variable.
+
 Options related to merging
 ~~~~~~~~~~~~~~~~~~~~~~~~~~
 
diff --git a/git-pull.sh b/git-pull.sh
index 5fe69fa..4edc06a 100755
--- a/git-pull.sh
+++ b/git-pull.sh
@@ -48,6 +48,7 @@ if test -z "$rebase"
 then
        rebase=$(git config --bool pull.rebase)
 fi
+autostash=$(git config --bool pull.autostash || echo false)
 dry_run=
 while :
 do
@@ -116,6 +117,12 @@ do
        --no-r|--no-re|--no-reb|--no-reba|--no-rebas|--no-rebase)
                rebase=false
                ;;
+       --autostash)
+               autostash=true
+               ;;
+       --no-autostash)
+               autostash=false
+               ;;
        --recurse-submodules)
                recurse_submodules=--recurse-submodules
                ;;
@@ -202,7 +209,8 @@ test true = "$rebase" && {
                then
                        die "$(gettext "updating an unborn branch with changes 
added to the index")"
                fi
-       else
+       elif test "$autostash" = false
+       then
                require_clean_work_tree "pull with rebase" "Please commit or 
stash them."
        fi
        oldremoteref= &&
@@ -219,6 +227,12 @@ test true = "$rebase" && {
                fi
        done
 }
+
+stash_required () {
+       ! (git diff-files --quiet --ignore-submodules &&
+       git diff-index --quiet --cached HEAD --ignore-submodules)
+}
+
 orig_head=$(git rev-parse -q --verify HEAD)
 git fetch $verbosity $progress $dry_run $recurse_submodules --update-head-ok 
"$@" || exit 1
 test -z "$dry_run" || exit 0
@@ -294,4 +308,19 @@ true)
        eval="$eval \"\$merge_name\" HEAD $merge_head"
        ;;
 esac
-eval "exec $eval"
+
+if test "$autostash" = true && stash_required
+then
+       git stash || die "$(gettext "Cannot autostash")" &&
+       require_clean_work_tree "pull" "Please commit or stash them." &&
+       if eval "$eval"
+       then
+               git stash pop || die "$(gettext "Cannot pop stash")"
+       else
+               exit_code=$?
+               echo "Please run 'git stash pop' after commiting the conflict 
resolution."
+               exit $exit_code
+       fi
+else
+       eval "exec $eval"
+fi
diff --git a/t/t5521-pull-options.sh b/t/t5521-pull-options.sh
index 3bdfe82..4545671 100755
--- a/t/t5521-pull-options.sh
+++ b/t/t5521-pull-options.sh
@@ -117,4 +117,67 @@ test_expect_success 'git pull --all' '
        )
 '
 
+test_expect_success 'pull --autostash with clean worktree' '
+       mkdir clonedautostash &&
+       (
+               cd clonedautostash &&
+               git init &&
+               git pull --autostash ../parent &&
+               test_must_fail test_path_is_file .git/refs/stash
+               test_commit one
+       ) &&
+       rm -rf clonedautostash
+'
+
+test_expect_success 'pull.autostash with clean worktree' '
+       mkdir clonedautostash &&
+       (
+               cd clonedautostash &&
+               git init &&
+               test_config pull.autostash true &&
+               git pull ../parent &&
+               test_must_fail test_path_is_file .git/refs/stash
+               test_commit one
+       ) &&
+       rm -rf clonedautostash
+'
+
+test_expect_success 'pull.autostash without conflict' '
+       mkdir clonedautostash &&
+       (
+               cd clonedautostash &&
+               git init &&
+               test_commit "root_commit" &&
+               cat >quux <<-\EOF &&
+               this is a non-conflicting file
+               EOF
+               git add quux &&
+               test_config pull.autostash true &&
+               git pull ../parent &&
+               test_must_fail test_path_is_file .git/refs/stash &&
+               test_path_is_file quux &&
+               test_commit one
+       ) &&
+       rm -rf clonedautostash
+'
+
+test_expect_success 'pull.autostash with conflict' '
+       mkdir clonedautostash &&
+       (
+               cd clonedautostash &&
+               git init &&
+               test_commit "will_conflict" file "this is a conflicting file" &&
+               cat >quux <<-\EOF &&
+               this is a non-conflicting file
+               EOF
+               git add quux &&
+               test_config pull.autostash true &&
+               test_must_fail git pull ../parent &&
+               test_must_fail test_commit one &&
+               test_path_is_file .git/refs/stash &&
+               test_must_fail test_path_is_file quux
+       ) &&
+       rm -rf clonedautostash
+'
+
 test_done
-- 
1.8.2.1.392.g6943ea6

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to