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

Re: [PATCH] pull: add angle brackets to usage string

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 16, 2015, 17:42 UTC
Message-ID
<xmqq4mhqg15y.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CAMMLpeQPngq-xM2vN4uX6+k5-uoOXyP3p0wgLO9LOkWxWtZBEA@mail.gmail.com>
Alex Henrie <alexhenrie24@gmail.com> writes:
Show 10 quoted lines
> 2015-10-16 10:36 GMT-06:00 Junio C Hamano <gitster@pobox.com>:
>> Makes sense, as all the other <placeholders> in the usage string are
>> bracketted.
>>
>> Does it make sense to do this for contrib/examples, which is the
>> historical record, though?
>
> I didn't know that contrib/examples was a historical record. The last
> patch I submitted, b7447679e84ed973430ab19fce87f56857b83068, also
> modified contrib/examples.
Yes, but that fixes historical "mistake", no?

With this, you are breaking historical practice by changing only one instance to deviate from the then-current practice of saying 'options' without brackets. It is based on the point of view that considers anything inside <bracket> and a fixed string 'options' are meant to be replaced by intelligent readers, which is as valid as the more recent practice to consider only things inside <bracket> are placeholders.

Previous: Alex HenrieNext: Alex Henrie
Message 4 of 10 in “pull: add angle brackets to usage string”
  1. pull: add angle brackets to usage stringAlex Henrie, Oct 16, 2015
  2. Junio C HamanoOct 16, 2015
  3. Alex HenrieOct 16, 2015
  4. Junio C HamanoOct 16, 2015
  5. Alex HenrieOct 19, 2015
  6. Junio C HamanoOct 20, 2015
  7. Alex HenrieOct 20, 2015
  8. Junio C HamanoOct 20, 2015
  9. Ralf ThielowOct 16, 2015
  10. Junio C HamanoOct 16, 2015

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.