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

Re: [CLOSED] git checkout <branch> allowed with uncommitted changes

From
AarQon <arqon@gmx.com>
Date
Oct 13, 2011, 17:09 UTC
Message-ID
<loom.20111013T181801-923@post.gmane.org>
In-Reply-To
<1318514356.4646.16.camel@centaur.lab.cmartin.tk>
Carlos Martín Nieto <cmn <at> elego.de> writes:
> When you changed branches, git told you that a file had been changed in
> the working tree.

That's a very good point, if it was actually documented at all that that's what it meant; and / or presented some warning that "BTW, if you edit this file again now, it'll screw up your whole tree" instead of an innocuous "M"; and didn't appear to contradict what the manpage says about "local modifications", we could probably have avoided half of this confusion. :P As it was, when I saw that M suddenly appear after several days of bouncing between branches without it and with everything working, I just thought "oh great, git's managed to break this tree", because I remembered the same thing from the previous trial run.

That there IS an indication though might just be enough for me to be able to deal with it. Realistically, switching branches with uncommitted changes (unless you're doing it because you've ALREADY screwed up and are changing the wrong branch) is basically a trainwreck waiting to happen.

git stash appears to be useless for any nontrivial change on the *other* branch, since there's no indication when you return to the stashed branch there's a stash sitting around, which is not something you're going to remember the next morning if fixing the master took the rest of the day, and you're not going to use "stash list" by then either.

But as long as you get the "warning", an alias that does a "commit -am 'temp commit to avoid git breaking the tree'" is something I think I can probably live with.

Thanks for all the help guys - very much appreciated. As far as I'm concerned, this topic's done.

(Though if someone can come up with a script / hook / whatever that improves the "visibility" of stash, that would be awesome. Or one that makes the refusal to switch branches consistent).

Looking at the manpage for checkout in the hope that there might be a "--safe"
switch, I don't understand why
  "-f  Proceed even if the index *or the working tree* differs from HEAD."
even exists, since it proceeds under those conditions anyway.
"--safe" appears to be exactly what the behavior should be if you DON'T
specify -f, except that -f nukes the working tree outright rather than just
bleeding it across. Hopefully it'll be clearer after some sleep.  :)
Previous: Carlos Martín NietoNext: Alexey Shumkin
Message 8 of 36 in “[BUG] git checkout <branch> allowed with uncommitted changes”
  1. arQonOct 13, 2011
  2. Nguyen Thai Ngoc DuyOct 13, 2011
  3. Alexey ShumkinOct 13, 2011
  4. arQonOct 13, 2011
  5. Andreas EricssonOct 13, 2011
  6. arQonOct 13, 2011
  7. Carlos Martín NietoOct 13, 2011
  8. arQonOct 13, 2011
  9. Alexey ShumkinOct 13, 2011
  10. Jakub NarebskiOct 13, 2011
  11. arQonOct 13, 2011
  12. Carlos Martín NietoOct 13, 2011
  13. arQonOct 13, 2011
  14. Alexey ShumkinOct 13, 2011
  15. Alexey ShumkinOct 14, 2011
  16. Holger HellmuthOct 13, 2011
  17. Carlos Martín NietoOct 13, 2011
  18. arQonOct 13, 2011
  19. Junio C HamanoOct 13, 2011
  20. arQonOct 13, 2011
  21. Jeff KingOct 14, 2011
  22. Holger HellmuthOct 14, 2011
  23. Victor EngmarkOct 14, 2011
  24. arQonOct 16, 2011
  25. Junio C HamanoOct 16, 2011
  26. Holger HellmuthOct 16, 2011
  27. Carlos Martín NietoOct 13, 2011
  28. Sergei OrganovOct 13, 2011
  29. PJ WeisbergOct 13, 2011
  30. Holger HellmuthOct 13, 2011
  31. arQonOct 13, 2011
  32. Holger HellmuthOct 13, 2011
  33. Victor EngmarkOct 13, 2011
  34. arQonOct 13, 2011
  35. Victor EngmarkOct 14, 2011
  36. Michael J GruberOct 13, 2011

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.