Web lists-archives.com

Re: [PATCH 11/12] grep: change the internal PCRE code & header names to be PCRE1




On Tue, Apr 11, 2017 at 12:45:55PM +0200, Ævar Arnfjörð Bjarmason wrote:

> On Tue, Apr 11, 2017 at 12:37 PM, Jeff King <peff@xxxxxxxx> wrote:
> > On Sat, Apr 08, 2017 at 01:25:05PM +0000, Ævar Arnfjörð Bjarmason wrote:
> >
> >> diff --git a/builtin/grep.c b/builtin/grep.c
> >> index 9478ab5dff..dffb9743b8 100644
> >> --- a/builtin/grep.c
> >> +++ b/builtin/grep.c
> >> @@ -490,7 +490,7 @@ static void compile_submodule_options(const struct grep_opt *opt,
> >>       case GREP_PATTERN_TYPE_FIXED:
> >>               argv_array_push(&submodule_options, "-F");
> >>               break;
> >> -     case GREP_PATTERN_TYPE_PCRE:
> >> +     case GREP_PATTERN_TYPE_PCRE1:
> >>               argv_array_push(&submodule_options, "-P");
> >>               break;
> >
> > Hmm. This isn't a problem yet, but wouldn't this need to pass some
> > pcre1-specific option instead of just "-P"?
> 
> Yes, this is a bug. I'll need to add a git_options along with
> submodule_options and pass -c grep.patternType=....

Maybe that's an indication we should have --pcre1-regexp and
--pcre2-regexp, so we don't have to resort to config tweaking.

-Peff