maint.mk: prohibit doubled words
[gnulib.git] / ChangeLog
index eeb8b8b..365dd2f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,7 +1,127 @@
+2011-04-10  Jim Meyering  <meyering@redhat.com>
+
+       maint.mk: prohibit doubled words
+       Detect them also when they're separated by a newline.
+       There are 3 ways to customize it:
+         - disable the test on a per file basis, as usual with rules using
+           $(VC_LIST_EXCEPT)
+         - replace the default doubled-word-selecting regexp (affects all files)
+         - ignore a particular file-vs-doubled-word match
+       I nearly used that last one to ignore the "is is" match in
+       coreutils' NEWS file, since the text was "ls -is is ..."
+       To do that, I would have added this line to cfg.mk:
+         ignore_doubled_word_match_RE_ = ^NEWS:[0-9]+:is[ ]is$
+       but it would have ignored any "is is" match in NEWS.
+       Low probability, but still...
+       Instead, I changed the text, slightly:
+         -  ls -is is now consistent with ls -lis in ignoring values returned
+         +  "ls -is" is now consistent with ls -lis in ignoring values returned
+       * top/maint.mk (prohibit_double_word_RE_): Provide default.
+       (prohibit_doubled_word_): Define.
+       (sc_prohibit_doubled_word): New rule.
+       (sc_prohibit_the_the): Remove.  Subsumed by the above.
+
+2011-04-10  Jim Meyering  <meyering@redhat.com>
+
+       maint: fix doubled-word typo in comment
+       * m4/gethostname.m4: s/is is/it is/
+       * m4/getdomainname.m4: Likewise.
+
+2011-04-10  Jim Meyering  <meyering@redhat.com>
+
+       maint: remove doubled word: s/it it/it/
+       * lib/stat-time.h (get_stat_birthtime): s/it it/it/
+
+2011-04-10  Jim Meyering  <meyering@redhat.com>
+
+       maint.mk: remove useless semicolon and backslash
+       * top/maint.mk (sc_prohibit_empty_lines_at_EOF): Remove stray
+       semicolon and backslash.
+
+2011-04-10  Bruno Haible  <bruno@clisp.org>
+
+       stdint test: Fix compilation failure on OSF/1 with DTK compiler.
+       * modules/stdint-tests (Depends-on): Add wchar.
+
+2011-04-10  Jim Meyering  <meyering@redhat.com>
+
+       maint: remove doubled words in comments, e.g., s/a a/a/
+       * lib/strptime.c (day_of_the_week): s/the the/the/
+       * tests/test-chown.h (test_chown): s/a a/a/
+
+       test-chown.h: correct a cast
+       * tests/test-chown.h (test_chown): Cast -1 to gid_t (not uid_t)
+       when the destination is a stat.st_gid.
+
+2011-04-09  Mats Erik Andersson  <mats.andersson@gisladisker.se>  (tiny change)
+
+       getaddrinfo: Fix test for sa_len member.
+       * m4/getaddrinfo.m4 (gl_PREREQ_GETADDRINFO): When testing for sa_len,
+       include <sys/types.h> before <sys/socket.h>.
+
+2011-04-09  Paul Eggert  <eggert@cs.ucla.edu>
+
+       maint: change "can not" to "cannot"
+       * doc/posix-functions/iconv.texi (iconv): This one crossed line
+       boundaries.
+
+2011-04-09  Jim Meyering  <meyering@redhat.com>
+
+       maint: change "a a" to "a"
+       * tests/test-lchown.h (test_lchown): s/a a/a/
+
+       maint.mk: prohibit \<the the\>
+       * top/maint.mk (sc_prohibit_the_the): New rule.
+
+       maint: fix "the the" in comment
+       * lib/count-one-bits.h: s/the the/the/
+
+       maint: change "can not" to "cannot"
+       But do not change the occurrences in maintain.texi or in
+       build-aux/po/Makefile.in.in, which I presume comes from gettext.
+       * doc/gnulib-tool.texi: s/can not/cannot/
+       * doc/posix-functions/accept.texi (accept): Likewise.
+       * doc/posix-functions/socket.texi (socket): Likewise.
+       * lib/mbrtowc.c: Likewise.
+
+       maint.mk: prohibit use of "can not"
+       * top/maint.mk (sc_prohibit_can_not): New rule.
+       Writing "can not" (rather than "cannot") is too common.  Prohibit it.
+
+2011-04-09  Bruno Haible  <bruno@clisp.org>
+
+       careadlinkat: Guard against misuse of careadlinkatcwd.
+       * lib/careadlinkat.c: Include <stdlib.h>.
+       (careadlinkatcwd): Check that the fd argument is as expected.
+
+2011-04-09  Bruno Haible  <bruno@clisp.org>
+
+       careadlinkat: Use common coding style.
+       * lib/careadlinkat.c: Move gnulib includes after system includes.
+
+2011-04-09  Bruno Haible  <bruno@clisp.org>
+
+       careadlinkat: Clarify specification.
+       * lib/careadlinkat.h (careadlinkat): Clarify preadlinkat argument.
+       (careadlinkatcwd): Add comment.
+       * lib/careadlinkat.c (careadlinkat): Clarify preadlinkat argument.
+
+2011-04-09  Bruno Haible  <bruno@clisp.org>
+
+       areadlinkat: Avoid link error on many platforms.
+       * modules/areadlinkat (Depends-on): Add areadlink.
+
+2011-04-09  Bruno Haible  <bruno@clisp.org>
+
+       allocator, careadlinkat: Fix double-inclusion guard.
+       * lib/allocator.h: Fix double-inclusion guard.
+       * lib/careadlinkat.h: Likewise.
+
 2011-04-09  Bruno Haible  <bruno@clisp.org>
 
        relocatable-prog-wrapper: Update after module 'areadlink' changed.
        * lib/relocwrapper.c: Update dependencies hierarchy.
+       * build-aux/install-reloc: Update list of files to be compiled.
        * modules/relocatable-prog-wrapper (Files): Add lib/careadlinkat.[hc],
        lib/allocator.[hc].
 
        * tests/test-open.h (test_open): Likewise.
        * doc/posix-headers/fcntl.texi (fcntl.h): Document the replacement.
 
-       careadlink: fix compilation error on mingw
+       careadlinkat: fix compilation error on mingw
        * lib/careadlinkat.c (standard_allocator): Avoid renaming fields
        within struct allocator.