On 17/01/17 01:13, J William Piggott wrote: > > Hello all, I am not subscribed so please cc me. > >>From my local gnulib git repo when I run: > ./gnulib-tool --create-testdir --dir=/tmp/test3 parse-datetime > > It fails with: > /bin/sh ../build-aux/ylwrap ./parse-datetime.y \ > y.tab.c parse-datetime.c \ > y.tab.h parse-datetime.h \ > y.output parse-datetime.output \ > -- bison -y && \ > 's|".*/parse-datetime.y"|"parse-datetime.y"|' \ > < parse-datetime.c > parse-datetime.c-t && \ > mv parse-datetime.c-t ./parse-datetime.c > /bin/sh: line 6: s|".*/parse-datetime.y"|"parse-datetime.y"|: No such file or > directory > make: *** [parse-datetime.c] Error 127 > > --- > > Because SED is not being declared. All is well if I run it like this: > SED=/usr/bin/sed ./gnulib-tool --create-testdir --dir=/tmp/test3 > parse-datetime > > > It seems to be caused by this recent commit: > > commit 73ecb533f82b613245451f05d12726a41ceca95d > Author: Pádraig Brady <[email protected]> > Date: Tue Jan 3 00:52:13 2017 +0000 > > parse-datetime: fix generated paths for coverage files > > 8< --- > > - mv parse-datetime.c parse-datetime.c-t && \ > + $(SED) 's|".*/parse-datetime.y"|"parse-datetime.y"|' \ > + < parse-datetime.c > parse-datetime.c-t && \ > > 8< --- > > Doesn't the AC_PROG_SED macro have to be called somewhere to set SED? > I don't see it. > > Or, perhaps it should just be sed instead of $(SED)? That is the only > place in the Makefile that $(SED) is used.
Fixed in http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=440ea67aa sorry about that
