+2013-06-11 Paul Eggert <eggert@cs.ucla.edu>
+
+ tests: port large-fd POSIX spawn tests to OS X
+ Problem reported by Daiki Ueno in
+ <http://lists.gnu.org/archive/html/bug-gnulib/2013-06/msg00031.html>.
+ * tests/test-posix_spawn_file_actions_addclose.c:
+ * tests/test-posix_spawn_file_actions_adddup2.c:
+ * tests/test-posix_spawn_file_actions_addopen.c:
+ Include <limits.h>, for OPEN_MAX, if available.
+ (big_fd): New static function.
+ (main): Use it.
+
+2013-06-04 Bernhard Voelker <mail@bernhard-voelker.de>
+
+ tests/nap.h: use an adaptive delay to avoid ctime update issues
+ The recent change in nap.h (5191133e) decreased the probability of lost
+ races to about a third, however such problems could still be observed
+ in virtual machines and openSUSE's OBS.
+ Before, nap() detected the needed time once empirically and then used
+ that delay (together with a small correction multiplier) in further
+ calls. This problem has been reported and discussed several times,
+ including guesses about possible kernel issues:
+ https://lists.gnu.org/archive/html/bug-gnulib/2013-04/msg00071.html
+ http://lists.gnu.org/archive/html/coreutils/2012-03/msg00088.html
+ https://lists.gnu.org/archive/html/bug-gnulib/2011-11/msg00226.html
+ http://bugs.gnu.org/12820
+ https://lists.gnu.org/archive/html/bug-gnulib/2010-11/msg00113.html
+ https://lists.gnu.org/archive/html/bug-gnulib/2009-11/msg00007.html
+ Now, nap() avoids the race alltogether by verifying on a reference
+ file whether a timestamp difference has happened.
+ * tests/nap.h (nap_fd): Define file descriptor variable for the
+ witness file.
+ (nap_works): Change return value to bool. Change passing
+ the old file's status by value instead of by reference as this function
+ does no longer update that timestamp; rename the function argument from
+ st to old_st. Remove the local variables cdiff and mdiff because that
+ function now returns true/false instead of the precise delay.
+ (guess_delay): Remove function.
+ (clear_tmp_file): Add new function to close and unlink the witness file.
+ (nap): Instead of re-using the delay which has been calculated during
+ the first call, avoid the race by actually verifying that a timestamp
+ difference can be observed on the current file system. Use an adaptive
+ approach for the delay to minimize execution time. Assert that the
+ maximum delay is <= ~2 seconds, more precisely sum(2^n) from 0 to 30
+ = 2^31 - 1 = 2.1s.
+ Use atexit to call clear_tmp_file when the process terminates.
+
+2013-06-02 Paul Eggert <eggert@cs.ucla.edu>
+
+ sig2str: port to C++
+ * lib/sig2str.h (sig2str, str2sig): Declare as extern "C".
+ Reported by Daniel J Sebald in
+ <http://lists.gnu.org/archive/html/bug-gnulib/2013-06/msg00000.html>.
+
+2013-05-30 Eric Blake <eblake@redhat.com>
+
+ docs: mention cygwin shortcoming in <sys/un.h>
+ * doc/posix-headers/sys_un.texi (sys/un.h): Mention problem.
+
+ vasnprintf: silence mingw compiler warning
+ * lib/vasnprintf.c (VASNPRINTF): Avoid unused variable warning.
+
+2013-05-29 Paul Eggert <eggert@cs.ucla.edu>
+
+ c-ctype, regex, verify: port to gcc -std=c90 -pedantic
+ Avoid constructions that are rejected by gcc -std=c90 -pedantic.
+ This fixes a porting bug I recently reintroduced in regex, and
+ some other instances that I discovered while testing the fix.
+ * lib/c-ctype.h [__STRICT_ANSI__]: Avoid ({ ... }).
+ * lib/regcomp.c (utf8_sb_map) [__STRICT_ANSI__]: Avoid [0 ... N] = E.
+ * lib/regex_internal.h [!_LIBC && GNULIB_LOCK]: Do not use a macro
+ with an empty argument if this is a pedantic pre-C99 GCC.
+ * lib/verify.h: Do not use _Static_assert if this is a pedantic
+ pre-C11 GCC.
+
+ regex: adapt to locking regime instead of depending on pthread
+ Instead of depending on pthread, adapt to whatever thread
+ modules are in use. Problem reported by Ludovic Courtès in
+ <http://lists.gnu.org/archive/html/bug-gnulib/2013-05/msg00082.html>
+ and by Mats Erik Andersson in
+ <http://lists.gnu.org/archive/html/bug-gnulib/2013-05/msg00100.html>.
+ * lib/regex_internal.h (lock_define, lock_init, lock_fini):
+ Support either the 'lock' module, or the 'pthread' module, or
+ no module.
+ (lock_lock, lock_unlock): New macros.
+ * lib/regexec.c (regexec, re_search_stub): Use the new macros.
+ * modules/lock, modules/pthread (configure.ac): Add module indicator.
+ * modules/regex (Depends-on): Remove pthread.
+
+2013-05-22 Eric Blake <eblake@redhat.com>
+
+ getgroups: document portability issues
+ * doc/glibc-functions/initgroups.texi (initgroups): Mention
+ multithread safety.
+ * doc/posix-functions/getpwuid.texi (getpwuid): Likewise.
+ * doc/posix-functions/getpwuid_r.texi (getpwuid_r): Likewise.
+ * doc/glibc-functions/getgrouplist.texi (getgrouplist): Mention
+ getugroups.
+ * doc/posix-functions/getgroups.texi (getgroups): Mention
+ multithread safety and mgetgroups.
+
+2013-05-22 Bernhard Voelker <mail@bernhard-voelker.de>
+
+ test-lchown, test-chown: also skip test if chown fails with EPERM
+ * tests/test-lchown.h (test_lchown): Add EPERM to the condition to
+ skip this test, to handle FAT file systems.
+ * tests/test-chown.h (test_chown): Likewise.
+
+2013-05-19 Paul Eggert <eggert@cs.ucla.edu>
+
+ regex: fix dfa race in multithreaded uses
+ Problem reported by Ludovic Courtès in
+ <http://lists.gnu.org/archive/html/bug-gnulib/2013-05/msg00058.html>.
+ * lib/regex_internal.h (lock_define, lock_init, lock_fini):
+ New macros. All uses of __libc_lock_define, __libc_lock_init
+ changed to use the first two of these.
+ (__libc_lock_lock, __libc_lock_unlock): New macros, for
+ non-glibc platforms.
+ (struct re_dfa_t): Define the lock unconditionally.
+ * lib/regexec.c (regexec, re_search_stub): Remove some now-incorrect
+ '#ifdef _LIBC"s.
+ * modules/regex (Depends-on): Add pthread, if we use the
+ included regex.
+
+ * lib/regcomp.c: Do actions that are not needed for glibc,
+ but may be needed elsewhere.
+ (regfree, re_compile_internal): Destroy the lock.
+ (re_compile_internal): Check for lock-initialization failure.
+
+ malloca: port to compilers that reject size-zero arrays
+ This fixes a bug introduced in my previous patch.
+ * lib/malloca.c (struct preliminary_header): Use an int
+ rather than a character array of size int; that's simpler.
+ (struct header): Remove, replacing with ...
+ (union header): New type. This avoids the need for declaring a
+ character array of size zero, which is not allowed on some platforms.
+ All uses changed.
+
2013-05-18 Paul Eggert <eggert@cs.ucla.edu>
parse-datetime, tests: don't use "string" + int
projects / gnulib.git / blobdiff