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

Re: [PATCH 4/3] parse-options: disallow --no-no-sth

From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
Date
Feb 29, 2012, 18:06 UTC
Message-ID
<4F4E6912.7060204@lsrfire.ath.cx>
In-Reply-To
<7vlinmy80x.fsf@alter.siamese.dyndns.org>
Am 28.02.2012 22:15, schrieb Junio C Hamano:
Show 9 quoted lines
> René Scharfe<rene.scharfe@lsrfire.ath.cx>  writes:
>
>> Now that options whose definition starts with "no-" can be negated
>> by removing said "no-", there is no need anymore to allow them to
>> be negated by adding a second "no-", which just looks silly.
>
> Thanks.  But accepting them silently and do what the user would have
> expected, especially if we do not advertise it, would not hurt anybody,
> no?

Yes, that's why I didn't include this fourth patch from the beginning. However, I understood your comment "Among the existing 13 that you listed, I do not think there is any that tempts any existing user to ask for negation with "--no-no-foo" form, so I think we should be Ok." to mean that nobody uses the double-no form of the existing negative options anyway, and therefore we should disallow it.

Please drop this extra patch if I misunderstood you, in order to keep backward compatibility.

René
Previous: Junio C HamanoNext: Junio C Hamano
Message 11 of 19 in “parse-options: no- symmetry”
  1. 0/3 parse-options: no- symmetryRené Scharfe, Feb 25, 2012
  2. 1/3 test-parse-options: convert to OPT_BOOL()René Scharfe, Feb 25, 2012
  3. 2/3 parse-options: allow positivation of options starting, with no-René Scharfe, Feb 25, 2012
  4. Junio C HamanoFeb 26, 2012
  5. Thomas RastFeb 27, 2012
  6. Junio C HamanoFeb 27, 2012
  7. René ScharfeFeb 27, 2012
  8. Junio C HamanoFeb 27, 2012
  9. 4/3 parse-options: disallow --no-no-sthRené Scharfe, Feb 28, 2012
  10. Junio C HamanoFeb 28, 2012
  11. René ScharfeFeb 29, 2012
  12. Junio C HamanoFeb 29, 2012
  13. 3/3 parse-options: remove PARSE_OPT_NEGHELPRené Scharfe, Feb 25, 2012
  14. Jeff KingFeb 27, 2012
  15. Junio C HamanoFeb 27, 2012
  16. René ScharfeFeb 27, 2012
  17. Jeff KingFeb 28, 2012
  18. 3/3 parse-options: remove PARSE_OPT_NEGHELPRené Scharfe, Feb 28, 2012
  19. Jeff KingFeb 28, 2012

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.