Re: gettext dependency?

2011-05-02 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: Simon Josefsson wrote: Gnulib's DEPENDENCIES says: * GNU gettext 0.16.1. Oops. that should be the newest release. Now it's 0.18.1.1. The file wasn't updated in a long time. I'm applying this patch: Thank you! /Simon

Re: remove 'exit'?

2011-05-02 Thread Simon Josefsson
k...@freefriends.org (Karl Berry) writes: FWIW, I also agree with just removing the module instead of trying to ever-escalate warnings. My experience is that few people are likely to deal with it until it becomes an error regardless of warning. Alas, I don't think it becomes an error even if

Re: question about correct usage of gnulib

2011-05-02 Thread Lorenzo Bettini
On 04/25/2011 06:36 PM, Bruno Haible wrote: Lorenzo Bettini wrote: on the manual I read These Gnulib substitute header files rely onconfig.h being already included. Furthermoreconfig.h must be the first include in every compilation unit. This means that to all your source files and likely

Re: [bug report] sharutils-4.1.1 only builds on linux and solaris

2011-05-02 Thread Bruno Haible
Gary V. Vaughan wrote: I notice that the latest sharutils relies on iconv.m4 and friends from gnulib to provide --with-libiconv-prefix and --with-libintl-prefix. But, these macros are assuming that you will link using libtool,. No, they don't. They define a variable for use with

Re: [bug report] sharutils-4.1.1 only builds on linux and solaris

2011-05-02 Thread Gary V. Vaughan
Hi Bruce, On Sun, May 01, 2011 at 01:46:18PM +, Gary V. Vaughan wrote: I notice that the latest sharutils relies on iconv.m4 and friends from gnulib to provide --with-libiconv-prefix and --with-libintl-prefix. But, these macros are assuming that you will link using libtool, which knows

[bug report] sharutils-4.1.1 only builds on linux and solaris

2011-05-02 Thread Gary V. Vaughan
Howdy Bruce! I notice that the latest sharutils relies on iconv.m4 and friends from gnulib to provide --with-libiconv-prefix and --with-libintl-prefix. But, these macros are assuming that you will link using libtool, which knows how to translate -R arguments into -R/-rpath/-Wl,-rpath/-blibpath

Re: verify fails on ia64-hpux11.23 in libunistring-0.9.3

2011-05-02 Thread Gary V. Vaughan
Hi Paul, Thanks for following up. On Sun, May 01, 2011 at 10:52:48AM -0700, Paul Eggert wrote: On 05/01/11 10:15, Gary V. Vaughan wrote: $ cat ,t.c #include stdint.h #include intprops.h int main (int argc, char **argv) { printf(%lu\n, (unsigned long) ULONG_MAX); printf(%lu\n,

Re: question about correct usage of gnulib

2011-05-02 Thread Bastien ROUCARIES
On Mon, May 2, 2011 at 12:02 PM, Lorenzo Bettini lorenzo.bett...@gmail.com wrote: On 04/25/2011 06:36 PM, Bruno Haible wrote: Lorenzo Bettini wrote: on the manual I read These Gnulib substitute header files rely onconfig.h  being already included. Furthermoreconfig.h  must be the first

Re: RFC socketpair

2011-05-02 Thread Paolo Bonzini
On 05/01/2011 04:29 PM, Bastien ROUCARIES wrote: Did you test if dup2 work for socket ? According tohttp://www.suacommunity.com/dictionary/dup-entry.php andhttp://blog.csdn.net/linlu11/archive/2008/06/06/2518571.aspx it will need more work. dup2 indeed may not work for a socket on Windows.

Re: VC-tag again

2011-05-02 Thread Reuben Thomas
On 1 May 2011 22:39, Reuben Thomas r...@sc3d.org wrote: On 30 April 2011 21:39, Reuben Thomas r...@sc3d.org wrote: Great, will modify patch for that. Attached. Just noticed I failed to switch the name of README-release back to that from README-release.in in modules file; updated patch

Re: [PATCH] fclose: don't fail on non-seekable input stream

2011-05-02 Thread Eric Blake
On 05/01/2011 04:38 AM, Bruno Haible wrote: In fact, the dependency on 'fflush' should not be needed, because the fixes of that module are not relevant here: ! Portability problems fixed by Gnulib: ! @itemize ! @item ! @code{fflush} followed by @code{fseek} or @code{fseeko}, applied to an

Re: [PATCH] fclose: don't fail on non-seekable input stream

2011-05-02 Thread Eric Blake
On 05/01/2011 04:21 AM, Bruno Haible wrote: Eric Blake wrote: + /* We only need to flush the file if it is not reading or if it is + seekable. */ + if ((!freading (fp) || fseeko (fp, 0, SEEK_CUR) == 0) fflush (fp)) Would it be equivalent to do if ((!freading (fp) || ftello

Re: fchdir dependencies

2011-05-02 Thread Eric Blake
On 05/01/2011 10:49 AM, Bruno Haible wrote: Does anyone see the reason for this dependency? Can something go wrong if I remove it? --- modules/fchdir.orig Sun May 1 18:48:07 2011 +++ modules/fchdirSun May 1 18:47:44 2011 @@ -15,7 +15,6 @@ fcntl-h filenamecat-lgpl

Re: verify fails on ia64-hpux11.23 in libunistring-0.9.3

2011-05-02 Thread Paul Eggert
On 05/02/11 05:09, Gary V. Vaughan wrote: example, it could be that limits.h (which intprops.h includes) redefines UINT_FAST8_MAX to the wrong value. That seems to me to be what is happening. Is this something that gnulib stdint.h can fix? It looks like a stupid mistake on the part of the

Re: [bug report] sharutils-4.1.1 only builds on linux and solaris

2011-05-02 Thread Bruce Korb
On 05/02/11 03:25, Bruno Haible wrote: The bug is that while sharutils does not use libtool, in src/Makefile.am it uses $(LTLIBINTL) when it should use $(LIBINTL). Fixed in source. Build under way. I'll re-release when I get the next opportunity. Thank you all. Regards, Bruce

[PATCH] freading: relax license from LGPLv3+ to LGPLv2+

2011-05-02 Thread Eric Blake
Agreed to in http://lists.gnu.org/archive/html/bug-gnulib/2011-05/msg00055.html * modules/freading (License): Relax LGPL version. Signed-off-by: Eric Blake ebl...@redhat.com --- Now pushed. ChangeLog|5 + modules/freading |3 +-- 2 files changed, 6 insertions(+), 2

Re: [PATCH] fclose: don't fail on non-seekable input stream

2011-05-02 Thread Eric Blake
On 05/02/2011 08:40 AM, Eric Blake wrote: ! @item ! @code{fflush} on an input stream changes the position of the stream to the ! end of the previous buffer, on some platforms: mingw. Likewise not relevant: the position of the stream is irrelevant in fclose(). Yes it is. test-closein has a

Re: remove 'exit'?

2011-05-02 Thread Eric Blake
On 05/02/2011 12:47 AM, Simon Josefsson wrote: k...@freefriends.org (Karl Berry) writes: FWIW, I also agree with just removing the module instead of trying to ever-escalate warnings. My experience is that few people are likely to deal with it until it becomes an error regardless of warning.

[PATCH] exit: drop remaining clients

2011-05-02 Thread Eric Blake
Anyone using EXIT_FAILURE needs to depend on stdlib, now that the exit module is gone. * modules/argmatch (Depends-on): Replace exit with stdlib. * modules/copy-file (Depends-on): Likewise. * modules/execute (Depends-on): Likewise. * modules/exitfail (Depends-on): Likewise. * modules/obstack

Re: URL module

2011-05-02 Thread Jose E. Marchesi
it seems like a good idea. AFAICS, url.h has many functions, which probably are not useful to expose in a gnulib module. What functions do you exactly need? For the disk filesystem module of libgnudpf we would be using url_parse (valid url checking), url_escape, url_scheme,

[PATCH] fclose: reduce dependencies

2011-05-02 Thread Eric Blake
In a non-GPL project, the last change to fclose dragged in inappropriate modules. * modules/fclose (Depends-on): Switch from fflush/fseeko to simpler lseek. * lib/fclose.c (rpl_fclose): Likewise. Reported by Simon Josefsson. Signed-off-by: Eric Blake ebl...@redhat.com --- Would it be

[PATCH] fclose: add some tests

2011-05-02 Thread Eric Blake
POSIX requires that fclose() on seekable input streams rewind back to the next byte not actually given to the application. Glibc fails this test. See: http://sourceware.org/bugzilla/show_bug.cgi?id=3746 http://sourceware.org/bugzilla/show_bug.cgi?id=12724 Likewise for FreeBSD. Cygwin 1.7.9 and

[PATCH] fflush: also replace fclose when fixing fflush

2011-05-02 Thread Eric Blake
This fixes the fclose failures detected in the previous patch, but only when the GPL fflush module is also in use. That is because the need for behavior of resetting seekable input streams is much less common, and the fix more complex. The LGPLv2+ test for fclose() in isolation is relaxed to

Doubled word check annoying in languages without statement separators

2011-05-02 Thread Reuben Thomas
It's all over my Lua. Really, it should only be checking the comments. Since most programs that know (roughly) what comments are seem to be either compilers, code line counters or syntax colorers, I'm not sure what accurate-ish tool one could use for this, but it seems a pity either to turn off

Re: Doubled word check annoying in languages without statement separators

2011-05-02 Thread Jim Meyering
Reuben Thomas wrote: It's all over my Lua. Really, it should only be checking the comments. Since most programs that know (roughly) what comments are seem to be either compilers, code line counters or syntax colorers, I'm not sure what accurate-ish tool one could use for this, but it seems a