Re: gnulib-tool bug

2009-03-01 Thread Bruno Haible
Paolo Bonzini wrote: > FWIW, the bug mentioned earlier in the thread is already fixed in git sed. Thanks for the info. I'm re-enabling the use of the --posix option for when sed 4.2 will be released. 2009-03-01 Bruno Haible * gnulib-tool (sed): Do alias as "sed --posix" if sed's vers

Re: gnulib-tool bug

2009-02-28 Thread Paolo Bonzini
Bruno Haible wrote: > Eric Blake wrote: >>> This looks like a bug to me: \\ expands to nothing. No way to produce a >>> backslash in the replacement. >> I concur. > > Thanks. So I'm disabling --posix. Patch appended. > >> "... For each other encountered, the following >> character shall lose it

Re: gnulib-tool bug

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > > This looks like a bug to me: \\ expands to nothing. No way to produce a > > backslash in the replacement. > > I concur. Thanks. So I'm disabling --posix. Patch appended. > "... For each other encountered, the following > character shall lose its special meaning (if any).

Re: gnulib-tool bug

2009-02-27 Thread Eric Blake
Bruno Haible clisp.org> writes: > Ah! The culprit is the --posix option that gnulib-tool adds: > > This looks like a bug to me: \\ expands to nothing. No way to produce a > backslash in the replacement. I concur. > Can you confirm my interpretations? > > > POSIX says that \/ is not portable w

Re: gnulib-tool bug

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > > > /^test-freadptr.c/d > > > > ok so far. Actually, that was not ok: it should be /^test-freadptr.c$/d > > > /^/test-freadptr.c/d > > > > This should be /^\/test-freadptr.c/d Actually this should be /^\/test-freadptr.c$/d > > For me, with bash and sed-4.1.5, t

Re: gnulib-tool bug

2009-02-27 Thread Eric Blake
Bruno Haible clisp.org> writes: > > > /^/test-freadptr.c/d > > This should be /^\/test-freadptr.c/d > > This may be a difference between sh or sed versions. What's the result of these > two sets of commands for you? > > $ doubly_escaped_anchor='\\/' > $ echo 'test-freadseek.sh' | \ > sed

Re: gnulib-tool bug

2009-02-27 Thread Bruno Haible
Hi Eric, > at which point, ignore-removed contains: > test-freadptr.c > test-freadptr.sh > test-freadptr2.c > test-freadptr2.sh > test-freadseek.c > test-freadseek.sh > test-memchr2.c Looks ok. The directory part of the filename is removed at the point, because it's updating the tests/.gitignore

Re: gnulib-tool bug

2009-02-27 Thread Eric Blake
Bruno Haible clisp.org> writes: > > sed: can't find label for jump to `est-strsignal.c/d' > > Finished. > > Hmm. I thought I fixed this code already (around lines 3567..3575 of > gnulib-tool). Can you, after line 3571, make a backup copy of the files > "$tmp"/ignore-removed and "$tmp"/sed-ignore

Re: gnulib-tool bug

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > When removing a module during gnulib-tool --update, the sed script to remove > files from the test subdir is treating the t as a sed label, rather than as > part of the file name to be operated on: > > Removing file tests/test-strsignal.c (backup in tests/test-strsignal.c~) >

gnulib-tool bug

2009-02-27 Thread Eric Blake
When removing a module during gnulib-tool --update, the sed script to remove files from the test subdir is treating the t as a sed label, rather than as part of the file name to be operated on: Removing file tests/test-strsignal.c (backup in tests/test-strsignal.c~) Updating file lib/stdint.in.h