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: spurious "(cached)" output from AC_CACHE_VAL

2009-02-27 Thread Bruno Haible
[Adding bug-autoconf list to CC.] For the context: in gnulib, I today added a macro: # gl_CACHE_VAL_SILENT(cache-id, command-to-set-it) # is like AC_CACHE_VAL(cache-id, command-to-set-it), except that it does not # output a spurious "(cached)" mark in the midst of other configure output. # This m

Re: [PATCH] filevercmp: fix regular expression of file suffix

2009-02-27 Thread Kamil Dudka
Hello Bruno, thanks for review! On Friday 27 of February 2009 17:02:56 Bruno Haible wrote: > As I understand it, the patch does not fix two things that were mentioned > negatively by Sven: > - that hidden files appear at the end, rather than at the beginning, Well, I don't know which order is "

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: Building uninorm tests fail with external gettext

2009-02-27 Thread Bruno Haible
Tom G. Christensen wrote: > Trying to build the daily snapshot on IRIX 6.5.30 I run into problems > when building with external gettext: > > cc -c99 -g -L/usr/nekoware/lib -Wl,-rpath,/usr/nekoware/lib > -Wl,--as-needed -o test-u32-nfc-big test-u32-nfc-big.o test-u32-normalize > -big.o ../gllib/l

Re: fts: make find *much* faster on dirent.d_type-challenged FS

2009-02-27 Thread Tom G. Christensen
On Fri, Feb 13, 2009 at 11:21:11AM +0100, Simon Josefsson wrote: > How does that work? Do most compilers out there really support > decl-after-stmt (not likely?!)? > MIPSpro on IRIX in C89 mode doesn't like decl-after-stmt. The MIPSpro 7.4.x releases has a C99 mode where decl-after-stmt is not a

Building uninorm tests fail with external gettext

2009-02-27 Thread Tom G. Christensen
Trying to build the daily snapshot on IRIX 6.5.30 I run into problems when building with external gettext: cc -c99 -g -L/usr/nekoware/lib -Wl,-rpath,/usr/nekoware/lib -Wl,--as-needed -o test-u32-nfc-big test-u32-nfc-big.o test-u32-normalize -big.o ../gllib/libgnu.a -lm (null): WARNING 1 : Unk

Re: computing long double NaN on Irix 6.5

2009-02-27 Thread Tom G. Christensen
On Thu, Feb 26, 2009 at 09:17:28PM -0700, Eric Blake wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > According to Eric Blake on 2/26/2009 4:57 PM: > > I've done some investigation into failures of test-frexpl, test-isnanl, and > > test-vasnprintf-posixt-ldexpl.c:130: assertion failed

Re: compilation of findutils-4.4.0 with gcc-4.4.0-20090225 -- error with wctype.h in gnulib code

2009-02-27 Thread Bruno Haible
Hello, Frank Erens wrote: > I'm trying to compile findutils-4.4.0, Please, on what system? You mentioned the compiler you are using, but the platform (`uname -srm`) is also mandatory information. > but get the following errors from make: > ... > depbase=`echo mbchar.o | sed 's|[^/]*$|.deps/&|;s

Re: coreutils-7.1 - Compile Issue - Assuming defined preprocessor symbols have a value

2009-02-27 Thread Peter Bray
Eric, > Thanks for the report. This is indeed a bug in the gnulib > replacement header. Does replacing '_LP64' with 'defined > _LP64' fix things? An elegant and functional solution, both gmake and gmake check are fine (only tested Solaris 10 Update 6 / gcc-4.1.2 / SPARC 64-bit) Regards, Pet

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: spurious roundl output (was: computing long double NaN on Irix 6.5)

2009-02-27 Thread Eric Blake
Bruno Haible clisp.org> writes: > > A variable with _cv_ in the name is still cached, regardless of whether > > you used AC_CACHE_VAL; all AC_CACHE_VAL does is wrap additional logic to > > produce output, thus it should not be used in situations where you don't > > want the side effect of output.

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~) >

Re: spurious roundl output (was: computing long double NaN on Irix 6.5)

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > However, the spurious output is still > misleading, so what do you think of this patch to clean it up? I have not > pushed it yet... > > A variable with _cv_ in the name is still cached, regardless of whether > you used AC_CACHE_VAL; all AC_CACHE_VAL does is wrap additional lo

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

compilation of findutils-4.4.0 with gcc-4.4.0-20090225 -- error with wctype.h in gnulib code

2009-02-27 Thread Frank Erens
Hello, I'm trying to compile findutils-4.4.0, but get the following errors from make: - seysa...@localhost:/mnt/lfs/Sources/findutils-4.4.0$ make make all-recursive make[1]: Entering directory `/mnt/lfs/Sources/findu

Re: [PATCH] filevercmp: fix regular expression of file suffix

2009-02-27 Thread Bruno Haible
Hello Kamil, > as reported by Sven Joachim at > http://lists.gnu.org/archive/html/bug-coreutils/2009-02/msg00264.html > current filevercmp implementation does not work well for hidden files. > Attached patch fixes the strange behavior. As I understand it, the patch does not fix two things that

Re: coreutils-7.1 - Compile Issue - Assuming defined preprocessor symbols have a value

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > > In lib/inttypes.in.h the following line appears 4 times: > > > > # if (@APPLE_UNIVERSAL_BUILD@ ? _LP64 : @INT64_MAX_EQ_LONG_MAX@) > > > > The problem arises if _LP64 is defined but does not have a value. > > Thanks for the report. This is indeed a bug in the gnulib repl

Re: %.1s format with vasnprintf reads more than one byte from argument

2009-02-27 Thread Eric Blake
Eric Blake byu.net> writes: > +# printf.m4 serial 32 > dnl Copyright (C) 2003, 2007-2009 Free Software Foundation, Inc. > dnl This file is free software; the Free Software Foundation > dnl gives unlimited permission to copy and/or distribute it, > @@ -653,13 +653,19 @@ int main () > { >ch

Re: gcc on Irix 6.5

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > > but this is due to a gcc bug: If floorl(-0.7L) is executed before > > floorl(-1.0L), and both calls are compiled by gcc - regardless whether > > in the same function, in different functions, or even in different > > compilation > > units -, then the floorl(-1.

Re: fseek/ftell tests on platforms with bad ungetc

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > Would someone with access to a system that has previously been failing the > ftell/fseek tests (older glibc, HPUX, and perhaps other old platforms) please > check this patch? On HP-UX 11.11, I get these test results: PASS: test-fflush PASS: test-fflush2.sh PASS: test-fseeko.

Re: coreutils-7.1 - Compile Issue - Assuming defined preprocessor symbols have a value

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 [adding bug-gnulib] According to Peter Bray on 2/27/2009 7:10 AM: > Hi All, > > I ran into a small problem compiling coreutils-7.1 on Solaris 10. The > problem only occurred on Solaris 10 SPARC in 64-bit mode, and not Solaris 10 > x86 in 64-bit,

[PATCH] filevercmp: fix regular expression of file suffix

2009-02-27 Thread Kamil Dudka
Hello, as reported by Sven Joachim at http://lists.gnu.org/archive/html/bug-coreutils/2009-02/msg00264.html current filevercmp implementation does not work well for hidden files. Attached patch fixes the strange behavior. It does not affect visible files. The only thing changed is the regular e

Re: computing long double NaN on Irix 6.5

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 2/27/2009 6:22 AM: [adding bug-autoconf, about an Irix 6.5 situation] >> After the patch, under gcc 3.4.3, 2 tests still fail (probably due to gcc >> mishandling Irix's weird notion of a 128-bit long double formed by addi

spurious roundl output (was: computing long double NaN on Irix 6.5)

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 2/27/2009 6:22 AM: >> (cached) (cached) checking for signbit macro... no >> checking for signbit compiler built-ins... no > > This comes from the gl_FUNC_FLOORL_LIBS and gl_FUNC_CEILL_LIBS macros, which > are > invoked by

latest round of cygwin 1.7 functions

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 9 more wide-character POSIX functions (plus the BSD reallocf, not documented by gnulib) have been added to cygwin 1.7 since my last update. Committing: - -- Don't work too hard, make some time for fun as well! Eric Blake e...@byu.net ---

Re: computing long double NaN on Irix 6.5

2009-02-27 Thread Bruno Haible
Hi Eric, > Before the patch, gcc gave 9 of 16 tests failing on assertions, all > dealing with NaN, when I ran: > > gnulib-tool --with-tests --test isfinite isnan ceill floorl frexpl isnanl > ldexpl roundl truncl > > After the patch Nice improvement! > 2 tests still fail For me: - with CC=

Re: %.1s format with vasnprintf reads more than one byte from argument

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 2/27/2009 4:22 AM: > Eric Blake wrote: >>> if (sprintf (buf, "%ls", wstring) != -1) >>> return 2; >>> because it does not fail on any known platform. >> Actually, it DOES fail, on cygwin 1.7.0. But that is due to

Re: %.1s format with vasnprintf reads more than one byte from argument

2009-02-27 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Eric Blake on 2/26/2009 7:49 PM: > According to Bruno Haible on 2/26/2009 6:13 PM: >> I removed the part of the test that should normally always fail: >> if (sprintf (buf, "%ls", wstring) != -1) >> return 2; >> because it does no

Re: %.1s format with vasnprintf reads more than one byte from argument

2009-02-27 Thread Bruno Haible
Eric Blake wrote: > > if (sprintf (buf, "%ls", wstring) != -1) > > return 2; > > because it does not fail on any known platform. > > Actually, it DOES fail, on cygwin 1.7.0. But that is due to a bug in > cygwin wctomb[1] So, activating the *printf replacements for this bug would not he