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

Re: [PATCH v1 1/3] doc: commit: --fixup/--squash can take a commit revision

From
Philip Oakley <philipoakley@iee.org>
Date
Aug 14, 2016, 23:29 UTC
Message-ID
<F18C2CCC29E5470CA1374D9718B2A3E6@PhilipOakley>
In-Reply-To
<xmqqbn0vj6ze.fsf@gitster.mtv.corp.google.com>
From: "Junio C Hamano" <gitster@pobox.com>
Show 21 quoted lines
> "Philip Oakley" <philipoakley@iee.org> writes:
>
>>> I think the
>>> use of "commit" in an angle-bracket-pair in the label for the
>>> section, i.e. "--fixup=<commit>", has been considered to be clear
>>> enough to tell that you can use usual extended SHA-1 syntax to
>>> specify the commit you want to talk about,
>>
>> I certainly hadn't picked up on that ability to use the extended sha1
>> syntax (specifying revisions...) here.
>
> By "has been considered", I meant that the documentation text is
> still open for improvement.  I just didn't find rewording "commit"
> with "commit revision" is that improvement we need there.
>
> Perhaps we need to have somewhere central a section that explains
> various notations used in the documentation set.  I think it is safe
> to say something like "unless otherwise qualified, <commit> (or any
> object type in an angle-bracket-pair) is used as a placeholder to
> take any acceptable way to spell object names (cf. gitrevisions for
> details)" these days [*1*].

True. I'm cautious that we may accidentally still hide it in another document that the user doesn't see when reading "this" (or any other) man page.

Your sentence is short enough to be added to those few key pages that users refer to to get them started in the right direction.

> *1* In ancient days I think some plumbing commands only took 40-hex
> object names, but as far as I know they've all been updated.
> --
Philip 
Previous: Junio C HamanoNext: Philip Oakley
Message 6 of 16 in “fixup fixup documenation”
  1. 0/3 fixup fixup documenationPhilip Oakley, Aug 14, 2016
  2. 1/3 doc: commit: --fixup/--squash can take a commit revisionPhilip Oakley, Aug 14, 2016
  3. Junio C HamanoAug 14, 2016
  4. Philip OakleyAug 14, 2016
  5. Junio C HamanoAug 14, 2016
  6. Philip OakleyAug 14, 2016
  7. Philip OakleyAug 16, 2016
  8. 3/3 doc: rebase: clarify fixup! fixup! constraintPhilip Oakley, Aug 14, 2016
  9. Junio C HamanoAug 14, 2016
  10. Philip OakleyAug 14, 2016
  11. Junio C HamanoAug 15, 2016
  12. 2/3 doc: rebase: fixup! can take an object namePhilip Oakley, Aug 14, 2016
  13. Junio C HamanoAug 14, 2016
  14. Philip OakleyAug 14, 2016
  15. Junio C HamanoAug 14, 2016
  16. Philip OakleyAug 14, 2016

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.