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

Re: What's cooking in git.git (Oct 2023, #01; Mon, 2)

From
Sergey Organov <sorganov@gmail.com>
Date
Oct 5, 2023, 20:59 UTC
Message-ID
<87lecgeqfu.fsf@osv.gnss.ru>
In-Reply-To
<xmqqjzs1mkma.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> writes:
> Sergey Organov <sorganov@gmail.com> writes:
[...]
Show 10 quoted lines
>>> If I have to pick a candidate for "get me diff" that is the most
>>> useful among those currently are available, it is "give patches to
>>> all single-parent commit, and show tricky conflict resolution part
>>> only for merge commits".
>>
>> I'm afraid you need to pick a candidate that will be natural for '-d',
>> not just most useful output for your workflows, whatever it happens to
>> be.
>
> Literal match to word "diff" does not necessarily mean it is useful,
Sure, who argues? I don't.
> and short-and-sweet single-letter option name is primarily about
> letting users reach useful features with minimum typing [*1*], so you
> cannot avoid "most useful" being a large part of the equation.

I don't try to avoid "most useful" either, quite opposite. With whom do you argue?

I just pointed that a short-cut would better be natural (or mnemonic) /as well/, so you probably don't actually want:

  -d:
     give patches to all single-parent commits, and show tricky conflict
     resolution part only for merge commits.
, or do you?

Overall, as an example, I'd understand if you had deflected the patch with "let's rather use -d for '--decorate=short', or '--date=relative'", or something like that, but you don't, leaving me uncertain about your actual worries and intentions.

Anyway, I re-submitted the patches avoiding precious, too hard to deserve single-letter option.

Thanks, -- Sergey Organov

Previous: Junio C HamanoNext: Junio C Hamano
Message 8 of 12 in “What's cooking in git.git (Oct 2023, #01; Mon, 2)”
  1. Junio C HamanoOct 3, 2023
  2. Sergey OrganovOct 3, 2023
  3. Junio C HamanoOct 3, 2023
  4. Sergey OrganovOct 3, 2023
  5. Junio C HamanoOct 4, 2023
  6. Sergey OrganovOct 4, 2023
  7. Junio C HamanoOct 5, 2023
  8. Sergey OrganovOct 5, 2023
  9. Junio C HamanoOct 5, 2023
  10. Sergey OrganovOct 6, 2023
  11. Eric W. BiedermanOct 4, 2023
  12. Junio C HamanoOct 4, 2023

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.