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

Re: [RFC] The correct and consistent alternative to quote a command ?

From
Stefan Beller <sbeller@google.com>
Date
Aug 2, 2017, 17:32 UTC
Message-ID
<CAGZ79kbCg+xdVE1U0dv8MJ6vUuWvraWQCMSzKOTBsOvtdSvkGg@mail.gmail.com>
In-Reply-To
<1501676577.1957.4.camel@gmail.com>

On Wed, Aug 2, 2017 at 5:22 AM, Kaartic Sivaraam <kaarticsivaraam91196@gmail.com> wrote:

Show 12 quoted lines
> On Tue, 2017-08-01 at 10:38 -0700, Stefan Beller wrote:
>> On Tue, Aug 1, 2017 at 8:59 AM, Kaartic Sivaraam wrote:
>> > I assume you mean the consistency in quoting i.e., you're expecting the
>> > patch to use (") instead of (') for quoting. Correct me, if I'm wrong.
>>
>> Actually I did not imply any expectation on the outcome, because I
>> do not know what the consistency end game looks like for this issue.
>>
>> So maybe we'd want to go with the currently most used way?
> On scanning through Documentation/SubmittingPatches, I saw that most of
> the commands were quoted using double quotes and chose to use it for
> this patch. I'm not sure if it's the right one to use.
Thanks for checking. I think it is a reasonable thing to do then.
Show 9 quoted lines
>
> Changing the subject of this mail; hoping some one who has thoughts
> about this would dive-in.
>
>> (Are there only three? (a) with dash, (b) with single quotes and
>> (c) with double quotes?)
>
> --
> Kaartic
Previous: Junio C HamanoNext: Kaartic Sivaraam
Message 12 of 23 in “doc: fix small issues in SubmittingPatches”
  1. 1/2 doc: fix small issues in SubmittingPatchesKaartic Sivaraam, Jul 30, 2017
  2. 2/2 doc: add another way to identify if a patch has been mergedKaartic Sivaraam, Jul 30, 2017
  3. Philip OakleyJul 30, 2017
  4. Kaartic SivaraamJul 30, 2017
  5. 1/2 doc: fix small issues in SubmittingPatchesKaartic Sivaraam, Jul 30, 2017
  6. Stefan BellerJul 31, 2017
  7. Junio C HamanoJul 31, 2017
  8. Kaartic SivaraamAug 1, 2017
  9. Stefan BellerAug 1, 2017
  10. Kaartic SivaraamAug 2, 2017
  11. Junio C HamanoAug 2, 2017
  12. Stefan BellerAug 2, 2017
  13. Kaartic SivaraamAug 3, 2017
  14. 1/2 doc: fix small issues in SubmittingPatchesKaartic Sivaraam, Aug 1, 2017
  15. 2/2 doc: add another way to identify if a patch has been mergedKaartic Sivaraam, Aug 1, 2017
  16. Stefan BellerAug 1, 2017
  17. Kaartic SivaraamAug 2, 2017
  18. Junio C HamanoAug 2, 2017
  19. Junio C HamanoAug 2, 2017
  20. Stefan BellerAug 2, 2017
  21. Kaartic SivaraamAug 7, 2017
  22. Kaartic SivaraamAug 7, 2017
  23. Philip OakleyAug 1, 2017

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.