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

Re: [RFC PATCH] parse-options: disallow double-negations of options starting with no-

From
Jeff King <peff@peff.net>
Date
Apr 19, 2017, 21:24 UTC
Message-ID
<20170419212423.i6jnu3q2npnbq2gz@sigill.intra.peff.net>
In-Reply-To
<CA+P7+xpahRim9H1cHuHmzjGggYt2qbvxrDYyi+ri0QsDEUAkcw@mail.gmail.com>
On Wed, Apr 19, 2017 at 02:22:47PM -0700, Jacob Keller wrote:
Show 10 quoted lines
> > I think the breakage in that case would be caused by "--no-stage" taking
> > over "--stage" as well. And your patch doesn't change that; it happened
> > already in 2012.
> >
> > Your patch only affects the --no-no- form, which I think we would never
> > want. I don't think it needs callers to trigger it explicitly.
> 
> Right, I was just thinking in the weird cause were we *do* have a
> "no-option" that does want the "no-no-option" to negate it. Maybe this
> isn't ever a thing and we don't need to worry at all..?
Yeah, my assumption is that we don't need to worry about that case.
-Peff
Previous: Jacob Keller
Message 9 of 9 in “parse-options: disallow double-negations of options starting with no-”
  1. parse-options: disallow double-negations of options starting with no-Jacob Keller, Apr 19, 2017
  2. Ævar Arnfjörð BjarmasonApr 19, 2017
  3. René ScharfeApr 19, 2017
  4. Jeff KingApr 19, 2017
  5. Jacob KellerApr 19, 2017
  6. Jeff KingApr 19, 2017
  7. Jacob KellerApr 19, 2017
  8. Jacob KellerApr 19, 2017
  9. Jeff KingApr 19, 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.