No, probably not. I'll remove.


On Fri, Aug 30, 2013 at 4:20 PM, Adrien Thebo <notifications@github.com>wrote:

> In lib/puppet/provider/group/gpasswd.rb:
>
> > @@ -0,0 +1,69 @@
> > +Puppet::Type.type(:group).provide :gpasswd, :parent => Puppet::Type::Group::ProviderGroupadd do
> > +  require 'shellwords'
> > +
> > +  desc <<-EOM
> > +    Group management via `gpasswd`. This allows for local group
> > +    management when the users exist in a remote system.
> > +  EOM
> > +
> > +  commands  :add => "groupadd",
> > +            :delete => "groupdel",
> > +            :modify => "groupmod",
> > +            :addmember => 'gpasswd',
> > +            :delmember => 'gpasswd'
> > +
> > +  has_feature :system_groups unless %w{HP-UX Solaris}.include? Facter.value(:operatingsystem)
>
> Since this feature is defined on the parent class, does it need to be
> defined in this class as well?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/puppetlabs/puppet/pull/1848/files#r6095211>
> .
>



-- 
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
tvaughan@onyxpoint.com

-- This account not approved for unencrypted proprietary information --