[clamav] use 'groupmems', not 'usermod'

ensc ensc at fedoraproject.org
Wed Aug 11 11:14:17 UTC 2010


commit 1b9c20a71e26c3e39e8f6488566f83be1c81bb78
Author: Enrico Scholz <enrico.scholz at informatik.tu-chemnitz.de>
Date:   Wed Aug 11 13:12:59 2010 +0200

    use 'groupmems', not 'usermod'
    
    'usermod' does not work when user does not exist in local /etc/passwd
    but e.g. in LDAP

 clamav.spec |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
---
diff --git a/clamav.spec b/clamav.spec
index cd0a2c1..80d5a9c 100644
--- a/clamav.spec
+++ b/clamav.spec
@@ -541,7 +541,7 @@ test -e %freshclamlog || {
 %triggerin milter -- clamav-scanner
 # Add the milteruser to the scanuser group; this is required when
 # milter and clamd communicate through local sockets
-/usr/sbin/usermod -a -G %scanuser %milteruser &>/dev/null || :
+/usr/sbin/groupmems -g %scanuser -a %milteruser &>/dev/null || :
 
 %pre milter
 %__fe_groupadd 5 -r %milteruser &>/dev/null || :
@@ -709,6 +709,8 @@ test "$1" != "0" || /sbin/initctl -q stop clamav-milter || :
 * Wed Aug 11 2010 Enrico Scholz <enrico.scholz at informatik.tu-chemnitz.de>
 - removed old %%trigger which renamed the 'clamav' user- and groupnames
   to 'clamupdate'
+- use 'groupmems', not 'usermod' to add a user to a group because
+  'usermod' does not work when user does not exist in local /etc/passwd
 
 * Tue Jul 13 2010 Dan Horák <dan[at]danny.cz> - 0.96.1-1401
 - ocaml not available (at least) on s390(x)


More information about the scm-commits mailing list