summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog6
-rw-r--r--NEWS3
-rwxr-xr-xsrc/groups.sh4
3 files changed, 11 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 45d7f556d..7586f5462 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-09-26 Jim Meyering <jim@meyering.net>
+
+ * NEWS: Mention the bug fix.
+ * src/groups.sh: Don't hide a write failure.
+ Reported by Iain Calder <ic56@rogers.com>.
+
2006-09-25 Jim Meyering <jim@meyering.net>
* src/chown.c (usage): Clarify --dereference description.
diff --git a/NEWS b/NEWS
index ab9386804..50fbcc5f7 100644
--- a/NEWS
+++ b/NEWS
@@ -22,6 +22,9 @@ GNU coreutils NEWS -*- outline -*-
With --verbose (-v), cp and mv would sometimes generate no output,
or neglect to report file removal.
+ "groups user" now exits nonzero when it gets a write error.
+
+
* Major changes in release 6.2 (2006-09-18) [stable candidate]
** Changes in behavior
diff --git a/src/groups.sh b/src/groups.sh
index c9acf82ec..dc64d12f1 100755
--- a/src/groups.sh
+++ b/src/groups.sh
@@ -1,6 +1,6 @@
#!/bin/sh
# groups -- print the groups a user is in
-# Copyright (C) 1991, 1997, 2000, 2002, 2004 Free Software Foundation, Inc.
+# Copyright (C) 1991, 1997, 2000, 2002, 2004, 2006 Free Software Foundation, Inc.
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
@@ -61,7 +61,7 @@ else
groups=`id -Gn -- $name`
status=$?
if test $status = 0; then
- echo $name : $groups
+ echo $name : $groups || fail=1
else
fail=$status
fi