Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Ralf Wildenhues
* Karl Berry wrote on Fri, Aug 11, 2006 at 12:51:10AM CEST: > > 1) First, the order of subsections output in GCS is inconsistent between > the DVI and the info output (this does not happen with sections as in > the GNU make manual, but only with the section-lowering as done for the >

Re: snprintf fixes in gnulib

2006-08-10 Thread Paul Eggert
Simon Josefsson <[EMAIL PROTECTED]> writes: > Looks good to me. I think there were several suggestions on the > original code, and too little sanity checking on the end result after > applying all patches; perhaps it would be a good idea to add Bruno as > author too. OK, thanks, I've added him (

Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Karl Berry
Hi Ralf and all, 1) First, the order of subsections output in GCS is inconsistent between the DVI and the info output (this does not happen with sections as in the GNU make manual, but only with the section-lowering as done for the GCS; makeinfo bug lingering here?). Sorry, I don'

Re: [bug-gnulib] wcwidth on OSF/1

2006-08-10 Thread Albert Chin
On Mon, Aug 07, 2006 at 08:47:38PM +0200, Bruno Haible wrote: > Gary V. Vaughan wrote: > > Here is the culprit: > > > > configure:14791: checking whether wcwidth is declared > > configure:14828: cc -c -g conftest.c >&5 > > cc: Error: /usr/include.dtk/wchar.h, line 78: Ill-formed parameter type >

Re: snprintf fixes in gnulib

2006-08-10 Thread Simon Josefsson
Paul Eggert <[EMAIL PROTECTED]> writes: > I installed the following (and made myself a maintainer while we're at > it :-). Simon, if you object, I'll back out this patch, but I hope > you like it. Looks good to me. I think there were several suggestions on the original code, and too little sani

Re: importing regex changes from libc

2006-08-10 Thread Paul Eggert
Paul Eggert <[EMAIL PROTECTED]> writes: > I may have found a bug in the gnulib regex code, It turned out to be a bug in coreutils/src/csplit.c. Sorry about the false alarm.

importing regex changes from libc

2006-08-10 Thread Paul Eggert
I may have found a bug in the gnulib regex code, but the first step before fixing it is to import recent changes from libc, so I installed this: 2006-08-10 Paul Eggert <[EMAIL PROTECTED]> Import the following changes from libc: 2006-06-02 Jakub Jelinek <[EMAIL PROTECTED]>

snprintf fixes in gnulib

2006-08-10 Thread Paul Eggert
coreutils just now started using the gnulib snprintf module and I noticed some problems with it. The first thing I noticed was this comment about two size_t variables: if (len > size - 1) /* equivalent to: (size > 0 && len >= size) */ The comment is not correct if size_t promotes to int, and

Re: sync from gnulib to coreutils

2006-08-10 Thread Paul Eggert
Eric Blake <[EMAIL PROTECTED]> writes: > Now that getaddrinfo has more module dependencies upstream, we need to > pull in new files. OK, I grabbed some more files and installed the following change into coreutils along with these files. This led me to look at the snprintf module for the first ti

Re: 'restrict' module removed

2006-08-10 Thread Simon Josefsson
Paul Eggert <[EMAIL PROTECTED]> writes: > Now that we're assuming Autoconf 2.59 or later, we can remove the > 'restrict' module. (Actually, it is needed only for pre-2.58.) I > installed the following. This steps on the toes of Simon, I believe > -- if you object please let me know, but it is a

Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Karl Berry
As mentioned in standards.texi please send a "diff -c" patch to ; Yes, although in practice bug-standards is me (filtering for rms), and I'm obviously on bug-gnulib too, so no need to do it right away :). I'll read the actual diff shortly.

Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Paul Eggert
Ralf Wildenhues <[EMAIL PROTECTED]> writes: > Does this need to go by RMS, and if yes, is there > somebody to do this for me, after this has been improved to a phrasing > that finds consensus? As mentioned in standards.texi please send a "diff -c" patch to ; it'll eventually go by RMS. The patch

Re: FYI: new single-.h-file modules: dev-ino, same-inode

2006-08-10 Thread Jim Meyering
Ralf Wildenhues <[EMAIL PROTECTED]> wrote: > m4/cycle-check.m4 does not exist in gnulib at the moment. Thanks, Ralf. I've just added it: * cycle-check.m4: New file. Require gl_STRUCT_DEV_INO and gl_SAME_INODE. --- #serial 3 dnl Copyright (C) 2005, 2006 Free Software

Re: FYI: new single-.h-file modules: dev-ino, same-inode

2006-08-10 Thread Ralf Wildenhues
Hello Jim, * Jim Meyering wrote on Thu, Aug 10, 2006 at 03:45:46PM CEST: > 2006-08-10 Jim Meyering <[EMAIL PROTECTED]> > > + * modules/same-inode: New module. > + * modules/dev-ino: New module. > + * modules/cycle-check: Depend on these modules, rather than simply > + includin

Re: [bug-gnulib] Re: new module proposal: verror

2006-08-10 Thread Bruno Haible
Eric Blake wrote: > Checking in as follows: > > 2006-08-10 Eric Blake <[EMAIL PROTECTED]> > and Ralf Wildenhues <[EMAIL PROTECTED]> > > * gnulib-tool (func_import): Detect unexpanded macros in gnulib > namespace. Hmm, this may make a "gnulib-tool --import" fail while the corr

Re: new module proposal: verror

2006-08-10 Thread Eric Blake
Ralf Wildenhues gmx.de> writes: > > So, better would be > m4_pattern_forbid([^gl_[A-Z]]) > m4_pattern_allow([^gl_ES]) > > the first of which is similarly used in coreutils/m4/prereq.m4 > already (which in turn was part of Gnulib for a while). Agreed. Checking in as follows: 2006-08-10 E

Re: new module proposal: verror

2006-08-10 Thread Ralf Wildenhues
Hello Eric, * Eric Blake wrote on Thu, Aug 10, 2006 at 03:08:38PM CEST: > > * gnulib-tool (func_import): Check for unexpanded gl_ macros. This looks like a very good idea to me, in general; minor issues: - gl_ES is a valid locale name, - `LC_ALL=C grep "gl_[abd-z]" gnulib/m4/*' shows many

FYI: new single-.h-file modules: dev-ino, same-inode

2006-08-10 Thread Jim Meyering
The two .h files have been in gnulib for a while. This change gives each its own module. Ralf posted a patch with part of this change a few weeks ago. Index: ChangeLog === RCS file: /sources/gnulib/gnulib/ChangeLog,v retrieving revisi

FYI: chown: add missing dependent

2006-08-10 Thread Jim Meyering
2006-08-10 Jim Meyering <[EMAIL PROTECTED]> * modules/chown (Depends-on): Add stat-macros. Index: modules/chown === RCS file: /sources/gnulib/gnulib/modules/chown,v retrieving revision 1.6 diff -u -p -r1.6 chown --- modules

Re: new module proposal: verror

2006-08-10 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Eric Blake on 8/8/2006 3:59 PM: >> 2006-08-08 Eric Blake byu.net> >> >> * modules/verror: New module. >> * MODULES.html.sh: Document it. And another followup, since configure is now outputting: checking for vasprintf... yes

Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Ralf Wildenhues
Hello Bruno, * Bruno Haible wrote on Thu, Aug 10, 2006 at 02:14:39PM CEST: > Ralf Wildenhues wrote: > > * standards.texi (Configuration): Document that `configure' > > should accept arguments of the form `VARIABLE=VALUE' and why > > they are preferable over environment variables. > >

Re: CC=cc ./configure or ./configure CC=cc

2006-08-10 Thread Bruno Haible
Ralf Wildenhues wrote: > * standards.texi (Configuration): Document that `configure' > should accept arguments of the form `VARIABLE=VALUE' and why > they are preferable over environment variables. > Suggested by Bruno Haible. Looks quite good. Just two proposed further cha