bug#14972: cp docs should mention permissions result when destination already exists

2013-07-28 Thread Pádraig Brady
Better to be explicit I suppose.
How about:

-In the absence of this option, each destination file is created with the
+In the absence of this option, the permissions of existing destination
+files are unchanged, while each new file is created with the
 mode bits of the corresponding source file, minus the bits set in the
 umask and minus the set-user-ID and set-group-ID bits.
 @xref{File permissions}.

thanks,
Pádraig.





bug#14972: cp docs should mention permissions result when destination already exists

2013-07-28 Thread jidanni
Thanks.





bug#14976: [patch] use semicolons in option descriptions, not periods

2013-07-28 Thread Benno Schulenberg

Hi,

Attached patch replaces in several usage texts some periods with
semicolons, removes from them several inconsistent final periods,
properly double-indents continuation lines, and rephrases two or
three sentences a little for increased clarity.

Regards,

Benno

-- 
http://www.fastmail.fm - Accessible with your email software
  or over the web

From 2bac2f05a68bc3534b54b480469f58bfd2489c8f Mon Sep 17 00:00:00 2001
From: Benno Schulenberg 
Date: Sun, 28 Jul 2013 13:54:20 +0200
Subject: [PATCH] textual: use semicolon instead of period in option descriptions

Also do not end option descriptions with a period, properly indent
continuation lines, and make some tiny clarifications.

* src/du.c (usage): Lowercase after semicolon.
* src/ls.c (usage): Semicolons instead of periods, small rephrasing
and two hyphens for clarity, proper indentation.
* src/mktemp.c (usage): Semicolons and lowercase.
* src/od.c (usage): Semicolons.
* src/ptx.c (usage): Use the standard phrase, clarify default option.
* src/setuidgid.c (usage): Properly indent continuation line.
* src/split.c (usage): Semicolons, lowercase, no final period.
* src/stat.c (usage): Semicolons, lowercase.
* src/tail.c (usage): Proper indentation, one shorter rephrasing,
semicolons, no final periods.
* src/timeout.c (usage): Properly indent, semicolons, no final periods.
---
 src/du.c|4 ++--
 src/ls.c|   28 ++--
 src/mktemp.c|   12 ++--
 src/od.c|6 +++---
 src/ptx.c   |2 +-
 src/setuidgid.c |2 +-
 src/split.c |8 
 src/stat.c  |4 ++--
 src/tail.c  |   21 ++---
 src/timeout.c   |   16 
 10 files changed, 51 insertions(+), 52 deletions(-)

diff --git a/src/du.c b/src/du.c
index 9f1f98c..849ddab 100644
--- a/src/du.c
+++ b/src/du.c
@@ -312,8 +312,8 @@ Summarize disk usage of each FILE, recursively for directories.\n\
 "), stdout);
   fputs (_("\
   --files0-from=F   summarize disk usage of the NUL-terminated file\n\
-  names specified in file F;\n\
-  If F is - then read names from standard input\n\
+  names specified in file F; if F is -, then read\n\
+  names from standard input\n\
   -Hequivalent to --dereference-args (-D)\n\
   -h, --human-readable  print sizes in human readable format (e.g., 1K 234M 2G)\
 \n\
diff --git a/src/ls.c b/src/ls.c
index e341c67..2936498 100644
--- a/src/ls.c
+++ b/src/ls.c
@@ -4758,19 +4758,19 @@ Sort entries alphabetically if none of -cftuvSUX nor --sort is specified.\n\
   -b, --escape   print C-style escapes for nongraphic characters\n\
 "), stdout);
   fputs (_("\
-  --block-size=SIZE  scale sizes by SIZE before printing them.  E.g.,\n\
+  --block-size=SIZE  scale sizes by SIZE before printing them; e.g.,\n\
'--block-size=M' prints sizes in units of\n\
-   1,048,576 bytes.  See SIZE format below.\n\
+   1,048,576 bytes; see SIZE format below\n\
   -B, --ignore-backups   do not list implied entries ending with ~\n\
   -c with -lt: sort by, and show, ctime (time of last\n\
-   modification of file status information)\n\
-   with -l: show ctime and sort by name\n\
+   modification of file status information);\n\
+   with -l: show ctime and sort by name;\n\
otherwise: sort by ctime, newest first\n\
 "), stdout);
   fputs (_("\
   -C list entries by columns\n\
   --color[=WHEN] colorize the output.  WHEN defaults to 'always'\n\
-   or can be 'never' or 'auto'.  More info below\n\
+   or can be 'never' or 'auto'; more info below\n\
   -d, --directorylist directory entries instead of contents,\n\
and do not dereference symbolic links\n\
   -D, --diredgenerate output designed for Emacs' dired mode\n\
@@ -4788,8 +4788,8 @@ Sort entries alphabetically if none of -cftuvSUX nor --sort is specified.\n\
 "), stdout);
   fputs (_("\
   --group-directories-first\n\
- group directories before files.\n\
-   augment with a --sort option, but any\n\
+ group directories before files;\n\
+   can be augmented with a --sort option, but any\n\
use of --sort=none (-U) disables grouping\n\
 "), stdout);
   fputs (_("\
@@ -4803,7 +4803,7 @@ Sort entries alphabetically if none of -cftuvSUX nor --sort is specified.\n\
  follow symbolic links listed 

bug#14977: texinfo build fails due to missing "@item", after du/inode change

2013-07-28 Thread Assaf Gordon

Hello,

Trying to compile the latest revision (v8.21-102-g9e52eba), I get the following 
error:
===
$ make
<< snip >>
test -f doc/.dirstamp || make  doc/.dirstamp
make[3]: Entering directory `/home/gordon/temp/coreutils'
make[3]: Leaving directory `/home/gordon/temp/coreutils'
Updating ./doc/version.texi
  GEN  doc/constants.texi
  MAKEINFO doc/coreutils.info
./doc/coreutils.texi:11427: @itemx must follow @item
make[2]: *** [doc/coreutils.info] Error 1
make[2]: Leaving directory `/home/gordon/temp/coreutils'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/gordon/temp/coreutils'
===

I assume it's related to the recent du/inode commit 
(333dc83d52e014a0b532e316ea8cd93b048f1ac6),
The "--inode" option in the docs has only "@itemx":
===
$ sed -n 11425,11430p ./doc/coreutils.texi
@optHumanReadable

@itemx --inodes
@opindex --inodes
@cindex inode usage, dereferencing in @command{du}
List inode usage information instead of block usage.
===

I'm using makeinfo version 5.1 from stock debian package.

Simply renaming the "@itemx" to "@item" works around this issue.

-gordon


diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index 40bff7a..fb36866 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -11424,7 +11424,7 @@ Equivalent to @option{--dereference-args} (@option{-D}).
 
 @optHumanReadable
 
-@itemx --inodes

+@item --inodes
 @opindex --inodes
 @cindex inode usage, dereferencing in @command{du}
 List inode usage information instead of block usage.








bug#14977: texinfo build fails due to missing "@item", after du/inode change

2013-07-28 Thread Paul Eggert
Thanks, I fixed that typo.