From: Jim Meyering <jim@meyering.net>
Date: Tue, 26 Sep 2006 09:28:17 +0000 (+0000)
Subject: * NEWS: Mention the bug fix.
X-Git-Tag: v6.3~23
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5f65d05616fa46fa892c4ee550c6f4df2f4d0b03;p=thirdparty%2Fcoreutils.git

* NEWS: Mention the bug fix.
* src/groups.sh: Don't hide a write failure.
Reported by Iain Calder <ic56@rogers.com>.
---

diff --git a/ChangeLog b/ChangeLog
index 45d7f556d5..7586f54626 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 ab9386804d..50fbcc5f7b 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 c9acf82ecd..dc64d12f1e 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