Re: Removing an individual stashed state

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: David Kastrup <dak@...>
Cc: Johannes Schindelin <Johannes.Schindelin@...>, Russ Brown <pickscrape@...>, <git@...>
Date: Tuesday, September 11, 2007 - 12:25 am

On Tue, Sep 11, 2007 at 12:30:33AM +0200, David Kastrup wrote:


Then perhaps the right solution is to move it to a "DONE" pile (rather
than leaving it on "IN" or deleting it entirely).

It _should_ be a simple matter of just removing the N'th line of the
reflog and appending it to an 'applied-stash' reflog. However, there are
a few problems:

  - Finding the line number is non-trivial. git-stash never actually
    parses stash@{1}, but hands it off to rev-parse. So it is legal to
    say stash@{10.minutes.ago}.
  - When you apply a stash, there might be conflicts. What should the
    semantics be in that case? If we move the stash to a 'DONE' pile (or
    delete it), then conflicts become a bit clunky:

      git-stash apply stash@{0}
      # oops, conflicts
      git-reset --hard
      hack hack hack
      git-commit
      git-stash apply stash@{0}
      # oops, we just applied a different stash

    If we wait until conflicts are resolved, then we are increasing the
    statefulness required. When do we mark the stash as successfully
    applied? Do we handle all cases where you "git-reset" to abort?


I think it is "herself" in this case. :)

-Peff
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Removing an individual stashed state, Russ Brown, (Mon Sep 10, 5:48 pm)
Re: Removing an individual stashed state, Russ Brown, (Tue Sep 11, 11:30 am)
Re: Removing an individual stashed state, David Kastrup, (Tue Sep 11, 5:39 pm)
Re: Removing an individual stashed state, Junio C Hamano, (Tue Sep 11, 5:53 pm)
Re: Removing an individual stashed state, Junio C Hamano, (Mon Sep 10, 10:06 pm)
Re: Removing an individual stashed state, Johannes Schindelin, (Mon Sep 10, 5:54 pm)
Re: Removing an individual stashed state, David Kastrup, (Mon Sep 10, 6:30 pm)
Re: Removing an individual stashed state, Jeff King, (Tue Sep 11, 12:25 am)
Re: Removing an individual stashed state, Wincent Colaiuta, (Mon Sep 10, 8:23 pm)
Re: Removing an individual stashed state, Martin Langhoff, (Mon Sep 10, 6:43 pm)
Re: Removing an individual stashed state, Steven Grimm, (Mon Sep 10, 7:01 pm)
Re: Removing an individual stashed state, Martin Langhoff, (Mon Sep 10, 7:17 pm)
Re: Removing an individual stashed state, Martin Langhoff, (Mon Sep 10, 6:03 pm)
Re: Removing an individual stashed state, Russ Brown, (Mon Sep 10, 6:01 pm)