I'm not sure if that will work.

Check out the nameservice code since that's what's causing all of this
nonsense. I was trying to avoid rewriting everything down there just for
this small change.


On Mon, Oct 7, 2013 at 1:36 PM, Adrien Thebo <notifications@github.com>wrote:

> In lib/puppet/provider/group/gpasswd.rb:
>
> > +
> > +  has_feature :manages_members unless %w{HP-UX Solaris}.include? Facter.value(:operatingsystem)
> > +
> > +  def addcmd
> > +    cmd = []
> > +    cmd << super.shelljoin
> > +
> > +    @resource[:members] and cmd << @resource[:members].map{ |x|
> > +      [ command(:addmember),'-a',x,@resource[:name] ].shelljoin
> > +    }
> > +
> > +    # NOTE: This will create the group but may not add some members to
> > +    # your group if they do not exist on the system. Any member that
> > +    # can be added will be added in an attempt to do "the right
> > +    # thing".
> > +    cmd.join(' && ')
>
> Instead of extending the behavior of addcmd, why don't we extend the
> create method and add this logic after the group has been created? That
> way we don't have to shim all of this behavior into the addcmd method
> (since the addcmd method mainly deals with the creation of the main object,
> and the users are secondary objects).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/puppetlabs/puppet/pull/1848/files#r6803959>
> .
>



-- 
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
tvaughan@onyxpoint.com

-- This account not approved for unencrypted proprietary information --