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

Re: [PATCH] Documentation: Link git-ls-files to core.quotePath variable.

From
Andreas Heiduk <asheiduk@gmail.com>
Date
Feb 22, 2017, 01:20 UTC
Message-ID
<61a3e1a7-e70c-cb68-184f-95dff28d1271@gmail.com>
In-Reply-To
<xmqqh93nfeyj.fsf@gitster.mtv.corp.google.com>
Am 21.02.2017 um 21:48 schrieb Junio C Hamano:
Show 19 quoted lines
> 
> I was waiting for others to comment on this patch but nobody seems
> to be interested.  Which is a bit sad, as this may not be a bad
> idea.
> 
> If we refer to core.quotePath, the mention of control characters
> being quoted can also be omitted, I think, as that is part of what
> appears in the description of core.quotePath variable.
> 
> Alternatively, instead of referring to another page, we can spend
> the additional lines to say what is more interesting to most of the
> readers from that page, e.g.
> 
>     When `-z` option is not used, a pathname with "unusual" characters
>     in it is quoted by enclosing it in a double-quote pair and with
>     backslashes the same way strings in C source code are quoted.  By
>     setting core.quotePath configuration to false, the bytes whose
>     values are higher than 0x80 are output verbatim.
>

Without `-z` but with core.quotePath=false the path may still be surrounded with double-quotes if it contains control characters (and some more). The documentation in `core.quotePath` mentions this, your "inline" alternative does not.

I will send second patch. :-)
Previous: Junio C HamanoNext: Andreas Heiduk
Message 3 of 18 in “Documentation: Link git-ls-files to core.quotePath variable.”
  1. Documentation: Link git-ls-files to core.quotePath variable.Andreas Heiduk, Feb 19, 2017
  2. Junio C HamanoFeb 21, 2017
  3. Andreas HeidukFeb 22, 2017
  4. Andreas HeidukFeb 22, 2017
  5. Junio C HamanoFeb 22, 2017
  6. Philip OakleyFeb 22, 2017
  7. Junio C HamanoFeb 22, 2017
  8. Andreas HeidukFeb 22, 2017
  9. 0/2 Documentation: Link git-ls-files to core.quotePath variable.Andreas Heiduk, Feb 24, 2017
  10. 1/2 Documentation: Improve description for core.quotePathAndreas Heiduk, Feb 24, 2017
  11. Jakub NarębskiFeb 24, 2017
  12. Andreas HeidukFeb 28, 2017
  13. 2/2 Documentation: Link descriptions of -z to core.quotePathAndreas Heiduk, Feb 24, 2017
  14. Jakub NarębskiFeb 24, 2017
  15. Andreas HeidukFeb 28, 2017
  16. Junio C HamanoFeb 28, 2017
  17. Andreas HeidukFeb 28, 2017
  18. Junio C HamanoFeb 28, 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.