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

Re: `git grep` is too picky about option parsing

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 7, 2020, 18:46 UTC
Message-ID
<xmqqa6upbgil.fsf@gitster.c.googlers.com>
In-Reply-To
<X85gMs1gPBNLff7f@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 5 quoted lines
> The problem is that it gives bundled short options a lower precedence
> than detecting possible typos against long options. My instinct is to
> say that this is wrong. We should allow valid things to work, and only
> add error heuristics if the user's request is nonsense (i.e., if one of
> the bundled options is not a valid one).
Thanks, I was coming to the same conclusion.
Show 25 quoted lines
> But that actually contradicts
> the original example given in 3a9f0f41db! There it was trying to make:
>
>   git commit -amend
>
> an error. But that's a set of valid options, the same as:
>
>   git commit -a -m end
>
> So we'd be losing that protection. Another option would be to make the
> typo-checker a little more picky:
>
>   - require more than 3 characters; this is just punting off the
>     problem, though. Doing "-line foo" is valid. So is "-linefoo", for
>     that matter, though that one would do what we want since it stops
>     being a prefix.
>
>   - be more aggressive about how much of a long option we match in the
>     prefix (at least for the typo checker). "lin" is an awfully small
>     part of "line-number". People may plausibly use "--lin" or "--line"
>     as a shortcut, but I'm not sure that merits blocking the valid
>     "-lin" for the typo-checker.
>
> Either of those would let "-amend" continue to be an error, but fix
> "-lin".

I am wondering if a rule like "you cannot concatenate a short option that takes argument with other short options" work. The problem with "-a -m end" is really that the 'm' takes arbitrary end-user input. So "commit -ave" would be fine, but "commit -ame" would not be. This would make both "-line foo" and "--linefoo" consistently invalid, but "-lin -e foo" is still OK and make the rule easier to explain.

Then we can probably lift the "more than 3 characters" heuristics, which may be a good thing independently.

Previous: Jeff KingNext: Jeff King
Message 3 of 5 in “`git grep` is too picky about option parsing”
  1. Jan EngelhardtDec 6, 2020
  2. Jeff KingDec 7, 2020
  3. Junio C HamanoDec 7, 2020
  4. Jeff KingDec 7, 2020
  5. Junio C HamanoDec 7, 2020

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.