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

[Jim Meyering]

On Sat, Sep 10, 2016 at 5:55 PM, Jim Meyering <address@hidden> wrote:
> On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon <address@hidden> wrote:
>> Hello Jim and all,
>>
>>> On Sep 10, 2016, at 14:44, Jim Meyering <address@hidden> 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, just in case:
>>> <sed-use-gnulib-dfa-module.diff>
>>
>> for sed,
>> It seems that pulling the latest gnulib (the first out of the two commits in 
>> the attached diff) causes the following link error (at least on mac os x):
>>
>>   [...]
>>   make[2]: Nothing to be done for `all'.
>>   Making all in .
>>     CCLD     sed/sed
>>   Undefined symbols for architecture x86_64:
>>     "_strverscmp", referenced from:
>>         _compile_program in sed_sed-compile.o
>>   ld: symbol(s) not found for architecture x86_64
>>   clang: error: linker command failed with exit code 1 (use -v to see 
>> invocation)
>>   make[2]: *** [sed/sed] Error 1
>>   make[1]: *** [all-recursive] Error 1
>
> Thanks for reporting that.
> I've pushed the attached to gnulib to address that:

Then, I rebased the sed commits on top of latest gnulib and pushed those, too.