On Sat, Sep 10, 2016 at 5:55 PM, Jim Meyering <j...@meyering.net> wrote: > On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon <assafgor...@gmail.com> wrote: >> Hello Jim and all, >> >>> On Sep 10, 2016, at 14:44, Jim Meyering <j...@meyering.net> 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.
