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

2016-09-24 Thread Jim Meyering
On Sat, Sep 24, 2016 at 9:22 PM, Paul Eggert wrote: > Jim Meyering wrote: >> >> Sounds like it must be something fundamental. I've just confirmed that >> at commit v2.25-100-gcd8082e `./bootstrap && ./configure && make && >> make distcheck` succeeds on both Fedora 24 and Debian unstable. >> >> Did

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

2016-09-24 Thread Paul Eggert
Jim Meyering wrote: Sounds like it must be something fundamental. I've just confirmed that at commit v2.25-100-gcd8082e `./bootstrap && ./configure && make && make distcheck` succeeds on both Fedora 24 and Debian unstable. Did you rerun ./bootstrap? Yes. I think the difference is that I'm not

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

2016-09-24 Thread Jim Meyering
On Sat, Sep 24, 2016 at 8:53 PM, Paul Eggert wrote: > Jim Meyering wrote: >> >> That is probably because you have a stray dfa.o file laying around. >> Since that corresponds to a now-VC-removed dfa.c file, "make clean" >> fails to remove the .o file. Just remove it manually and the test >> should

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

2016-09-24 Thread Paul Eggert
Jim Meyering wrote: That is probably because you have a stray dfa.o file laying around. Since that corresponds to a now-VC-removed dfa.c file, "make clean" fails to remove the .o file. Just remove it manually and the test should then pass. Right you are, thanks. Running 'make clean' fixes the p

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

2016-09-24 Thread Jim Meyering
On Sat, Sep 24, 2016 at 8:43 PM, Paul Eggert wrote: ... > PS. I updated to the latest gnulib to fix a 'make check' failure on AIX 7.1. Thank you!

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

2016-09-24 Thread Jim Meyering
On Sat, Sep 24, 2016 at 8:43 PM, Paul Eggert wrote: > After this change, 'make dist' no longer works on the current Ubuntu version > (16.04.1 LTS). The failure is as follows. I'm not quite sure how to debug > this sort of thing, as the syntax checks are a bit twisty. Although this > isn't a huge o

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

2016-09-24 Thread Paul Eggert
After this change, 'make dist' no longer works on the current Ubuntu version (16.04.1 LTS). The failure is as follows. I'm not quite sure how to debug this sort of thing, as the syntax checks are a bit twisty. Although this isn't a huge obstacle to issuing a new grep release (as one can run 'mak

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

2016-09-11 Thread Jim Meyering
On Sat, Sep 10, 2016 at 5:55 PM, Jim Meyering wrote: > On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon wrote: >> Hello Jim and all, >> >>> On Sep 10, 2016, at 14:44, Jim Meyering wrote: >>> >>> Finally, here is a similar change, but now for gnu sed. For it, I also >>> had to make some small code c

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

2016-09-10 Thread Jim Meyering
On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon wrote: > Hello Jim and all, > >> On Sep 10, 2016, at 14:44, Jim Meyering wrote: >> >> Finally, here is a similar change, but now for gnu sed. For it, I also >> had to make some small code changes to adapt to the new >> localeinfo.[ch] and to the dfasy

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

2016-09-10 Thread Assaf Gordon
Hello Jim and all, > On Sep 10, 2016, at 14:44, Jim Meyering wrote: > > Finally, here is a similar change, but now for gnu sed. For it, I also > had to make some small code changes to adapt to the new > localeinfo.[ch] and to the dfasyntax API change. I'll wait a day or so > before pushing this,

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

2016-08-17 Thread Paul Eggert
This all sounds good, thanks. Your attachment was a submodule diff (basically, a pair of git commit IDs), so not much to review. 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-curre

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

2016-08-17 Thread Jim Meyering
Now that two gnulib-using packages are using grep's DFA matcher, I am going to move that code into gnulib, so there is one clear source of truth. GNU sed began using dfa.[ch] very recently. While gawk has been copying grep's dfa.[ch] for a long time, it does not use gnulib. To that end, I've writ