On Fri, Mar 11, 2011 at 8:26 AM, James Turnbull <[email protected]> wrote: > > Signed-off-by: James Turnbull <[email protected]> > --- > lib/puppet/provider/package/aptitude.rb | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/lib/puppet/provider/package/aptitude.rb > b/lib/puppet/provider/package/aptitude.rb > index 8bdf984..2eafd3e 100755 > --- a/lib/puppet/provider/package/aptitude.rb > +++ b/lib/puppet/provider/package/aptitude.rb > @@ -12,6 +12,7 @@ Puppet::Type.type(:package).provide :aptitude, :parent => > :apt, :source => :dpkg > args.flatten! > # Apparently aptitude hasn't always supported a -q flag. > args.delete("-q") if args.include?("-q") > + args.delete("--force-yes") if args.include?("--force-yes") > output = aptitude(*args) > > # Yay, stupid aptitude doesn't throw an error when the package is missing.
At the least we need a comment in this provider indicating that this argument comes from the parent provider apt, and doesn't work with aptitude. I'll see if I can at least whip up a test for this specific case to get test coverage for the aptitude provider working. -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
