Fix freebsd group provider regression introduced by b06510323102fcb35fb7b02891591b3a540ddf27 when there is no members while creating the group.

1 files changed, 9 insertions, 9 deletions

diff --git a/lib/chef/provider/group/pw.rb b/lib/chef/provider/group/pw.rb
index c39c20da67..048ba60bf6 100644
--- a/lib/chef/provider/group/pw.rb
+++ b/lib/chef/provider/group/pw.rb
@@ -40,19 +40,19 @@ class Chef
           command = "pw groupadd"
           command << set_options
 
-          # pw group[add|mod] -M is used to set the full membership list on a
-          # new or existing group. Because pw groupadd does not support the -m
-          # and -d options used by manage_group, we treat group creation as a
-          # special case and use -M.
-          Chef::Log.debug("#{@new_resource} setting group members: #{@new_resource.members.join(',')}")
-          member_options = [" -M #{@new_resource.members.join(',')}"]
+          unless @new_resource.members.empty?
+            # pw group[add|mod] -M is used to set the full membership list on a
+            # new or existing group. Because pw groupadd does not support the -m
+            # and -d options used by manage_group, we treat group creation as a
+            # special case and use -M.
+            Chef::Log.debug("#{@new_resource} setting group members: #{@new_resource.members.join(',')}")
+            member_options = [" -M #{@new_resource.members.join(',')}"]
 
-          if member_options.empty?
-            run_command(:command => command)
-          else
             member_options.each do |option|
               run_command(:command => command + option)
             end
+          else
+            run_command(:command => command)
           end
         end