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

Re: Call Me Gitless

From
Daniel Barkalow <barkalow@iabervon.org>
Date
Aug 21, 2008, 13:58 UTC
Message-ID
<alpine.LNX.1.00.0808210928010.19665@iabervon.org>
In-Reply-To
<7viqtukbec.fsf@gitster.siamese.dyndns.org>
On Thu, 21 Aug 2008, Junio C Hamano wrote:
Show 36 quoted lines
> Daniel Barkalow <barkalow@iabervon.org> writes:
> 
> > On Mon, 18 Aug 2008, Junio C Hamano wrote:
> > ...
> >>     If we had a configuration for "index-free" people, that changes the
> >>     semantics of "git add" to register object name of an empty blob when a
> >>     new path is added, makes "git add" for existing blobs a no-op, but
> >>     keeps "git commit -a" and "git commit <paths>" to operate as they
> >>     currently do, then people with such configuration could:
> >> 
> >> 	$ >new-file
> >>         $ git add new-file
> >>         $ edit old-file
> >>         $ edit new-file
> >>         $ git diff
> >> 
> >>     to always see what's the difference from the HEAD is with "git diff",
> >>     and any of these three:
> >> 
> >> 	$ git commit -a
> >>         $ git commit old-file
> >>         $ git commit old-file new-file
> >> 
> >>     would work as expected by them.  We still need to support the three
> >>     diff variants for normal git people, but people who do not use index
> >>     do not have to know the two variants ("git diff" vs "git diff HEAD");
> >>     such a change could be argued as a "UI improvement" [*1*].
> >
> > I think that having the possibility of adding an empty blob (or maybe a 
> > magical "nothing currently here but git-ls-files includes it") would be 
> > preferrable to a no-index mode.
> 
> I am not sure if you are really saying something different from what I am
> saying.  We'll see after this three patch series.  The first one is an
> unrelated bugfix (but the bug won't trigger with existing callers -- only
> triggered with the added codepath).

I see this primarily as something you can use if you're worried about leaving files out of commits. When you create the file, you can use "git add -N" to make sure that it won't get overlooked when you're adding things. If you weren't using the index for anything important, you could just use a normal add, but that would get confusing if you're adding completed changes as well as some half-written version of any file that happens to be new. That is, it'll let you cause "git diff" to report the contents as unstaged changes in your working tree, rather than not reporting it (either because it's not tracked at all, or because the changes are now staged). Then you can decide to stage them.

(The thing that I'd ideally like to have different is for:

$ echo "content" > new-name $ git add -N new-name $ git commit

Say:

# On branch master # Changed but not updated: # (use "git add <file>..." to update what will be committed) # # added: new-name # no changes added to commit (use "git add" and/or "git commit -a")

rather than committing the empty blob. But that's tricky to implement and keep from breaking other stuff and really minor; and the documentation doesn't exclude that being what happens with -N)

	-Daniel
*This .sig left intentionally blank*
Previous: Jonathan NiederNext: Tarmigan
Message 46 of 67 in “Call Me Gitless”
  1. TransAug 18, 2008
  2. Benjamin SergeantAug 18, 2008
  3. Martin LanghoffAug 18, 2008
  4. Pascal ObryAug 18, 2008
  5. Jon LoeligerAug 18, 2008
  6. Daniel BarkalowAug 18, 2008
  7. Marcus GriepAug 18, 2008
  8. Junio C HamanoAug 18, 2008
  9. Daniel BarkalowAug 18, 2008
  10. Junio C HamanoAug 18, 2008
  11. Daniel BarkalowAug 18, 2008
  12. Junio C HamanoAug 19, 2008
  13. Marcus GriepAug 19, 2008
  14. Junio C HamanoAug 19, 2008
  15. Junio C HamanoAug 19, 2008
  16. diff: vary default prefix depending on what are comparedJunio C Hamano, Aug 20, 2008
  17. Jakub NarebskiAug 20, 2008
  18. Stephen R. van den BergAug 19, 2008
  19. Jakub NarebskiAug 19, 2008
  20. Junio C HamanoAug 19, 2008
  21. Jeff KingAug 19, 2008
  22. Daniel BarkalowAug 19, 2008
  23. Jeff KingAug 19, 2008
  24. Daniel BarkalowAug 19, 2008
  25. Jeff KingAug 19, 2008
  26. Daniel BarkalowAug 19, 2008
  27. Petr BaudisAug 19, 2008
  28. Daniel BarkalowAug 19, 2008
  29. Junio C HamanoAug 19, 2008
  30. Junio C HamanoAug 19, 2008
  31. Daniel BarkalowAug 19, 2008
  32. Sverre Hvammen JohansenAug 21, 2008
  33. Junio C HamanoAug 21, 2008
  34. 1/3 sha1_object_info(): pay attention to cached objectsJunio C Hamano, Aug 21, 2008
  35. 2/3 cached_object: learn empty blobJunio C Hamano, Aug 21, 2008
  36. 3/3 git-add --intent-to-add (-N)Junio C Hamano, Aug 21, 2008
  37. Paolo BonziniAug 21, 2008
  38. Jonathan NiederAug 21, 2008
  39. Jonathan NiederAug 22, 2008
  40. Daniel BarkalowAug 22, 2008
  41. Junio C HamanoAug 22, 2008
  42. Jonathan NiederAug 22, 2008
  43. Junio C HamanoAug 22, 2008
  44. Jonathan NiederAug 22, 2008
  45. Jonathan NiederAug 22, 2008
  46. Daniel BarkalowAug 21, 2008
  47. TarmiganAug 18, 2008
  48. Daniel BarkalowAug 19, 2008
  49. TarmiganAug 19, 2008
  50. Peter Valdemar Mørch (Lists)Aug 19, 2008
  51. Junio C HamanoAug 19, 2008
  52. Imran M YousufAug 19, 2008
  53. Peter Valdemar Mørch (Lists)Aug 19, 2008
  54. Imran M YousufAug 19, 2008
  55. Alexander E GenaudAug 19, 2008
  56. Matthieu MoyAug 19, 2008
  57. Mike HommeyAug 19, 2008
  58. Alexander E GenaudAug 19, 2008
  59. Pascal ObryAug 19, 2008
  60. Paolo BonziniAug 21, 2008
  61. Elijah NewrenAug 22, 2008
  62. Peter Valdemar Mørch (Lists)Aug 19, 2008
  63. Mark StrubergAug 19, 2008
  64. Alexander E GenaudAug 19, 2008
  65. Junio C HamanoAug 19, 2008
  66. Teemu LikonenAug 19, 2008
  67. Jakub NarebskiAug 19, 2008

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.