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

Re: [PATCH] Add two grep config options

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 28, 2011, 17:12 UTC
Message-ID
<7vvcz3yxs8.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20110328115421.GA9232@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
> One shortcoming of such a scheme, though, is that it is an
> all-or-nothing proposal...

I fully agree with this assessment, and I think that was the primary reason that we rejected --plumbing / GIT_PLUMB (i.e. as too naive to be useful).

Show 5 quoted lines
> So what we really want is to let the script "allow" certain options from
> the user's preferences. This could be done easily with individual config
> options, like:
>
>   git --allow=grep.extended grep ...

I think this is probably the right thing to do _if_ we wanted to add such a configuration variable and give a way to let script writers protect themselves.

But would any user go all that trouble, just not to say "-nE" from the command line (or use an alias that was designed not to crash with scripts)?

Previous: Jeff KingNext: Jeff King
Message 7 of 19 in “Add two grep config options”
  1. Add two grep config optionsJoe Ratterman, Mar 25, 2011
  2. Junio C HamanoMar 25, 2011
  3. Michael J GruberMar 28, 2011
  4. Jeff KingMar 28, 2011
  5. Michael J GruberMar 28, 2011
  6. Jeff KingMar 28, 2011
  7. Junio C HamanoMar 28, 2011
  8. Jeff KingMar 28, 2011
  9. Junio C HamanoMar 28, 2011
  10. Joe RattermanMar 28, 2011
  11. Bert WesargMar 28, 2011
  12. Joe RattermanMar 28, 2011
  13. Junio C HamanoMar 28, 2011
  14. grep: allow -E and -n to be turned on by default via configurationJunio C Hamano, Mar 28, 2011
  15. Junio C HamanoMar 28, 2011
  16. Joe RattermanMar 28, 2011
  17. Junio C HamanoMar 28, 2011
  18. Joe RattermanMar 29, 2011
  19. grep: allow -E and -n to be turned on by default via configurationJoe Ratterman, Mar 30, 2011

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.