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

Re: git am mangles commit author name.

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jul 16, 2010, 19:19 UTC
Message-ID
<20100716191954.GB16371@burratino>
In-Reply-To
<AANLkTimIkepwy93dYCu5mkdSMHNuh3dbMyxcHPl4hGmt@mail.gmail.com>
Hi Daniel,
Daniel F wrote: 
> would simply setting minimum length to 1 have any negative
> side effects?

I am not a git maintainer, but I would suggest writing a patch to do that. Then you can try it and see, and send us a copy of the patch when reporting the results.

My guess is Linus wanted to make sure he had meaningful names in patches he applies[1]. There is no technical limit I know of that would lead one to forbid an author name longer than 60 characters, either. So the questions are:

 - did any code learn to rely on those limits later?
 - are any _people_ relying on those limits now?

I suspect the answer to question 2, at least for the lower bound, is no.

Hope that helps, Jonathan

[1] See v0.99.2~51^2~21 (Start of early patch applicator tools for git., 2005-04-11) for the original script.

Previous: Daniel F
Message 10 of 10 in “git am mangles commit author name.”
  1. Daniel FJul 13, 2010
  2. Jonathan NiederJul 13, 2010
  3. Daniel FJul 13, 2010
  4. Jay SoffianJul 13, 2010
  5. Daniel FJul 13, 2010
  6. Ævar Arnfjörð BjarmasonJul 13, 2010
  7. Tor ArntsenJul 14, 2010
  8. Ævar Arnfjörð BjarmasonJul 14, 2010
  9. Daniel FJul 16, 2010
  10. Jonathan NiederJul 16, 2010

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.