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

Re: git-commit bug

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 26, 2012, 18:08 UTC
Message-ID
<7vy5naaqut.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20120626150436.GU9682@arachsys.com>
Chris Webb <chris@arachsys.com> writes:
Show 30 quoted lines
> Chris Webb <chris@arachsys.com> writes:
>
>> PS Whilst experimenting, I also noticed a (presumably unintentional)
>> behaviour:
>> 
>>   $ git init .
>>   Initialized empty Git repository in /tmp/foo/.git/
>>   $ git checkout --detach
>>   $ touch bar
>>   $ git add bar
>>   $ git commit -m test
>>   [(null) (root-commit) 17b5bf9] test
>>    0 files changed
>>    create mode 100644 bar
>>   $ ls .git/refs/heads/
>>   (null)
>>   $
>> 
>> Here we've created a branch with the strange name '(null)' instead of
>> actually detaching, or refusing to detach because we're on an unborn
>> branch.
>
> This was introduced by abe199808c, which is intended to allow
>
>   git init . && git checkout --orphan newbranch
>
> but presumably wasn't also meant to enable
>
>   git checkout --orphan foo
>   git checkout --detach
Correct.

On days like this, I really regret accepting that --orphan crap in the first place.

Thanks for fixing things up.
Show 11 quoted lines
> This leads to a printf("%s", NULL) and thus
>
>   $ git symbolic-ref HEAD
>   refs/heads/(null)
>
> I've followed up to this message with a patch including a test to catch this
> in future.
>
> Best wishes,
>
> Chris.
Previous: Chris WebbNext: Chris Webb
Message 17 of 29 in “Editing the root commit”
  1. Chris WebbJun 19, 2012
  2. Junio C HamanoJun 19, 2012
  3. Chris WebbJun 19, 2012
  4. Chris WebbJun 20, 2012
  5. Junio C HamanoJun 20, 2012
  6. Jeff KingJun 20, 2012
  7. Chris WebbJun 20, 2012
  8. Jeff KingJun 20, 2012
  9. Chris WebbJun 22, 2012
  10. Junio C HamanoJun 22, 2012
  11. Chris WebbJun 22, 2012
  12. Chris WebbJun 22, 2012
  13. Junio C HamanoJun 22, 2012
  14. Chris WebbJun 23, 2012
  15. git-commit bug (was Re: Editing the root commit)Chris Webb, Jun 26, 2012
  16. git-checkout: disallow --detach on unborn branchChris Webb, Jun 26, 2012
  17. Junio C HamanoJun 26, 2012
  18. Chris WebbJun 26, 2012
  19. 1/2 rebase -i: support --root without --ontoChris Webb, Jun 26, 2012
  20. Junio C HamanoJun 26, 2012
  21. Chris WebbJun 26, 2012
  22. Junio C HamanoJun 26, 2012
  23. Chris WebbJun 26, 2012
  24. Junio C HamanoJun 26, 2012
  25. Chris WebbJun 26, 2012
  26. 2/2 Add tests for rebase -i --root without --ontoChris Webb, Jun 26, 2012
  27. Chris WebbJun 20, 2012
  28. Martin von ZweigbergkJun 25, 2012
  29. jaseem abidJun 19, 2012

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.