RE: source(builtin) and read(2)

2007-04-03 Thread Schwarz, Konrad
> -Original Message- > From: Geoff Clare [mailto:[EMAIL PROTECTED] > Sent: Monday, April 02, 2007 10:22 AM > To: bug-gnulib; Matthew Woehlke; [EMAIL PROTECTED] > Subject: Re: source(builtin) and read(2) > I'll submit a defect report. Perhaps it would be possible to typedef ssize_t to ptr

Re: new module 'iconv_open'

2007-04-03 Thread Simon Josefsson
Bruno Haible <[EMAIL PROTECTED]> writes: > John Darrington wrote: >> ./configure could show a warning if gperf is not found or is too old. > > What do you gain by knowing this at configure time, rather than at "make" > time, a little later? To proceed with the build, you need to install gperf > an

Re: test-stat-time failure on FreeBSD 6

2007-04-03 Thread James Youngman
On 4/3/07, Bruno Haible <[EMAIL PROTECTED]> wrote: Hello James, On FreeBSD/x86 6.1, on a NFS mount, the test-stat-time test fails: Sorry. For some reason the sanity check on the value of st_birthtim.tv_sec ended up in the wrong arm of an #if and was therefore not performed. Here is a patch

Re: seekable stdin test failure on OS X

2007-04-03 Thread Gary V. Vaughan
Hi Eric, On 2 Apr 2007, at 18:15, Eric Blake-1 wrote: Even on Linux, the fflush man page doesn't mention input streams: http://www.die.net/doc/linux/man/man3/fflush.3.html So it seems that using fflush(stdin) is probably the wrong thing to do altogether for a portable application. On the

Re: missing dependency in *printf

2007-04-03 Thread Gary V. Vaughan
Hi Eric, Please either, indent patches so that gpg doesn't escape leading '-' signs, or use S/MIME for attaching as separate gpg signature. Either would allow feeding a saved copy of emails containing a diff to GNU patch (which copes well with consistent indentation) On 3 Apr 2007, at 04

Re: missing dependency in *printf

2007-04-03 Thread Simon Josefsson
"Gary V. Vaughan" <[EMAIL PROTECTED]> writes: > Hi Eric, > > Please either, indent patches so that gpg doesn't escape leading '-' > signs, or use S/MIME for attaching as separate gpg signature. I think you meant PGP/MIME (RFC3156), S/MIME doesn't use PGP at all, but yes, I agree with the suggesti

Re: group modules into subdirectories

2007-04-03 Thread Simon Josefsson
Simon Josefsson <[EMAIL PROTECTED]> writes: > Bruno Haible <[EMAIL PROTECTED]> writes: > >> Simon Josefsson wrote: >>> > I propose to move these modules to modules/crypto/ >>> >>> I've done that now. >> >> May I ask to put the 'memxor' module back to the top level? It is an >> independently usefu

Re: missing dependency in *printf

2007-04-03 Thread Gary V. Vaughan
Hi Simon, On 3 Apr 2007, at 11:22, Simon Josefsson wrote: "Gary V. Vaughan" <[EMAIL PROTECTED]> writes: Please either, indent patches so that gpg doesn't escape leading '-' signs, or use S/MIME for attaching as separate gpg signature. I think you meant PGP/MIME (RFC3156), S/MIME doesn't use P

Add 'Required Tools' to README?

2007-04-03 Thread Simon Josefsson
Bruno Haible <[EMAIL PROTECTED]> writes: > Eric Blake wrote: >> gperf --version >> GNU gperf 2.7.2 > > Come on! gperf 3.0.1 was released four years ago, and you are still using its > seven year old predecessor? I think some of the problems here could be fixed by noting what tools are required by

Re: source(builtin) and read(2)

2007-04-03 Thread Paul Eggert
"Schwarz, Konrad" <[EMAIL PROTECTED]> writes: > The argument that since ptrdiff_t is intended to represent the > difference of pointers makes it somehow unsuitable to represent the > result of read()/write() is missing the point: on any useful ABI, > ptrdiff_t can represent the size of any object,

gnulib tests vs. LDADD

2007-04-03 Thread Eric Blake
CVS findutils uses gnulib-tool --with-tests when importing. But on cygwin, compilation then fails in the gnulib-created directory: gcc -g2 -Wall -Werror -o test-dirname.exe test- dirname.o ../gnulib/lib/libgnulib.a ../gnulib/lib/libgnulib.a(xalloc-die.o): In function `xalloc_die': /home/ebla

stdint and cygwin bug

2007-04-03 Thread Eric Blake
cygwin currently defines wint_t as unsigned int (legal per POSIX), but then tries to use INT_MIN and INT_MAX for WINT_MIN and WINT_MAX. I've reported this as a bug to the cygwin list. In the meantime, is it worth fixing the stdint module to detect this brokenness, and redefine WINT_MIN and MAX

Re: test-stat-time failure on FreeBSD 6

2007-04-03 Thread Paul Eggert
"James Youngman" <[EMAIL PROTECTED]> writes: > Sorry. For some reason the sanity check on the value of > st_birthtim.tv_sec ended up in the wrong arm of an #if and was > therefore not performed. Thanks, I installed this slightly-different patch. 2007-04-03 James Youngman <[EMAIL PROTECTED]>

RE: source(builtin) and read(2)

2007-04-03 Thread Hartmut Brandt
On Tue, 3 Apr 2007, Schwarz, Konrad wrote: SK>> -Original Message- SK>> From: Geoff Clare [mailto:[EMAIL PROTECTED] SK>> Sent: Monday, April 02, 2007 10:22 AM SK>> To: bug-gnulib; Matthew Woehlke; [EMAIL PROTECTED] SK>> Subject: Re: source(builtin) and read(2) SK> SK>> I'll submit a defec

Re: Add 'Required Tools' to README?

2007-04-03 Thread Bruno Haible
Simon Josefsson wrote: > I think some of the problems here could be fixed by noting what tools > are required by gnulib developers. There is no such section in the > gnulib README or gnulib.texi, or did I miss it? I re-installed the > gnulib daily build bot yesterday, and I had to install these:

Re: stdint and cygwin bug

2007-04-03 Thread Paul Eggert
Eric Blake <[EMAIL PROTECTED]> writes: > cygwin currently defines wint_t as unsigned int (legal per POSIX), but then > tries to use INT_MIN and INT_MAX for WINT_MIN and WINT_MAX. I've reported > this > as a bug to the cygwin list. In the meantime, is it worth fixing the stdint > module to de

Re: gnulib tests vs. LDADD

2007-04-03 Thread Bruno Haible
Eric Blake wrote: > I noticed the following message from gnulib-tool: > > You may need to use the following Makefile variables when linking. > Use them in _LDADD when linking a program, or > in _a_LDFLAGS or _la_LDFLAGS when linking a library. > $(LIB_CLOCK_GETTIME) > $(LTLIBINTL) when linking

Re: missing dependency in *printf

2007-04-03 Thread Bruno Haible
Eric Blake wrote: > In trying to use sprintf-posix in m4, I came across this: > > cd .. && /bin/sh /home/eblake/m4-head/ltdl/config/missing --run autoconf > configure:25505: error: possibly undefined macro: gl_FUNC_LDEXPL_WORKS > If this token and others are legitimate, please use m4_pattern

iconv_open module need $(srcdir) in Makefile fragment

2007-04-03 Thread Ben Pfaff
Building a program that imports iconv_open with builddir != srcdir fails due to lack of $(srcdir) in the Makefile fragment. OK to commit the following? --- gnulib/modules/iconv_open.~1.1.~2007-03-31 14:00:19.0 -0700 +++ gnulib/modules/iconv_open 2007-04-03 16:49:40.0 -0700 @

Re: missing dependency in *printf

2007-04-03 Thread Bruno Haible
Eric Blake wrote: > libtool: compile: gcc -std=gnu99 -I. -I../../gnu -I../intl -g2 -Wall > -Werror -MT ldexpl.lo -MD -MP -MF .deps/ldexpl.Tpo -c ../../gnu/ldexpl.c > -DDLL_EXPORT -DPIC -o .libs/ldexpl.o > ../../gnu/ldexpl.c:29:20: isnanl.h: No such file or directory > ... > make[3]: *** [ldexpl.lo

Re: iconv_open module need $(srcdir) in Makefile fragment

2007-04-03 Thread Ben Pfaff
Ben Pfaff <[EMAIL PROTECTED]> writes: > Building a program that imports iconv_open with builddir != > srcdir fails due to lack of $(srcdir) in the Makefile fragment. > > OK to commit the following? Oops--the $(srcdir)/ is not necessary in the prerequisites, of course. -- "I was born lazy. I am

Re: iconv_open module need $(srcdir) in Makefile fragment

2007-04-03 Thread Bruno Haible
Ben Pfaff wrote: > Building a program that imports iconv_open with builddir != > srcdir fails due to lack of $(srcdir) in the Makefile fragment. Oops, you're right. > OK to commit the following? > > --- gnulib/modules/iconv_open.~1.1.~ 2007-03-31 14:00:19.0 -0700 > +++ gnulib/modules/ic

Re: new module 'iconv_open'

2007-04-03 Thread Bruno Haible
Ben Pfaff wrote: > In PSPP, we've been trying to display all the missing > prerequisite libraries together at the end of a single configure > run, instead of just failing the configure run after finding the > first missing prerequisite library. It would be a natural > extension to apply this to he

Re: new module 'iconv_open'

2007-04-03 Thread Bruno Haible
Simon Josefsson wrote: > For example, in GnuTLS, I recently received a patch that added a > configure-time warning about missing non-required tools: > > AC_PATH_PROG([GAA], [gaa]) > if test "x$GAA" = "x"; then >AC_MSG_WARN([[*** > *** GAA was not found. It is only needed if you wish to modify

Re: new module 'iconv_open'

2007-04-03 Thread Ben Pfaff
Bruno Haible <[EMAIL PROTECTED]> writes: > Ben Pfaff wrote: >> In PSPP, we've been trying to display all the missing >> prerequisite libraries together at the end of a single configure >> run, instead of just failing the configure run after finding the >> first missing prerequisite library. It wou

Re: gnulib tests vs. LDADD

2007-04-03 Thread Bruno Haible
Eric Blake wrote: > CVS findutils uses gnulib-tool --with-tests when importing. But on cygwin, > compilation then fails in the gnulib-created directory: > > gcc -g2 -Wall -Werror -o test-dirname.exe test- > dirname.o ../gnulib/lib/libgnulib.a > ../gnulib/lib/libgnulib.a(xalloc-die.o): In fun

Re: Add 'Required Tools' to README?

2007-04-03 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 4/3/2007 5:02 PM: > Yes, good suggestion. I'm creating a file 'DEPENDENCIES'. Feel free to add > a pointer to it from the README if you like. A bit strong on the m4 requirements, since debian stable doesn't ship 1.4.9 yet

Re: iconv_open module need $(srcdir) in Makefile fragment

2007-04-03 Thread Ben Pfaff
Bruno Haible <[EMAIL PROTECTED]> writes: > Ben Pfaff wrote: >> Building a program that imports iconv_open with builddir != >> srcdir fails due to lack of $(srcdir) in the Makefile fragment. > > The modification of the statement line is good. Please apply it. Thanks, done. -- Ben Pfaff [EMAIL PRO

Re: Add 'Required Tools' to README?

2007-04-03 Thread Bruno Haible
Eric Blake wrote: > I tweaked some of the > wording: since cc and make are used by the end user, while the rest of the > tools are developer-only, using native cc and make instead of gcc and > gmake helps test portability. As you like. But I don't agree with two of these changes: > + + VPATH bui

Re: Add 'Required Tools' to README?

2007-04-03 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 4/3/2007 7:50 PM: > Eric Blake wrote: >> I tweaked some of the >> wording: since cc and make are used by the end user, while the rest of the >> tools are developer-only, using native cc and make instead of gcc and >> gmake

Re: Add 'Required Tools' to README?

2007-04-03 Thread Ralf Wildenhues
Hello Bruno, * Bruno Haible wrote on Wed, Apr 04, 2007 at 01:02:09AM CEST: > > * GNU autoconf 2.59 or newer. [...] > * GNU automake 1.10 or newer. This does not make sense. Automake-1.10 requires Autoconf-2.60 or newer. Cheers, Ralf

RE: source(builtin) and read(2)

2007-04-03 Thread Schwarz, Konrad
> -Original Message- > From: Paul Eggert [mailto:[EMAIL PROTECTED] > "Schwarz, Konrad" <[EMAIL PROTECTED]> writes: > > > The argument that since ptrdiff_t is intended to represent the > > difference of pointers makes it somehow unsuitable to represent the > > result of read()/write() is