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

Re: [PATCH] git grep: Add "-Z/--null" option as in GNU's grep.

From
Raphael Zimmerer <killekulla@rdrz.de>
Date
Oct 1, 2008, 13:23 UTC
Message-ID
<20081001132351.GA20659@rdrz.de>
In-Reply-To
<alpine.DEB.1.00.0810011514530.22125@pacific.mpi-cbg.de.mpi-cbg.de>
On Wed, Oct 01, 2008 at 03:15:34PM +0200, Johannes Schindelin wrote:
Show 11 quoted lines
> On Wed, 1 Oct 2008, Raphael Zimmerer wrote:
> > On Tue, Sep 30, 2008 at 04:16:19PM -0700, Shawn O. Pearce wrote:
> > > Elsewhere in Git we call this "-z", like "git ls-tree -z", "git log 
> > > -z".  Should we match grep or git convention here?
> > 
> > I'd tend to grep's convention, as most options of git-grep mimic those 
> > of grep. grep uses "-z" for \0 on _input_, so that would be very 
> > confusing for grep users...
> 
> I tend to disagree.  Git is _already_ perceived as too heterogenous, and 
> we should not add to that pile.

How about discarding "-Z" from my patch, and only leave in "--null"? That removes ambiguity _and_ is compatible to GNU grep.

- Raphael
Previous: Johannes SchindelinNext: Shawn O. Pearce
Message 5 of 11 in “git grep: Add "-Z/--null" option as in GNU's grep.”
  1. git grep: Add "-Z/--null" option as in GNU's grep.Raphael Zimmerer, Sep 30, 2008
  2. Shawn O. PearceSep 30, 2008
  3. Raphael ZimmererSep 30, 2008
  4. Johannes SchindelinOct 1, 2008
  5. Raphael ZimmererOct 1, 2008
  6. Shawn O. PearceOct 1, 2008
  7. Raphael ZimmererOct 1, 2008
  8. Pierre HabouzitOct 1, 2008
  9. Shawn O. PearceOct 1, 2008
  10. git grep: Add "-z/--null" option as in GNU's grep.Raphael Zimmerer, Oct 1, 2008
  11. Shawn O. PearceOct 1, 2008

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.