Hi Marten, On 30/01/2018 11:05, Martin Häcker wrote: > > I just lost quite some work, because `git stash; git stash pop` > doesn’t seem to understand the concept of the index correctly. I`m afraid it actually seems you`re not fully understanding how `git stash pop` works - but the good news is none of your work is lost, as long as you have the stash sha1 (or you can find/restore it through `git fsck`). > What git stash does get right is that it does remove everything that > is stashed from the current state of the repo, but what it doesn’t > get right is restoring that state fatefully in `git stash pop`. According `git stash pop`[1] documentation, this is as expected, but I see the confusing part: Remove a single stashed state from the stash list and apply it on top of the current working tree state, i.e., do the inverse operation of `git stash push`. The working directory must match the index. It clearly says that stash is applied "on top of the current _working tree_ state" (so _not_ index), but then it says "i.e., do the inverse operation of `git stash push`", which is not exactly true (in regards to index handling, no options provided explicitly). Later below, it notes what to do to restore index as well: If the `--index` option is used, then tries to reinstate not only the working tree’s changes, but also the index’s ones. However, this can fail, when you have conflicts (which are stored in the index, where you therefore can no longer apply the changes as they were originally). > As a user, I would expect that `git stash pop` undoes the change that > `git stash` inflicted. This seems as a fair expectation, especially when documentation itself states that `git stash pop` is reverse operation of `git stash push`... As to why `git stash pop` behaves like this, I would leave to someone more informed to answer, what I myself managed to dig out so far is that it seems `--index` was originally discussed to be a default indeed, with `--skip-index` to turn it off if needed, see "[PATCH] Teach git-stash to "apply --index""[2] (2007-07-02, Johannes Schindelin) (note that `git stash pop` is essentially `git stash apply && git stash drop`). For the reference, here are some similar topics: - "[BUG] git stash doesn't use --index as default"[3] (2013-10-11, James) - "Asymmetric default behavior of git stash"[4] (2013-08-27, Uwe Storbeck) - "Newbie question: Is it possible to undo a stash?"[5] (2008-05-14, Iván V.) Finally, here`s your demo script, just modified to use `git stash pop --index` instead, yielding desired results: mkdir test cd test git init touch foo git add foo git commit -m "initial commit" echo "bar" >> foo git status git add foo git status echo "baz" >> foo git diff git status git stash git status git stash pop --index git diff Regards, Buga [1] https://git-scm.com/docs/git-stash#git-stash-pop--index-q--quietltstashgt [2] https://public-inbox.org/git/Pine.LNX.4.64.0707021213350.4438@xxxxxxxxxx/T/#u [3] https://public-inbox.org/git/1381467430.4130.38.camel@xxxxxxxxxxxxxxxxxxxx/T/#u [4] https://public-inbox.org/git/20130827132210.GA14266@xxxxxx/T/#u [5] https://public-inbox.org/git/509f40850805141256gce6ac1brf5ced6436f81dae8@xxxxxxxxxxxxxx/T/#u
