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

Re: [PATCH v1] config: add '--sources' option to print the source of a config value

From
Sebastian Schuberth <sschuberth@gmail.com>
Date
Feb 8, 2016, 11:25 UTC
Message-ID
<56B87B46.6050304@gmail.com>
In-Reply-To
<20160205135855.GA19154@sigill.intra.peff.net>
On 2/5/2016 14:58, Jeff King wrote:
Show 9 quoted lines
> Yeah, I agree it's unlikely. And the output is already ambiguous, as the
> first field could be a blob (though I guess the caller knows if they
> passed "--blob" or not). If we really wanted an unambiguous output, we
> could have something like "file:...", "blob:...", etc. But that's a bit
> less readable for humans, and I don't think solves any real-world
> problems.
>
> So I think it would be OK to use "<command line>" here, as long as the
> token is documented.

Thinking about it again, I actually do like Peff's prefix solution better. It would solve the real-world problem that my proposed "<command line>" marker could in fact be a file name.

Regards, Sebastian

Previous: Jeff KingNext: Jeff King
Message 12 of 14 in “config: add '--sources' option to print the source of a config value”
  1. config: add '--sources' option to print the source of a config valuelarsxschneider@gmail.com, Feb 5, 2016
  2. Sebastian SchuberthFeb 5, 2016
  3. Jeff KingFeb 5, 2016
  4. Lars SchneiderFeb 7, 2016
  5. Sebastian SchuberthFeb 8, 2016
  6. Jeff KingFeb 8, 2016
  7. Jeff KingFeb 5, 2016
  8. Sebastian SchuberthFeb 5, 2016
  9. Jeff KingFeb 5, 2016
  10. Lars SchneiderFeb 7, 2016
  11. Jeff KingFeb 8, 2016
  12. Sebastian SchuberthFeb 8, 2016
  13. Jeff KingFeb 8, 2016
  14. Lars SchneiderFeb 7, 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.