Re: gnulib and threaded execution

2010-12-28 Thread Paul Eggert
On 12/02/2010 04:39 PM, Paul Eggert wrote: > This should be fixable by making 'addr' auto instead of static, no? > Something like the following (untested) patch. No further comment on that, so I installed it: >From b9d6ad8aec25d6b7a01a70b7ebc0dc7c221f1cf6 Mon Sep 17 00:00:00 2001 From: Paul Egger

[PATCH] tests: minor indenting change

2010-12-28 Thread Paul Eggert
* tests/init.sh: Sync from coreutils housekeeping patch to keep lines within 80 columns. --- ChangeLog |7 +++ tests/init.sh |5 +++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/Cha

test-nl_langinfo failure on Irix 6.5

2010-12-28 Thread Eric Blake
Bruno, I'm getting a test failure on Irix 6.5. The native nl_langinfo is being used: -e 's|@''HAVE_NL_LANGINFO''@|1|g' \ -e 's|@''REPLACE_NL_LANGINFO''@|0|g' \ and YESEXPR and NOEXPR are defined in , but the test fails on this line during the C locale run: LC_ALL=C ./test-nl_langin

[PATCH] mountlist: fix local drive detection on cygwin

2010-12-28 Thread Eric Blake
* lib/mountlist.c (ME_REMOTE) [__CYGWIN__]: Provide implementation that works for cygwin. Signed-off-by: Eric Blake --- Any objections to applying this patch? I used it when building the official cygwin port of coreutils 8.8. In cygwin's getmntent() implementation, all mount points are mapped

Re: linkat, LINK_FOLLOWS_SYMLINKS, and Solaris

2010-12-28 Thread Paul Eggert
On 12/27/2010 05:52 PM, Paul Eggert wrote: > I was thinking more along the lines "don't use -xc99=all". Following up on this idea, I installed a change into Autoconf so that Autoconf-generated 'configure' scripts will no longer default to -xc99=all when using Solaris cc. Please see

RE: alphasort on HP-NonStop

2010-12-28 Thread Joachim Schmitz
Thanks, I'll mention this to them. Bye, Jojo -Original Message- From: Paul Eggert [mailto:egg...@cs.ucla.edu] Sent: Tuesday, December 28, 2010 5:44 PM To: Joachim Schmitz Cc: 'Bruno Haible'; bug-gnulib@gnu.org Subject: Re: alphasort on HP-NonStop On 12/28/2010 01:59 AM, Joachim Schmitz

Re: alphasort on HP-NonStop

2010-12-28 Thread Paul Eggert
On 12/28/2010 01:59 AM, Joachim Schmitz wrote: > I see that gnulib uses strcoll() rather than strcmp(), which seems to be what > POSIX mandates. > I'll request a bug fix for these issues from HP NonStop development... When you do that, you might tell them that there's a possible time bomb there.

[PATCH] regex: don't infloop on persistent failing calloc

2010-12-28 Thread Jim Meyering
FYI, I found that grep would infloop in a restricted-memory environment, and that this regex bug was at the root of the problem. I discovered this with a grep binary built to use glibc's regex code. Attempting the same exploit using a grep binary built --with-included-regex, the problem did not ar

Re: new modules for grapheme cluster breaking

2010-12-28 Thread Ben Pfaff
Bruno Haible writes: >> > The next modules will have a higher-level API, I imagine. You're welcome >> > to discuss the new API with me, before you implement it. >> >> I am thinking of something like: >> size_t u<#>_grapheme_len (const uint<#>_t *s, size_t n); >> which would return the nu

Re: new modules for grapheme cluster breaking

2010-12-28 Thread Bruno Haible
Hello Ben, > (I also had to convert GraphemeBreakTest.txt from DOS to Unix > line-ends and remove trailing white space, because the Git hook > refused to accept it otherwise. This is good; I convert these files from ftp.unicode.org to Unix line-ends myself when I retrieve them, because these DOS

Re: linkat, LINK_FOLLOWS_SYMLINKS, and Solaris

2010-12-28 Thread Bruno Haible
Paul Eggert wrote: > it makes sense to install that patch regardless of the other > issues we are talking about. OK, I pushed it. Bruno

RE: alphasort on HP-NonStop

2010-12-28 Thread Joachim Schmitz
Hi Bruno I just had a look at the code (of the library for HP-NonStop), and it does indeed look as if they have just been forgotten to get documented and declared in in dirent.c, here's the code: int alphasort(struct dirent **d1, struct dirent **d2) { return(strcmp((*d1)->d_name, (*d2)