bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: bug#24256: new module for grep's DFA matcher


From: Jim Meyering
Subject: Re: bug#24256: new module for grep's DFA matcher
Date: Thu, 8 Sep 2016 22:27:12 -0700

On Wed, Aug 17, 2016 at 5:11 PM, Jim Meyering <address@hidden> wrote:
> On Wed, Aug 17, 2016 at 2:14 PM, Paul Eggert <address@hidden> wrote:
>> This all sounds good, thanks. Your attachment was a submodule diff
>> (basically, a pair of git commit IDs), so not much to review.
>
> Oops. Wrong working directory. Thanks.
> I've attached the real diff, this time.
>
>> I have some dfa.c-related changes pending. I assume it'll be OK for me to
>> commit them to grep as needed, and that when you make the switch you'll grab
>> the then-current version from grep. If you need me to get out of the water
>> around that time, please just let me know.
>
> As long as there are still soon-to-be-committed changes for grep's
> dfa.[ch], I'm happy to wait.

I have sync'd the latest grep/dfa bits into my gnulib
dfa-module-adding patch, and updated the grep-side patch that makes
grep use that new module. Attached for anyone who cares to review.
If you're interested in trying it, you should first "cd" into grep's
gnulib directory and use "git am gnulib*.diff" to apply the first
patch there. Then "cd .. && git ci -m 'commit local gnulib+dfa change'
-a", then apply the grep patch and rerun the usual ./bootstrap &&
./configure ... && make && make test

Attachment: gnulib-new-dfa-module.diff
Description: Text document

Attachment: grep-use-gnulib-s-new-dfa-module.diff
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]