maint.mk: tell sc_prohibit_strcmp to ding "0 == strcmp (...)", too
[gnulib.git] / ChangeLog
index cbd8637..681d773 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,369 @@
+2012-02-24  Jim Meyering  <meyering@redhat.com>
+
+       maint.mk: tell sc_prohibit_strcmp to ding "0 == strcmp (...)", too
+       * top/maint.mk (sc_prohibit_strcmp): Also prohibit uses of strcmp
+       uses with "==" *before* the call, e.g., 0 == strcmp (...)
+       Remove now-unnecessary str''cmp obfuscation.
+       Suggested by Akim Demaille.
+
+2012-02-24  Bruno Haible  <bruno@clisp.org>
+
+       streq: Rename macro.
+       * lib/streq.h (STREQ_OPT): Renamed from STREQ.
+       * NEWS: Mention the change.
+       * lib/mbrtowc.c (mbrtowc): Update.
+       * lib/uniwidth/cjk.h (is_cjk_encoding): Update.
+       * lib/wcwidth.c (wcwidth): Update.
+       Suggested by Akim Demaille and Jim Meyering.
+
+2012-02-20  Paul Eggert  <eggert@cs.ucla.edu>
+
+       regex: fix typo in definition of MIN
+       * lib/regex_internal.h (MIN): Fix typo.  Problem reported by Thomas
+       Schwinge in <http://sourceware.org/bugzilla/show_bug.cgi?id=11638#c4>.
+
+2012-02-19  Paul Eggert  <eggert@cs.ucla.edu>
+           Bruno Haible  <bruno@clisp.org>
+
+       acl: Don't use ACL_CNT and similar ops, since they are unreliable.
+       * lib/file-has-acl.c (file_has_acl) [HP-UX, NonStop Kernel]: Read the
+       entries into a stack-allocated buffer directly.
+       * lib/copy-acl.c (qcopy_acl) [HP-UX, NonStop Kernel]: Likewise.
+
+2012-02-19  Paul Eggert  <eggert@cs.ucla.edu>
+           Bruno Haible  <bruno@clisp.org>
+
+       acl: Don't use GETACLCNT and similar ops, since they are unreliable.
+
+        - There were several instances of this pattern:
+
+            for (;;) {
+              n = acl (f, GETACLCNT, 0, NULL);
+              [ allocate an array A of size N ]
+              if (acl (f, GETACL, n, a) == n)
+                break;
+            }
+
+          This loop might never terminate if some other process is constantly
+          manipulating the file's ACL.  The loop should be rewritten to
+          terminate.
+
+        - The acl (... GETACLNT ...) call is merely an optimization; its value
+          is merely a hint as to how big to make the array.  A better
+          optimization is to avoid the acl (... GETACLNT ...)  call entirely,
+          and just guess a reasonably-big size, growing the size and trying
+          again if it's not large enough.  This guarantees termination, and
+          saves a system call.
+
+       * lib/acl-internal.h: Include <limits.h>.
+       (MIN, SIZE_MAX): New macros.
+       * lib/file-has-acl.c (file_has_acl) [Solaris]: Read the entries into
+       a stack-allocated buffer, and use malloc if it does not fit. Don't
+       use GETACLCNT.
+       * lib/set-mode-acl.c (qset_acl) [Solaris]: Likewise.
+
+2012-02-19  Bruno Haible  <bruno@clisp.org>
+
+       acl: Fix endless loop on Solaris with vxfs.
+       * lib/file-has-acl.c (file_has_acl) [Solaris]: Treat a failing
+       acl()/facl() call for ACE_GETACL like a failing call for ACE_GETACLCNT.
+       * lib/set-mode-acl.c (qset_acl) [Solaris]: Likewise.
+       * lib/copy-acl.c (qcopy_acl)[Solaris]: Likewise.
+       * tests/test-sameacls.c (main)[Solaris]: Likewise.
+       Reported by Bill Jones in
+       <http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10639>, via Paul Eggert.
+
+2012-02-19  Bruno Haible  <bruno@clisp.org>
+
+       acl: Fix copy-acl test failure on Solaris 11 2011-11.
+       * lib/file-has-acl.c (NEW_ACE_WRITEA_DATA): New macro.
+       (acl_ace_nontrivial): Relax the restrictions on access_masks[] so
+       that this function returns 0 in some more cases.
+
+2012-02-19  Bruno Haible  <bruno@clisp.org>
+
+       acl: Update doc references.
+       * doc/acl-resources.txt: Update links to Solaris documentation.
+
+2012-02-19  Bruno Haible  <bruno@clisp.org>
+
+       Fix test failure in many locales on Solaris 11.
+       * tests/test-pipe-filter-gi1.c (main): Don't use range expression in
+       'tr' arguments.
+       * tests/test-pipe-filter-ii1.c (main): Likewise.
+       * build-aux/bootstrap (check_versions): Run 'tr' command with range
+       expressions in the C locale.
+       * m4/fnmatch.m4 (gl_FUNC_FNMATCH_POSIX): Likewise.
+       * m4/host-os.m4 (gl_HOST_OS): Likewise.
+
+2012-02-19  Bruno Haible  <bruno@clisp.org>
+
+       gnulib-tool: Improve usage message.
+       * gnulib-tool (func_usage): Move doc of --help and --version to the
+       section "Operation modes".
+
+2012-02-18  Reuben Thomas  <rrt@sc3d.org>
+
+       README-release: make it easier to execute commands
+       * top/README-release: break commands out on to separate lines.
+
+2012-02-16  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       GNUmakefile: simplify detection of unconfigured trees
+       * top/GNUmakefile: Use $(wildcard) instead of $(shell) to determine
+       whether the tree make is being run from is already configured or
+       not.  Related simplifications.
+
+2012-02-13  Simon Josefsson  <simon@josefsson.org>
+
+       * gnulib-tool (func_usage): Document --help and --version.
+
+2012-02-11  Jim Meyering  <meyering@redhat.com>
+
+       bootstrap: don't exit 0 upon gnulib-tool failure
+       * build-aux/bootstrap (gnulib_tool): If gnulib-tool fails, exit with
+       its exit status, not 0.
+
+2011-12-19  Reuben Thomas  <rrt@sc3d.org>
+
+       README-release: various improvements
+       * top/README-release: Give a command to push changes for the
+       release.  Add "distcheck" to list of other pre-release checks.
+       Fix instance of "make stable" which should be "make TYPE".
+
+2012-02-09  Paul Eggert  <eggert@cs.ucla.edu>
+
+       maint: replace FSF snail-mail addresses with URLs
+       * config/argz.mk, lib/accept4.c, lib/alignof.h, lib/alloca.in.h:
+       * lib/alphasort.c, lib/arcfour.c, lib/arcfour.h, lib/arctwo.c:
+       * lib/arctwo.h, lib/argz.c, lib/arpa_inet.in.h, lib/asnprintf.c:
+       * lib/asprintf.c, lib/assert.in.h, lib/base32.c, lib/base32.h:
+       * lib/base64.c, lib/base64.h, lib/c-ctype.c, lib/c-ctype.h:
+       * lib/c-strcase.h, lib/c-strcasecmp.c, lib/c-strncasecmp.c:
+       * lib/check-version.c, lib/check-version.h, lib/config.charset:
+       * lib/ctype.in.h, lib/des.c, lib/des.h, lib/dup3.c, lib/errno.in.h:
+       * lib/float+.h, lib/fnmatch.c, lib/fnmatch.in.h, lib/fnmatch_loop.c:
+       * lib/fseeko.c, lib/gai_strerror.c, lib/gc-gnulib.c:
+       * lib/gc-libgcrypt.c, lib/gc-pbkdf2-sha1.c, lib/gc.h:
+       * lib/getaddrinfo.c, lib/getdelim.c, lib/getfilecon.c, lib/getline.c:
+       * lib/getlogin_r.c, lib/getpass.c, lib/getpass.h, lib/gettext.h:
+       * lib/gettimeofday.c, lib/glob.in.h, lib/glthread/cond.c:
+       * lib/glthread/cond.h, lib/glthread/lock.c, lib/glthread/lock.h:
+       * lib/glthread/thread.c, lib/glthread/thread.h:
+       * lib/glthread/threadlib.c, lib/glthread/yield.h, lib/hmac-md5.c:
+       * lib/hmac-sha1.c, lib/hmac.h, lib/iconv.c, lib/iconv.in.h:
+       * lib/iconv_close.c, lib/iconv_open.c, lib/inet_ntop.c, lib/isfinite.c:
+       * lib/isinf.c, lib/iswblank.c, lib/langinfo.in.h, lib/link.c:
+       * lib/localcharset.c, lib/localcharset.h, lib/lseek.c, lib/malloc.c:
+       * lib/malloca.c, lib/malloca.h, lib/md2.c, lib/md2.h, lib/md4.c:
+       * lib/md4.h, lib/md5.c, lib/md5.h, lib/memmem.c, lib/mempcpy.c:
+       * lib/memset.c, lib/memxor.c, lib/memxor.h, lib/minmax.h, lib/mktime.c:
+       * lib/msvc-inval.c, lib/msvc-inval.h, lib/msvc-nothrow.c:
+       * lib/msvc-nothrow.h, lib/netdb.in.h, lib/netinet_in.in.h, lib/nproc.c:
+       * lib/nproc.h, lib/obstack_printf.c, lib/pathmax.h, lib/pipe.c:
+       * lib/pipe2.c, lib/poll.c, lib/poll.in.h, lib/printf-args.c:
+       * lib/printf-args.h, lib/printf-parse.c, lib/printf-parse.h:
+       * lib/pselect.c, lib/pthread.in.h, lib/pty-private.h, lib/pty.in.h:
+       * lib/read-file.c, lib/read-file.h, lib/ref-add.sin, lib/ref-del.sin:
+       * lib/regcomp.c, lib/regex.c, lib/regex.h, lib/regex_internal.c:
+       * lib/regex_internal.h, lib/regexec.c, lib/rijndael-alg-fst.c:
+       * lib/rijndael-alg-fst.h, lib/rijndael-api-fst.c:
+       * lib/rijndael-api-fst.h, lib/rint.c, lib/rintf.c, lib/rintl.c:
+       * lib/round.c, lib/roundf.c, lib/roundl.c, lib/scandir.c, lib/select.c:
+       * lib/sha1.c, lib/sha1.h, lib/size_max.h, lib/snprintf.c:
+       * lib/stdalign.in.h, lib/stdarg.in.h, lib/stdbool.in.h:
+       * lib/stddef.in.h, lib/stdint.in.h, lib/stdio.in.h, lib/str-kmp.h:
+       * lib/str-two-way.h, lib/strcasecmp.c, lib/strcasestr.c, lib/strdup.c:
+       * lib/striconv.c, lib/striconv.h, lib/string.in.h, lib/strings.in.h:
+       * lib/strncasecmp.c, lib/strndup.c, lib/strnlen.c, lib/strpbrk.c:
+       * lib/strptime.c, lib/strsep.c, lib/strstr.c, lib/strverscmp.c:
+       * lib/sys_file.in.h, lib/sys_ioctl.in.h, lib/sys_select.in.h:
+       * lib/sys_socket.in.h, lib/sys_stat.in.h, lib/sys_time.in.h:
+       * lib/sys_times.in.h, lib/sys_types.in.h, lib/sys_uio.in.h:
+       * lib/sys_utsname.in.h, lib/sys_wait.in.h, lib/tcgetsid.c:
+       * lib/termios.in.h, lib/time.in.h, lib/time_r.c, lib/timegm.c:
+       * lib/times.c, lib/unictype/3level.h, lib/unictype/3levelbit.h:
+       * lib/unistd.in.h, lib/vasnprintf.c, lib/vasnprintf.h, lib/vasprintf.c:
+       * lib/vsnprintf.c, lib/waitpid.c, lib/wchar.in.h, lib/wctype.in.h:
+       * lib/xsize.h, tests/test-closein.c, tests/test-des.c:
+       * tests/test-fclose.c, tests/test-fgetc.c, tests/test-filevercmp.c:
+       * tests/test-fputc.c, tests/test-fread.c, tests/test-fwrite.c:
+       * tests/test-gc-arcfour.c, tests/test-gc-arctwo.c, tests/test-gc-des.c:
+       * tests/test-gc-hmac-md5.c, tests/test-gc-hmac-sha1.c:
+       * tests/test-gc-md2.c, tests/test-gc-md4.c, tests/test-gc-md5.c:
+       * tests/test-gc-pbkdf2-sha1.c, tests/test-gc-rijndael.c:
+       * tests/test-gc-sha1.c, tests/test-gc.c, tests/test-getdelim.c:
+       * tests/test-getline.c, tests/test-getndelim2.c, tests/test-md2.c:
+       * tests/test-md4.c, tests/test-parse-datetime.c, tests/test-perror.c:
+       * tests/test-perror2.c, tests/test-pipe.c, tests/test-pipe2.c:
+       * tests/test-poll.c, tests/test-quotearg-simple.c:
+       * tests/test-quotearg.c, tests/test-quotearg.h:
+       * tests/test-round-ieee.c, tests/test-round1.c:
+       * tests/test-roundf-ieee.c, tests/test-roundf1.c:
+       * tests/test-roundl-ieee.c, tests/test-roundl.c:
+       * tests/test-safe-alloc.c, tests/test-sigpipe.c:
+       * tests/test-spawn-pipe-child.c, tests/test-spawn-pipe-main.c:
+       * tests/test-strerror.c, tests/test-strerror_r.c:
+       * tests/test-strsignal.c, tests/test-strverscmp.c:
+       * tests/test-xmemdup0.c:
+       Replace FSF snail mail addresses with URLs, as per GNU coding
+       standards.  See glibc bug
+       <http://sourceware.org/bugzilla/show_bug.cgi?id=13673>.
+
+2011-12-22  Reuben Thomas  <rrt@sc3d.org>
+
+       README-release: capitalize a word and split a line
+       * top/README-release: Fix punctuation and spacing.
+
+2012-02-08  Akim Demaille  <demaille@gostai.com>
+
+       fatal-signal: use C prototypes (with explicit void).
+       * lib/fatal-signal.c (uninstall_handlers, install_handlers)
+       (init_fatal_signal_set, block_fatal_signals): Fix signatures.
+
+2012-02-07  Paul Eggert  <eggert@cs.ucla.edu>
+
+       regex: spelling fix
+       * lib/regexec.c: spelling fix
+
+       regex: rely on stdint.h for SIZE_MAX
+       * lib/regex_internal.h (SIZE_MAX): Remove; stdint.h supplies this now.
+
+2012-02-07  Paul Eggert  <eggert@cs.ucla.edu>
+
+       regex: merge glibc changes
+
+       * lib/regcomp.c (init_dfa): Tighten overflow checks to test
+       for IDX_MAX too, since IDX_MAX can be much less than SIZE_MAX.
+       (init_word_char): Work even if bitset words are not exactly 32 or
+       64 bits wide.  Don't assume there are no padding bits.
+       * lib/regex.c [_LIBC]: Do not include <config.h>.
+       [!_LIBC]: Add pragmas to ignore -Wsuggest-attributes=pure
+       and -Wtype-limits.
+       * lib/regex.h (__USE_GNU): Renamed from __USE_GNU_REGEX, to avoid
+       needless disagreement with glibc.  All uses changed.  Define it to
+       1 only if _GNU_SOURCE, to match glibc.
+       (_REG_RM_NAME): Remove; no longer needed, since the names in
+       question are now all protected by __USE_GNU.
+       (_REG_RE_NAME): Remove; replaced by glibc's __REPB_PREFIX.
+       (REG_TRANSLATE_TYPE): Remove; replaced by glibc's __RE_TRANSLATE_TYPE.
+       * lib/regex_internal.h (MIN): New macro.
+
+       2012-01-03 Ulrich Drepper <drepper@gmail.com>
+       * lib/regcomp.c (init_word_char): Optimize regex a bit.
+
+       2011-12-30 Jakub Jelinek <jakub@redhat.com>
+       * lib/regex_internal.c (re_string_fetch_byte_case):
+       Fix up regcomp/regexec.  The problem is that parse_bracket_symbol
+       is miscompiled, and it turns out it is because of an incorrect
+       attribute on re_string_fetch_byte_case.  Unlike
+       re_string_peek_byte_case, this one is really not pure, it modifies
+       memory (increments pstr->cur_idx), and with the pure attribute GCC
+       assumed it doesn't and it cached the presumed value of
+       regexp->cur_idx in a variable across the
+        for (;; ++i)
+          {
+            if (i >= BRACKET_NAME_BUF_SIZE)
+              return REG_EBRACK;
+            if (token->type == OP_OPEN_CHAR_CLASS)
+              ch = re_string_fetch_byte_case (regexp);
+            else
+              ch = re_string_fetch_byte (regexp);
+            if (re_string_eoi(regexp))
+              return REG_EBRACK;
+            if (ch == delim && re_string_peek_byte (regexp, 0) == ']')
+              break;
+            elem->opr.name[i] = ch;
+          }
+
+       2011-11-29 Andreas Schwab <schwab@redhat.com>
+       * lib/regcomp.c (build_equiv_class):
+       Fix access after end of search string in regex matcher.
+
+       2011-11-12 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex_internal.c, lib/regex_internal.h: Fix warnings in regex.
+
+       2011-10-12 Ulrich Drepper <drepper@redhat.com>
+       * lib/regcomp.c (parse_branch): One more regex memory leak fixed.
+
+       2011-10-11 Ulrich Drepper <drepper@redhat.com>
+       * lib/regcomp.c (parse_branch, parse_sub_exp):
+       More regex memory leak fixes and tests.
+       (parse_sub_exp, parse_bracket_exp):
+       Fix memory leak for some invalid regular expressions.
+
+       2011-05-28 Ulrich Drepper <drepper@gmail.com>
+       * lib/regex_internal.c, lib/regexec.c:
+       Fix unnecessary overallocation due to incomplete character.  When
+       incomplete characters are found at the end of a string the code
+       ran amok and allocated lots of memory.  Stricter limits are now in
+       place.
+
+       2011-05-20 Reuben Thomas <rrt@sc3d.org>
+       * lib/regex.h: Update documentation.
+
+       2011-05-16 Aharon Robbins <arnold@skeeve.com>
+       * lib/regex.h: Update RE_SYNTAX*_AWK constants.
+
+       2010-05-05 Andreas Schwab <schwab@redhat.com>
+       * lib/regexec.c (find_collation_sequence_value):
+       Fix lookup of collation sequence value during regexp matching.
+
+       2010-01-22 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex_internal.c (re_dfa_add_node): Extend overflow detection.
+
+       2008-01-16 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex.h: Cleanup namespace.
+
+       2007-11-26 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex.h (REG_ENOSYS): Define REG_ENOSYS also for __USE_XOPEN2K.
+
+       2007-08-26 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex_internal.h: Prevent some declarations and definitions
+       to be seen when used in tests.
+
+       2005-05-06 Ulrich Drepper <drepper@redhat.com>
+       * lib/regex_internal.h: Include bits/libc-lock.h or define dummy
+       __libc_lock_* macros if not _LIBC.
+       (struct re_dfa_t): Add lock.
+
+2012-02-07  Eric Blake  <eblake@redhat.com>
+
+       maint.mk: also prohibit lower-case @var@
+       * top/maint.mk (sc_makefile_at_at_check): Enhance check to cover
+       lower case, like @top_srcdir@.
+
+2012-02-04  Eric Blake  <eblake@redhat.com>
+
+       canonicalize: avoid uninitialized memory use
+       * lib/canonicalize-lgpl.c (__realpath): Avoid possibility of
+       random '/' left in dest.
+       * lib/canonicalize.c (canonicalize_filename_mode): Likewise.
+
+2012-02-04  Bruno Haible  <bruno@clisp.org>
+
+       isatty: Fix test failure of ptsname_r on native Windows.
+       * lib/isatty.c (_isatty_nothrow): Upon exception, return 0, not -1,
+       and don't set errno.
+       (isatty): Test first whether fd is valid. Set errno when returning 0.
+
+2012-02-04  Bruno Haible  <bruno@clisp.org>
+
+       spawn-pipe tests: Fix a NULL program name in a diagnostic.
+       * tests/test-spawn-pipe-main.c: Include progname.h.
+       (main): Invoke set_program_name.
+       * modules/spawn-pipe-tests (Depends-on): Add progname.
+
+       nonblocking-socket tests: Fix a NULL program name in a diagnostic.
+       * tests/test-nonblocking-socket-main.c: Include progname.h.
+       (main): Invoke set_program_name.
+       * modules/nonblocking-socket-tests (Depends-on): Add progname.
+
+       nonblocking-pipe tests: Fix a NULL program name in a diagnostic.
+       * tests/test-nonblocking-pipe-main.c: Include progname.h.
+       (main): Invoke set_program_name.
+       * modules/nonblocking-pipe-tests (Depends-on): Add progname.
+
 2012-02-04  Eric Blake  <eblake@redhat.com>
 
        canonicalize-lgpl: fix // handling