git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add a manual page for git-stash

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 30, 2007, 18:13 UTC
Message-ID
<7vk5tlcnuq.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<Pine.LNX.4.64.0706301853400.4438@racer.site>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 14 quoted lines
>> > +apply [<stashname>]::
>> > +	Try to apply the stashed changes to the current HEAD. You need
>> > +	a clean working directory for that, i.e. you must not have changes
>> > +	relative to HEAD in your working directory or index.
>> 
>> The implementation appears to apply on a clean index without
>> restriction to where the HEAD is.  I hinted that that behaviour
>> is fine in my previous message, but on the other hand haven't
>> convinced myself enough to say that it would not confuse end
>> users.  Maybe insisting on not just clean index but no changes
>> from the HEAD would reduce confusion?  I dunno.
>
> I am sure confused why the index state is stashed away when it is not 
> used...

Yeah, I think Nanako kept this from her previous round because I mentioned that "git show stash" (not "git stash show") is interesting in that it gives stashed (HEAD, index, working tree) in --cc fashion. I also mentioned that it might make sense to stash untracked files in the working tree portion of the commit, and if we do that then we would need to record what was known to git by the index tree so that we can discard them from the index upon applying, but (1) I no longer think it is useful to try carrying forward the untracked file states, and (2) it would make "git show stash" uninteresting (too much cruft).

So perhaps we would want to drop the index tree portion from the stash.

Show 10 quoted lines
>> > +Author
>> > +------
>> > +Written by Johannes E. Schindelin <johannes.schindelin@gmx.de>
>> 
>> You wrote that ;-)?
>
> No. ;-)
>
> Hey, be nice. It's a new role for me, usually others document what _I_ 
> wrote, not vice versa :-)

Actually I have been wondering if it is a good idea to start dropping the Author section. Perhaps replace it by list of stakeholders or people who have been active in the area recently and who are likely to be able to help when an end-user has troubles with the command.

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 15 of 34 in “Add git-stash script”
  1. Add git-stash scriptしらいしななこ, Jun 30, 2007
  2. Johannes SchindelinJun 30, 2007
  3. Add git-stash scriptしらいしななこ, Jun 30, 2007
  4. Jeff KingJun 30, 2007
  5. Junio C HamanoJun 30, 2007
  6. Johannes SchindelinJun 30, 2007
  7. Junio C HamanoJun 30, 2007
  8. しらいしななこJun 30, 2007
  9. Add a manual page for git-stashJohannes Schindelin, Jun 30, 2007
  10. Frank LichtenheldJun 30, 2007
  11. Johannes SchindelinJun 30, 2007
  12. Frank LichtenheldJun 30, 2007
  13. Junio C HamanoJun 30, 2007
  14. Johannes SchindelinJun 30, 2007
  15. Junio C HamanoJun 30, 2007
  16. Johannes SchindelinJun 30, 2007
  17. Document git-stashしらいしななこ, Jul 1, 2007
  18. Junio C HamanoJul 1, 2007
  19. Jeff KingJul 1, 2007
  20. Junio C HamanoJul 1, 2007
  21. しらいしななこJul 1, 2007
  22. Junio C HamanoJul 1, 2007
  23. Johannes SchindelinJul 1, 2007
  24. Jeff KingJul 2, 2007
  25. Johannes SchindelinJul 2, 2007
  26. git-stash: Make "save" the default operation againJohannes Schindelin, Jul 2, 2007
  27. Jeff KingJul 2, 2007
  28. Johannes SchindelinJul 2, 2007
  29. Junio C HamanoJul 2, 2007
  30. Junio C HamanoJul 1, 2007
  31. Add tests for git-stashJohannes Schindelin, Jun 30, 2007
  32. Jeff KingJul 1, 2007
  33. Junio C HamanoJul 1, 2007
  34. Jeff KingJul 2, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.