Merge pull request #548 from mavit/solaris-user-groups

Fix appending to a user's group on Solaris, someone thought it was a list but it is really a set.
2014-12-23 13:52:32 -05:00 · 2014-12-23 13:52:32 -05:00 · a2c1daf00e
commit a2c1daf00e
parent 62a53b1209 18c429d016
1 changed files with 1 additions and 1 deletions
--- a/system/user.py
+++ b/system/user.py
@ -1255,7 +1255,7 @@ class SunOS(User):
                cmd.append('-G')
                new_groups = groups
                if self.append:
-                    new_groups.extend(current_groups)
+                    new_groups.update(current_groups)
                cmd.append(','.join(new_groups))

        if self.comment is not None and info[4] != self.comment: