[PATCH] tests/test-vasprintf.c: Test %08lx.

2010-04-21 Thread Simon Josefsson
ddc0d09..64948d7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2010-04-21 Simon Josefsson si...@josefsson.org + + * tests/test-vasprintf.c: Test %08lx. + 2010-04-20 Eric Blake ebl...@redhat.com tests: be robust to ignored SIGPIPE diff --git a/tests/test-vasprintf.c b/tests

Re: Failed test on Solaris 8 Sparc

2010-04-21 Thread Simon Josefsson
Dagobert Michelsen d...@opencsw.org writes: Hi Simon, Am 31.03.2009 um 12:23 schrieb Simon Josefsson: Dagobert Michelsen d...@opencsw.org writes: I am getting a test failure on Solaris 8 Sparc w/Sun Studio 11: Hi! Thanks for the report. The problem is actually in a self-test from gnulib

Re: [PATCH] tests/test-vasprintf.c: Test %08lx.

2010-04-21 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: On 04/21/2010 05:21 AM, Simon Josefsson wrote: For some reason 'asprintf (tmp, %08lx, 1)' yields '61fdec3039'. I haven't started debugging the printf code yet, but will get to it... Is this on a 32-bit or 64-bit platform? It matters if you are asking

Re: portable openpty, forkpty

2010-04-21 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: HAVE_OPENPTY is not set in config.h though -- isn't that a bug? The system has openpty. ... Single stepping rpl_openpty: master = open (/dev/ptmx, O_RDWR | O_NOCTTY); if (master 0) = return -1; And indeed my Mac OS X

Re: visibility.m4

2010-04-20 Thread Simon Josefsson
Bruno, the patch looks simple to me, any objections to pushing it? I'm seeing a lot of useless warnings when building for Windows without it. /Simon Simon Josefsson si...@josefsson.org writes: Bruno, ping? Ok to push? /Simon Simon Josefsson si...@josefsson.org writes: Mike Gran spk

Re: testdir without unportable tests?

2010-04-20 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: Right now I create the directory like this: gnulib-tool --create-testdir --with-tests --dir build Is there a way to avoid all the unportable tests? For example, a --without-unportable-tests? I'm currently using --avoid

Re: visibility.m4

2010-04-20 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: It is bizarre, though, that the warning only appears at the first function definition. Seen on Cygwin 1.7.2: ... AC_TRY_COMPILE does add a function for 'main', but this deserves a comment at least. Yes, the entire semantics of this flag on Windows appears

Re: getaddrinfo on NetBSD 5.0

2010-04-16 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Ludovic Courtès wrote: I think the problem is that ‘gl_HEADER_NETDB’ determines ‘NETDB_H=''’ because netdb.h is indeed available and it does define ‘struct addrinfo’; it just happens to lack some of the AI_ macros. That explains it. I propose to generate

Re: portable openpty, forkpty

2010-04-15 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: This is from a package created with './gnulib-tool --dir m --create-testdir --with-tests forkpty' The only version of MacOS X I have access to is 10.5, and on this Mine is a powerpc system, maybe it could explain the difference? system it works fine,

Re: C++ libtool build error?

2010-04-15 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Additionally, the user who installs a package might want to disable the C++ part even if it is packaged as part of the tarball. I'm applying this. Thanks, but the original problem is still not addressed -- building libidn (and libunistring, and likely

Re: [PATCH] sys_time: avoid compiler warnings

2010-04-15 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: ./../gl/sys/time.h:353: error: redundant redeclaration of '_gl_cxxalias_dummy' [-Wredundant-decls] ./../gl/string.h:1103: note: previous declaration of '_gl_cxxalias_dummy' was here I have some non-default warnings enabled. Maybe we could protect

Re: duplocale cygwin failure

2010-04-15 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Hi Simon, In file included from test-locale-c++.cc:22: ../gllib/locale.h:329: error: ‘duplocale’ was not declared in this scope ../gllib/locale.h:329: error: invalid type in declaration before ‘;’ token make[3]: *** [test-locale-c++.o] Error 1 make[3]:

Re: new syntax-check rule for @acronym?

2010-04-15 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Simon Josefsson wrote: Jim Meyering j...@meyering.net writes: Simon Josefsson wrote: Eric Blake ebl...@redhat.com writes: ... # Don't use Texinfo @acronym{} as it is not a good idea. sc_texinfo_acronym: - @grep -nE '@acronym

Re: new syntax-check rule for @acronym?

2010-04-15 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Simon Josefsson wrote: Eric Blake ebl...@redhat.com writes: ... # Don't use Texinfo @acronym{} as it is not a good idea. sc_texinfo_acronym: -@grep -nE '@acronym{' \ -$$($(VC_LIST_EXCEPT) | grep

Re: new syntax-check rule for @acronym?

2010-04-15 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Simon Josefsson wrote: ... Heh, easy to miss. Please use the following: # Don't use Texinfo @acronym{} as it is not a good idea. texinfo_suffix_re_ ?= \\.(txi|texi(nfo)?)$$ sc_texinfo_acronym: @prohibit='@acronym

testdir without unportable tests?

2010-04-13 Thread Simon Josefsson
Back from vacation I re-built latest gnulib under latest Cygwin: http://autobuild.josefsson.org/gnulib/log-201004130216951714000.txt As you can see, it runs the new havelib checks which fails -- I assume these are part of the unportable tests? Right now I create the directory like this:

redundant redeclaration of '_gl_cxxalias_dummy'

2010-03-31 Thread Simon Josefsson
Building GnuTLS fails because of: ./../gl/sys/time.h:350: error: redundant redeclaration of '_gl_cxxalias_dummy' [-Wredundant-decls] ./../gl/string.h:1103: note: previous declaration of '_gl_cxxalias_dummy' was here ./../gl/sys/time.h:353: error: redundant redeclaration of '_gl_cxxalias_dummy'

duplocale cygwin failure (was: Re: lchmod cygwin failure)

2010-03-29 Thread Simon Josefsson
. (lstat): Don't warn about the use of this function if it is undefined or if we alias it to stat. Reported by Simon Josefsson. Thanks! The same class of problem appears to occur for duplocale too: In file included from test-locale-c++.cc:22: ../gllib/locale.h:329: error: ‘duplocale

Re: lchmod cygwin failure

2010-03-29 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: ../gllib/sys/stat.h:668: error: ‘lchmod’ was not declared in this scope ... (Not sure how the broken UTF-8 characters happened..) What terminal are you using? It is most likely a case of using a locale with a UTF-8 charset, but a terminal like rxvt

Re: [PATCH] build: avoid link failure on systems using gnulib's fcntl but not open

2010-03-28 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon, is it ok to remove the GNULIB_GC_CAMELLIA macro? I see it nowhere used in gnulib. Is it used in one of your packages? I don't recall -- but please remove it, and if I notice any problems I'll follow with a proper solution. /Simon

Re: putc vs printf

2010-03-26 Thread Simon Josefsson
Thien-Thi Nguyen t...@gnuvola.org writes: GNU RCS does not presently use gnulib, but it will in the future. With that in mind, i have a question inspired by this comment in the source code: /* Output the first character with `putc', not `printf'. Otherwise, an SVR4 stdio bug buffers

lchmod cygwin failure

2010-03-25 Thread Simon Josefsson
Building in cygwin fails because: depbase=`echo test-fcntl-h-c++.o | sed 's|[^/]*$|.deps/|;s|\.o$||'`;\ g++ -DHAVE_CONFIG_H -I. -I. -I. -I.. -I./.. -I../gllib -I./../gllib -MT test-fcntl-h-c++.o -MD -MP -MF $depbase.Tpo -c -o test-fcntl-h-c++.o test-fcntl-h-c++.cc \ mv -f

Re: visibility.m4

2010-03-25 Thread Simon Josefsson
Bruno, ping? Ok to push? /Simon Simon Josefsson si...@josefsson.org writes: Mike Gran spk...@yahoo.com writes: Hello- If I use visibility.m4 on a platform with a recent GCC (4.x) on Cygwin, the visibility.m4 correctly discovers that gcc can compile with -fvisibility=hidden

Re: new syntax-check rule for @acronym?

2010-03-24 Thread Simon Josefsson
17 00:00:00 2001 From: Simon Josefsson si...@josefsson.org Date: Wed, 24 Mar 2010 08:45:04 +0100 Subject: [PATCH] top/maint.mk (sc_texinfo_acronym): Add rule. Based on suggestions from k...@freefriends.org (Karl Berry). --- ChangeLog|5 + top/maint.mk |7 +++ 2 files changed

Re: new module 'pt_chown'

2010-03-24 Thread Simon Josefsson
by Simon Josefsson. ... 2010-03-23 Bruno Haible br...@clisp.org pt_chown: Make it work with automake 1.11. * modules/pt_chown (Makefile.am): Define pkglibexecdir. Reported by Simon Josefsson. Looks fine to me, please push! Thanks, /Simon

sc_Wundef_boolean

2010-03-24 Thread Simon Josefsson
This syntax-check warning appears to fail: j...@mocca:~/src/libidn master$ make sc_Wundef_boolean Wundef_boolean grep: : No such file or directory j...@mocca:~/src/libidn master$ Any reason we can't use the normal VC_LIST_EXCEPT instead? It will check more files than just config.h.

Re: sc_Wundef_boolean

2010-03-24 Thread Simon Josefsson
I have pushed the patch now. /Simon

Re: new module 'pt_chown'

2010-03-23 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: I put the module under LGPL rather GPL, because the modules 'openpty' and 'forkpty' should stay under LGPL and I'm not willing to go into long discussions about whether a module that uses (and cannot work without) a GPLed program is a combined work based

new syntax-check rule for @acronym?

2010-03-23 Thread Simon Josefsson
k...@freefriends.org (Karl Berry) writes: Hi Simon, Is the intention that even the n...@acronym{gnu} cases should be replaced? Then what purpose is the @acronym keyword for? I wrote about that earlier. Minor typographic change which is rarely used in GNU manuals. De facto

Re: portable openpty, forkpty

2010-03-23 Thread Simon Josefsson
(I'm adding bug-gnulib to the discussion, maybe it can move over there completely.) The forkpty code builds fine as a part of InetUtils on Mac OS X. I tried the gnulib self-checks for forkpty, but they failed: Making check in gltests make check-recursive make check-TESTS PASS: test-alignof

Re: [PATCHv4] gethostname: fix build on mingw

2010-03-22 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: Since commit 4e86671a, gethostname failed to compile on mingw. Gnulib's sys/socket.h includes winsock2.h, which then includes unistd.h prior to declaring gethostname. The fix is to ensure that our replacement unistd.h does not declare any replacements

Re: [PATCHv4] gethostname: fix build on mingw

2010-03-22 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: On 03/22/2010 11:39 AM, Simon Josefsson wrote: Eric Blake ebl...@redhat.com writes: Since commit 4e86671a, gethostname failed to compile on mingw. Gnulib's sys/socket.h includes winsock2.h, which then includes unistd.h prior to declaring gethostname

Re: [PATCH 2/3] forkpty, openpty: split functions into new modules

2010-03-19 Thread Simon Josefsson
I like these changes -- I think I mentioned that approach earlier when adding the pty module, but before we had replacement functions for forkpty/openpty in progress, the separation is kind of unnecessary. Let's hope we'll have replacement functions soon after this. Feel free to push your

Re: C++ libtool build error?

2010-03-17 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Actually, there are quite a number of tests that not every project may want to carry in its testsuite, but that should be packageable via --create-testdir: - The C++ tests of the modules 'string', 'stdlib', etc. - The tests that ask for the superuser

Re: ‘pty’ module fails on FreeBSD 7.2

2010-03-17 Thread Simon Josefsson
l...@gnu.org (Ludovic Courtès) writes: Hi Ralf, Ralf Wildenhues ralf.wildenh...@gmx.de writes: * Ludovic Courtès wrote on Tue, Mar 16, 2010 at 10:52:11AM CET: checking whether forkpty is declared... no checking whether forkpty is declared... (cached) no Looking at pty.m4, I’m wondering

Re: [bug-libunistring] Re: C++ libtool build error?

2010-03-12 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: So, the fix is to add an LT_LANG([CXX]) invocation to your configure.ac. This fixes it for libunistring. That doesn't make a lot of sense to me because 1) It violates the modularity of gnulib. Gnulib is adding the C++ code to my project, so I

[PATCH] gnulib-tool (func_import): Explain origin of code.

2010-03-12 Thread Simon Josefsson
--- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2010-03-12 Simon Josefsson si...@josefsson.org + + * gnulib-tool (func_import): Explain origin of code. + 2010-03-12 Bruno Haible br...@clisp.org Fix problem with automake's definition of CXXLINK. diff --git a/gnulib-tool b/gnulib

Re: C++ libtool build error?

2010-03-09 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Recently gnulib added some self-tests written in C++ for otherwise C-only modules. When imported into libidn (otherwise a strictly C library), this leads to an error message: /bin/bash ../libtool --preserve-dup-deps --mode=link g++ -o

Re: C++ libtool build error?

2010-03-09 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: On 03/09/2010 01:02 AM, Simon Josefsson wrote: No, I have version '2.2.6b' from debian. I noticed one thing: if I add AC_PROG_CXX to libidn/configure.ac everything works. So I believe this is a bug in gnulib, it is adding C++ files to my project so

C++ libtool build error?

2010-03-08 Thread Simon Josefsson
Recently gnulib added some self-tests written in C++ for otherwise C-only modules. When imported into libidn (otherwise a strictly C library), this leads to an error message: /bin/bash ../libtool --preserve-dup-deps --mode=link g++ -o test-fcntl-h-c++ test-fcntl-h-c++.o libtests.a

[PATCH] gnulib-tool (LIBTOOLPATH): Fix cut'n'paste bug.

2010-03-08 Thread Simon Josefsson
/ChangeLog index 6bb76f4..d9e17ae 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2010-03-09 Simon Josefsson si...@josefsson.org + + * gnulib-tool (LIBTOOLPATH): Fix cut'n'paste bug. + 2010-03-08 Bruno Haible br...@clisp.org gnulib-tool: Add support for --libtool in --create

copyright years x-z instead of x,y,z

2010-03-07 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: C++ programs will expect the POSIX prototype, even on mingw. On mingw, the types are effectively the same (ssize_t = int, socklen_t = int), therefore this should not introduce bugs (possibly some warnings, that can be ignored). I don't have any objection

Re: euidaccess

2010-03-06 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Bruno Haible wrote: Thanks -- I've pushed the modified patch. Now the only license issue within gnulib is 'euidaccess'... + euidaccess: relax license to LGPLv2+ Hi Bruno, This won't help much, because 'euidaccess' depends on 'group-member',

Re: relax licenses for both exitfail and exit

2010-03-05 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: I wrote: I'll see if Paul agrees. He does: Yes, it's OK to relax the licenses for both exit and exitfail. Thanks -- I've pushed the modified patch. Now the only license issue within gnulib is 'euidaccess'... /Simon

Re: fstab.h for ARM

2010-03-04 Thread Simon Josefsson
Jon Grosshart jgrossh...@gmail.com writes: Trying to build samba 3.5.0 on a DNS-323 embeded NAS running on ARM with Uclibc. mount.cifs fails with not finding fstab.h Is Samba using gnulib? If not, anything done in gnulib will not help. It seems gnulib could easily provide a fstab.h

exit

2010-03-04 Thread Simon Josefsson
Building all modules leads to this warning: gnulib-tool: warning: module obstack depends on a module with an incompatible license: exit I took a look at this. Here is the 'exit' module definition: , | Description: | exit() function: program termination. | | Files: | | Depends-on: |

exitfail

2010-03-04 Thread Simon Josefsson
. : ]) It seems this file could be dropped, and the AC_LIBOBJ moved to modules/exitfail. See patch below. /Simon From 63d31953c3f950bfac3b51ac1cd63928969a7204 Mon Sep 17 00:00:00 2001 From: Simon Josefsson si...@josefsson.org Date: Thu, 4 Mar 2010 14:42:41 +0100 Subject: [PATCH] exitfail: Relax license

utime

2010-03-04 Thread Simon Josefsson
Jim, the utime module says: Notice: This module is obsolete. It can be removed on 2010-01-01. Why is it obsolete? Should we remove it now? The code looks useful to me, but I may be missing something. I cannot find anything in NEWS or doc/posix-functions/utime.texi to explain this either.

Re: exitfail

2010-03-04 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: I suppose you intend LGPLv2+ here, so it's like all of the others, and not LGPLv2.1+. Likewise in the patch below. Yes, sorry. I'll see if Paul agrees. Thanks! /Simon

Re: exit

2010-03-04 Thread Simon Josefsson
Eric Blake ebl...@redhat.com writes: According to Simon Josefsson on 3/4/2010 6:33 AM: Building all modules leads to this warning: gnulib-tool: warning: module obstack depends on a module with an incompatible license: exit Is the 'exit' module really useful, other than transition

Re: exit

2010-03-04 Thread Simon Josefsson
appropriate to change Gnulib module: exit to Gnulib module: stdlib. I've pushed the patch below. /Simon From fa93ceab8152b5382d328d2c5e6347e53b291cd0 Mon Sep 17 00:00:00 2001 From: Simon Josefsson si...@josefsson.org Date: Fri, 5 Mar 2010 07:51:36 +0100 Subject: [PATCH] exit: Obsolete module, use

Re: gnulib module install

2010-03-02 Thread Simon Josefsson
k...@freefriends.org (Karl Berry) writes: What I mean - I wouldn't mind commiting gnulib stuff to VCS, but only if I can omit files generated by autoreconf. FWIW, that is exactly what I do for Texinfo. It helps my contributors to not spend time dealing with gnulib, and it helps me

Re: gnulib-tool feature request

2010-03-02 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 3/1/10, Simon Josefsson si...@josefsson.org wrote: It would be nice it gnulib-tool could do it, but I have a hard time thinking how that would actually be implemented. There are so many different ways you may want to organize your gnulib directories

Re: gnulib-tool feature request

2010-03-02 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 3/2/10, Simon Josefsson si...@josefsson.org wrote: Sam Steingold s...@gnu.org writes: On 3/1/10, Simon Josefsson si...@josefsson.org wrote: It would be nice it gnulib-tool could do it, but I have a hard time thinking how that would actually

Re: gnulib-tool feature request

2010-03-01 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 2/27/10, Simon Josefsson si...@josefsson.org wrote: Since I use gnulib in several sub-modules, I need to avoid conflicts between different gnulib imports. thus I need to make all those _GL_* constants module-specific. thus I need gnulib

Re: gnulib-tool feature request

2010-03-01 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 3/1/10, Simon Josefsson si...@josefsson.org wrote: Since I use gnulib in several sub-modules, I need to avoid conflicts between different gnulib imports. thus I need to make all those _GL_* constants module-specific. thus I need

Re: gnulib-tool feature request

2010-03-01 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 3/1/10, Simon Josefsson si...@josefsson.org wrote: Sam Steingold s...@gnu.org writes: as I said above, I do have a separate configure.in, gllib, glm4 in each module. I am confused. If they are separate, how could the gnulib generated files

Re: gnulib-tool feature request

2010-03-01 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: systems. Did you try to measure the bloat size? Which modules with object code do you need in every sub-module? here is what I pass with --avoid: no-c++ stdint stdbool havelib gettext localcharset uniwidth/width streq uniname/uniname unitypes

Re: gnulib-tool feature request

2010-03-01 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: On 3/1/10, Simon Josefsson si...@josefsson.org wrote: On my debian system, how about the poor souls stuck with non-kosher fedora? :-) You are in the same boat as I compared to those gNewSense users. ;-) Further, these functions seems rather independent

Re: gnulib-tool feature request

2010-02-26 Thread Simon Josefsson
Sam Steingold s...@gnu.org writes: Since I use gnulib in several sub-modules, I need to avoid conflicts between different gnulib imports. thus I need to make all those _GL_* constants module-specific. thus I need gnulib-tool to accept a --macro-prefix option and this patch: I believe the

Re: [PATCH] lib/malloca.c, lib/md5.c, lib/sha1.c: don't include config.h unless HAVE_CONFIG_H is defined

2010-02-24 Thread Simon Josefsson
Peter Simons sim...@cryp.to writes: The following patches are necessary to make those modules compile in projects that don't use a config.h header. That is generally not supported by gnulib. The majority of code in gnulib assumes there is a config.h already; 95% if my grep is working. /Simon

Re: Cygwin build report on Windows XP

2010-02-18 Thread Simon Josefsson
Ralf Wildenhues ralf.wildenh...@gmx.de writes: * Simon Josefsson wrote on Thu, Feb 18, 2010 at 08:58:14AM CET: The build times are interesting: gnulib-tool: 324 minutes ./configure: 36 minutes make check: 34 minutes The build times are now: gnulib-tool: 418 minutes ./configure

Re: ./init.sh: line 139: `test-acos': not a valid identifier

2010-02-18 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Right. The *.exe names are sorted, and test-acos is the first one encountered. Ah. Is there a more functional shell on that system? If so, is it already detected and thus mentioned somewhere in config.status? checking for a shell that conforms to

Re: ./init.sh: line 139: `test-acos': not a valid identifier

2010-02-18 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Can you confirm that those tests succeed when you replace #!/bin/sh with #!/bin/bash in the test scripts that use init.sh? Yes, then the self-test works. (I only tested test-pread.sh, but expect the others to behave the same.) /Simon

Re: interix and rpl_select

2010-02-18 Thread Simon Josefsson
Markus Duft md...@gentoo.org writes: Hey! a while ago you guys implemented a select() fix for interix 3.5 (thanks again ;)). now i was surprised to see the same issue come up again (it's there since a while now again, just had no time to dig into the problem). i now had some time to look

Re: Cygwin build report on Windows XP

2010-02-18 Thread Simon Josefsson
Eric Blake e...@byu.net writes: According to Simon Josefsson on 2/18/2010 12:52 AM: Bruno Haible br...@clisp.org writes: Starting test_cond .../bin/sh: line 5: 3124 Aborted (core dumped) FAIL: test-cond.exe Starting test_lock .../bin/sh: line 5: 1336 Aborted

Re: NetBSD 5.0’s netdb.h lacks AI_* macros

2010-02-17 Thread Simon Josefsson
From be51f2652b95b8b7b8778039c012d0c9c70741a0 Mon Sep 17 00:00:00 2001 From: Simon Josefsson si...@josefsson.org Date: Wed, 17 Feb 2010 09:19:03 +0100 Subject: [PATCH] netdb.h: Define AI_V4MAPPED, AI_ALL, AI_ADDRCONFIG when needed. --- ChangeLog |6 ++ lib/netdb.in.h | 28

Re: NetBSD 5.0’s netdb.h lacks AI_* macros

2010-02-17 Thread Simon Josefsson
Eric Blake e...@byu.net writes: According to Simon Josefsson on 2/17/2010 1:27 AM: Thanks, fixed as below. Note that _support_ for these symbols is not required by POSIX (it is conditional on IPv6 support), but the symbols are required to be defined. +# define AI_ADDRCONFIG 0 /* 0x0020

Re: NetBSD 5.0’s netdb.h lacks AI_* macros

2010-02-17 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: fixed as below. ... So this is a bug in NetBSD netdb.h I'm adding a mention of it to the doc: Thanks. /Simon

Re: Cygwin build report on Windows XP

2010-02-17 Thread Simon Josefsson
Simon Josefsson si...@josefsson.org writes: The build times are interesting: gnulib-tool: 324 minutes ./configure: 36 minutes make check: 34 minutes Another build, on the same physical hardware, is available from: http://autobuild.josefsson.org/gnulib/log-201002171652277193000.txt

Re: Bogus getaddrinfo(3) on Darwin 9.2.2

2010-02-16 Thread Simon Josefsson
l...@gnu.org (Ludovic Courtès) writes: Hello! The getaddrinfo(3) function appears to have bogus behavior on ‘i386-apple-darwin9.2.2’ (Darwin 9.6 apparently doesn’t have the problem). Here’s the test program: #include netdb.h #include stdio.h #include stdlib.h #include string.h int

userspec-tests: cygwin link failure due to missing -lintl

2010-02-16 Thread Simon Josefsson
@@ -1,3 +1,8 @@ +2010-02-16 Simon Josefsson si...@josefsson.org + + * modules/userspec-tests (test_userspec_LDADD): Add variable, for + linking with -lintl. + 2010-02-15 Jim Meyering meyer...@redhat.com maint.mk: mark syntax-check sc_*.m rules as .PHONY diff --git

Re: Bogus getaddrinfo(3) on Darwin 9.2.2

2010-02-16 Thread Simon Josefsson
l...@gnu.org (Ludovic Courtès) writes: What I meant to say is that it should return an error because resolving the ‘does-not-exist’ service fails. Does it only fail (well, falsely succeed) for unknown services? The key piece of info here is port = 0, which makes no sense. Right, although I

Re: Bogus getaddrinfo(3) on Darwin 9.2.2

2010-02-16 Thread Simon Josefsson
l...@gnu.org (Ludovic Courtès) writes: Simon Josefsson si...@josefsson.org writes: l...@gnu.org (Ludovic Courtès) writes: What I meant to say is that it should return an error because resolving the ‘does-not-exist’ service fails. Does it only fail (well, falsely succeed) for unknown

Re: library symbol versioning hassles

2010-02-05 Thread Simon Josefsson
After more debugging, it seems the behavior I wanted is actually the default -- however the dynamic linker prints a warning, which can cause problems. http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=542512#45 for background. This seems good enough to not worry seriously about it further --

Re: Argp fix

2010-02-05 Thread Simon Josefsson
Sergey Poznyakoff g...@gnu.org.ua writes: Hello, Argp failed to properly recognize alias options that have short option letters. For instance, in this setup: { dstaddr, 'd', ADDR, 0, set destination (peer) address to ADDR }, { peer, 'p', ADDR, OPTION_ALIAS }, both --peer and -p

library symbol versioning hassles

2010-02-04 Thread Simon Josefsson
This isn't strictly gnulib related, but this list is the best community I know that would be able to answer this question, so pardon this slightly off-topic discussion... I maintain some packages that contain shared libraries, for example libidn and libgsasl, and in recent releases I've added

copyright statement

2010-01-27 Thread Simon Josefsson
17 00:00:00 2001 From: Simon Josefsson si...@josefsson.org Date: Wed, 27 Jan 2010 16:13:04 +0100 Subject: [PATCH] Fix copyright statement. --- ChangeLog | 42 ++ build-aux/git-version-gen |4 ++-- build-aux/gnupload |4

Re: copyright statement

2010-01-27 Thread Simon Josefsson
Eric Blake e...@byu.net writes: build-aux/gnupload |4 ++-- This is synced from automake; the fix needs to be pushed upstream to remain persistent. Oops! Here is a patch against automake, we can sync it to gnupload once this has been applied. /Simon diff --git a/lib/gnupload

Re: copyright statement

2010-01-27 Thread Simon Josefsson
Eric Blake e...@byu.net writes: -# Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Free Software Foundation +# Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Free Software Foundation, Inc. That, and this created a long line. Wouldn't it be better to rerun the update-

Re: copyright statement

2010-01-27 Thread Simon Josefsson
Eric Blake e...@byu.net writes: Eric Blake ebb9 at byu.net writes: Simon Josefsson simon at josefsson.org writes: Most likely a problem I introduced at some point which has been cut'n'pasted a few time since then... Nice catch. Also problematic: build-aux/ncftpput-ftp

Re: copyright statement

2010-01-27 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Jim Meyering wrote: I've changed the hybrids to be a simple range: git grep -l -E '[0-9]{4} *Free Software F' \ | xargs perl -pi -e 's/-2009, 2010 +(Free Software)/-2010 $1/' And also changed the two spaces before Free Software... to one: grep

Re: cosl test failure

2010-01-22 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Hi Paolo, I added trivial tests for the mathl functions, that verify that 1) there are no link errors, 2) on a randomly picked value, the function's result is correct to 10 decimal digits. For the cosl() function, when compiled from the gnulib

Re: [PATCH] tests/test-xalloc-die.sh: Use $EXEEXT.

2010-01-13 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: diff --git a/tests/test-xalloc-die.sh b/tests/test-xalloc-die.sh index 03bad60..80d6208 100755 --- a/tests/test-xalloc-die.sh +++ b/tests/test-xalloc-die.sh @@ -18,7 +18,7 @@ . ${srcdir=.}/init.sh; path_prepend_ . -test

Re: Cygwin build report on Windows XP

2010-01-13 Thread Simon Josefsson
Simon Josefsson si...@josefsson.org writes: I'll try to set the machine up to build it once per day or so, depending on how long it takes. Another report is available at: http://autobuild.josefsson.org/gnulib/log-201001130336758941000.txt It has the same failures as before since it didn't

Re: [PATCH] tests/test-xalloc-die.sh: Use $EXEEXT.

2010-01-13 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: I wonder if there is some way to write/use a command-not-found handler to automate the error-prone add-$EXE-suffix at run-time. We could create a macro in init.sh that needs to be used when invoking a program, but I think it would be as error-prone as

Re: Cygwin build report on Windows XP

2010-01-13 Thread Simon Josefsson
Ralf Wildenhues ralf.wildenh...@gmx.de writes: Hello, * Simon Josefsson wrote on Wed, Jan 13, 2010 at 10:49:19AM CET: The build times are interesting: gnulib-tool: 324 minutes ./configure: 36 minutes make check: 34 minutes Now, gnulib-tool invokes run ./configure too, but that seems

Re: getlogin(_r) self-test failures

2010-01-12 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Hi Simon, These tests are fail unnecessarily easy, I think. See: $ rm -rf m ./gnulib-tool --create-testdir --dir m --with-tests getlogin_r cd m ./configure nohup make check log 21 Indeed, I did not think about this situation. How about this

Re: open MinGW failure

2010-01-12 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Removing the mingw* case in the 'case' statement is wrong, because the * case executes an AC_CACHE_CHECK which would yield 'no' when cross-compiling to mingw or natively compiling to mingw via wine but yield 'yes' when natively compiling to mingw via

Re: visibility.m4

2010-01-12 Thread Simon Josefsson
Mike Gran spk...@yahoo.com writes: Hello- If I use visibility.m4 on a platform with a recent GCC (4.x) on Cygwin, the visibility.m4 correctly discovers that gcc can compile with -fvisibility=hidden.  But, on Cygwin (and probably most non-ELF) gcc does not actualy implement the visibility

[PATCH] tests/test-xalloc-die.sh: Use $EXEEXT.

2010-01-12 Thread Simon Josefsson
Pushed. /Simon --- ChangeLog|4 tests/test-xalloc-die.sh |2 +- 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index fac2f33..7e9fc3e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,8 @@ 2010-01-12 Simon Josefsson si

maint.mk syntax-check

2010-01-12 Thread Simon Josefsson
It would be useful to have 'syntax-check' never check certain files/directories. This patch makes it possible to set VC_LIST_NEVER in cfg.mk. Objections to pushing? /Simon 2010-01-12 Simon Josefsson si...@josefsson.org * top/maint.mk (VC_LIST_EXCEPT): Filter list through

warn-on-use.m4 requires modern M4?

2010-01-12 Thread Simon Josefsson
I get errors from autoreconf when bootstrapping Libidn on a Ubuntu 8.04 LTS system, and removing the recent warn-on-use.m4 file fixes it. Is there something in the file that requires a recent GNU M4? Ubuntu 8.04 has GNU M4 1.1.10, and according to gnulib's DEPENDENCIES 1.4.5 or later should be

Re: warn-on-use.m4 requires modern M4?

2010-01-12 Thread Simon Josefsson
Simon Josefsson si...@josefsson.org writes: I get errors from autoreconf when bootstrapping Libidn on a Ubuntu 8.04 LTS system, and removing the recent warn-on-use.m4 file fixes it. Is there something in the file that requires a recent GNU M4? Ubuntu 8.04 has GNU M4 1.1.10, and according

Re: warn-on-use.m4 requires modern M4?

2010-01-12 Thread Simon Josefsson
Eric Blake e...@byu.net writes: The problem is not old m4, but old autoconf. Which version are you using? Ubuntu 8.04 has autoconf 2.61. The problem is not old m4, but old autoconf. So, the right patch would be to move gl_AS_VAR_IF out of warnings.m4, and make gnulib-common.m4 define

Re: warn-on-use.m4 requires modern M4?

2010-01-12 Thread Simon Josefsson
Ben Pfaff b...@cs.stanford.edu writes: Eric Blake e...@byu.net writes: Meanwhile, based on the recent security hole in older automake, should we bite the bullet and require automake 1.10.3 or better (which in turn bumps the minimum autoconf requirement from 2.59 up to 2.60)? Debian stable

Re: maint.mk syntax-check

2010-01-12 Thread Simon Josefsson
Jim Meyering j...@meyering.net writes: Simon Josefsson wrote: It would be useful to have 'syntax-check' never check certain files/directories. This patch makes it possible to set VC_LIST_NEVER in cfg.mk. Objections to pushing? /Simon 2010-01-12 Simon Josefsson si...@josefsson.org

striconv typo

2010-01-12 Thread Simon Josefsson
I happened to run 'syntax-check' on some gnulib files, and notices this in lib/striconv.c: if (result != NULL) free (result); Bruno, do you object to removing the unnecessary if? /Simon

Cygwin build report on Windows XP

2010-01-12 Thread Simon Josefsson
FYI, a build of gnulib on cygwin on Windows XP just finished, a build report here: http://autobuild.josefsson.org/gnulib/log-201001121401712536000.txt I'll try to set the machine up to build it once per day or so, depending on how long it takes. The tests that failed were:

Re: striconv typo

2010-01-12 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: I happened to run 'syntax-check' on some gnulib files, and notices this in lib/striconv.c: if (result != NULL) free (result); Bruno, do you object to removing the unnecessary if? It is in a part of the code

<    1   2   3   4   5   6   7   8   9   10   >