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

Re: t7810.94 (git-grep -E -F -G) never passed

From
Brian Gernhardt <brian@gernhardtsoftware.com>
Date
May 25, 2011, 12:48 UTC
Message-ID
<1AEF57D5-49E0-4F37-BE8D-DEBD25965D83@gernhardtsoftware.com>
In-Reply-To
<4DDCD845.1030800@drmicha.warpmail.net>
On May 25, 2011, at 6:21 AM, Michael J Gruber wrote:
> "Never" is a strong statement when this test always passed for everyone
> else running tests, assuming they would have reported otherwise. What's
> your system?
My apologies.  It was late and I wasn't explicit enough.  What I meant was:
"This test has failed on my system ever since the commit that introduced it."
My system is running OS X 10.6.7.
Show 9 quoted lines
> reg-ex.info says:
> 
> "The other BRE metacharacters require a backslash to give them their
> special meaning."
> 
> "Some implementations support \? and \+... but \? and \+ are not part of
> the POSIX standard."
> 
> So I guess we're going with "some".
Some would be right.  I tend to find that the BSD (and BSD-like) libraries on OS X are more strict and have less features than the GNU equivalents.  (The fact that OS X's libreadline is actually the only mostly compatible libedit drives me batty sometimes.)
This does leave us with the question of what to do with the test.  Perhaps a pattern like "a*+b" to match "aa+b"?  Plain + should always match just + in a BRE, even if \+ is special, and the * makes sure that it is a regex instead of plain string matching.
~~ Brian
Previous: Michael J GruberNext: Michał Kiedrowicz
Message 3 of 4 in “t7810.94 (git-grep -E -F -G) never passed”
  1. Brian GernhardtMay 25, 2011
  2. Michael J GruberMay 25, 2011
  3. Brian GernhardtMay 25, 2011
  4. Michał KiedrowiczMay 25, 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.