Issue #8011 has been updated by Dominic Cleal.

I've just given it a try with this:
    hostclass :packages do
        package "foo",
            :ensure   => :present,
            :source   => [ "foo", "bar" ],
            :provider => :rpm
    end
    node 'default' do
        include 'packages'
    end

And a couple of lines in my rpm.rb provider:
    Puppet.warning("source is_a String? #{@resource[:source].is_a?(String)}")
    Puppet.warning("source is_a Array? #{@resource[:source].is_a?(Array)}")

And it seems it can be an array via the Ruby DSL:
    $ bin/puppet apply ~/tmp/dsl.rb 
    warning: source is_a String? false
    warning: source is_a Array? true

So it isn't unique to the unit tests.

I've added another commit to the branch that unit tests both ways.
----------------------------------------
Feature #8011: Support temp repo URL in pkgutil provider
https://projects.puppetlabs.com/issues/8011

Author: Dominic Cleal
Status: In Topic Branch Pending Review
Priority: Normal
Assignee: Jacob Helwig
Category: package
Target version: Telly
Affected Puppet version: 
Keywords: pkgutil solaris
Branch: https://github.com/domcleal/puppet/tree/tickets/master/8011b


The pkgutil provider should support using the "source" attribute of the package 
type as a temporary repo URL, then supply it to pkgutil with the -t option.

Other providers such as gem, pip, openbsd, freebsd and ports support this usage 
already.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to