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

Re: [PATCH v3 1/3] bundle: framework for options before bundle file

From
Jeff King <peff@peff.net>
Date
Nov 11, 2019, 03:54 UTC
Message-ID
<20191111035420.GA6379@sigill.intra.peff.net>
In-Reply-To
<20191110204126.30553-1-robbat2@gentoo.org>
On Sun, Nov 10, 2019 at 12:41:24PM -0800, Robin H. Johnson wrote:
> Make it possible for any of the git-bundle subcommands to include
> options:
> - before the sub-command
> - after the sub-command, before the bundle filename

On reading your subject and the start of the commit message, I thought you mean that you meant to allow both:

  git bundle --foo create
and
  git bundle create --foo

But looking at the patch, this is about creating two separate sets of options, one for all sub-commands, and one for each individual sub-command. That makes sense to me. I don't know if it's worth trying to spell that out more explicitly in the commit message.

-Peff
Previous: Junio C HamanoNext: Johannes Schindelin
Message 10 of 13 in “bundle: framework for options before bundle file”
  1. 1/3 bundle: framework for options before bundle fileRobin H. Johnson, Nov 10, 2019
  2. 3/3 bundle-verify: add --quietRobin H. Johnson, Nov 10, 2019
  3. Jeff KingNov 11, 2019
  4. 2/3 bundle-create: progress output controlRobin H. Johnson, Nov 10, 2019
  5. Jeff KingNov 11, 2019
  6. Robin H. JohnsonNov 11, 2019
  7. Jeff KingNov 11, 2019
  8. Junio C HamanoNov 11, 2019
  9. Junio C HamanoNov 11, 2019
  10. Jeff KingNov 11, 2019
  11. Johannes SchindelinNov 11, 2019
  12. Junio C HamanoNov 11, 2019
  13. Johannes SchindelinNov 12, 2019

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.