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

Re: [RFC RFC PATCH] builtin/repo.c: change info default behavior to show all fields

From
Lucas Seiki Oshiro <lucasseikioshiro@gmail.com>
Date
Feb 25, 2026, 16:08 UTC
Message-ID
<E0F6937A-7B0D-4162-9752-4AF5C6A88531@gmail.com>
In-Reply-To
<20260225153132.7387-1-valusoutrik@gmail.com>
Hi!
> Yup, after reading it I did realise how pointless my suggestion was.

I wouldn't say it was pointless, since you identified something that you thought that could be improved, found the code to be changed and sent a patch proposing your change.

The default behaviour of git-repo-info with no arguments was not a complete consensus at the first place, some people thought it needed to return all the values, and some thought that no value should be returned.

Although in this case I was on the "winner" side (I'm using quotes because I don't think this is a competition), many changes that I did were rejected at first place, and I couldn't see at first what were the problems with them, which I couldn't see only after the reviewing process. The reviewing process is very strict, and it couldn't be different, since Git is used by more than 90% of the developers [2].

Finally, I must say that most things that I think that should do falls under three situations:

1. Git already does that, and I didn't know
2. It was already discussed, and people agreed that Git shouldn't
   do it
3. It's not possible to do that

In your case, it falls in the situation 2 and the discussion, being already discussed [3]. But I understand that it can be frustrating sometimes, and hope you don't feel discouraged in contributing to Git.

[1] 20250610152117.14826-1-lucasseikioshiro@gmail.com [2] https://survey.stackoverflow.co/2022/#version-control-version-control-system-prof [3] 20250915223618.13093-1-lucasseikioshiro@gmail.com

Previous: SoutrikDas
Message 14 of 14 in “builtin/repo.c: change info default behavior to show all fields”
  1. builtin/repo.c: change info default behavior to show all fieldsSoutrikDas, Feb 24, 2026
  2. builtin/repo.c: change info default behavior to show all fieldsSoutrikDas, Feb 24, 2026
  3. Junio C HamanoFeb 24, 2026
  4. SoutrikDasFeb 24, 2026
  5. K JayatheerthFeb 25, 2026
  6. Junio C HamanoFeb 25, 2026
  7. SoutrikDasFeb 25, 2026
  8. Junio C HamanoFeb 25, 2026
  9. Kristoffer HaugsbakkFeb 25, 2026
  10. JAYATHEERTH KFeb 25, 2026
  11. Lucas Seiki OshiroFeb 25, 2026
  12. SoutrikDasFeb 25, 2026
  13. SoutrikDasFeb 25, 2026
  14. Lucas Seiki OshiroFeb 25, 2026

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.