[Puppet-dev] [PATCH/puppet 25/26] Revert "(#4258) Bug fix: populating instances for aliases"

2011-04-04 Thread Juerg Walz
This reverts commit 517e30539c8b23ed4d91fe082e9c64bb61d5b56b. Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 26/26] (#4258) Check wgetopts in pkgutil.conf

2011-04-04 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 30f23e0..08d74df

[Puppet-dev] [PATCH/puppet 23/26] (#4258) Bug fix: populating instances for aliases

2011-04-04 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 350cacc..97625f2 100755

[Puppet-dev] [PATCH/puppet 21/26] (#4258) Update pkgutil spec for recent impl changes

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Fix test output of pkgutil for commands using --single Fix expected upgrade command to match impl Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 22 +- 1 files changed, 9 insertions

[Puppet-dev] [PATCH/puppet 22/26] (#4258) Use pkgutil -a to reliably determine package common names/aliases

2011-04-04 Thread Juerg Walz
detection. Updated pkgutil_spec test to show various edge cases. Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 86 --- spec/unit/provider/package/pkgutil_spec.rb | 60 +-- 2 files

[Puppet-dev] [PATCH/puppet 24/26] (#4258) Fix hash duplication affecting canonical provider instance

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 20/26] (#4258) pkgutil: bug fix: if shortname is not equal to package name

2011-04-04 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 4a87932..3a23796 100755

[Puppet-dev] [PATCH/puppet 13/26] Changing indentation to 2-spaces as per 2.6+ style

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 212 1 files changed, 106 insertions(+), 106 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 16/26] * Fix exception on parse failure of pkgutil output * Fix exception when querying latest version for unknown package

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 19/26] (#4258) pkgutil provider: better handling of short package names

2011-04-04 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 0e2056b..4a87932 100755

[Puppet-dev] [PATCH/puppet 18/26] (#4258) pkgutil provider: misc enhancements

2011-04-04 Thread Juerg Walz
uot;) - use the "-u" command line switch for updates Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.r

[Puppet-dev] [PATCH/puppet 17/26] Add spec tests for pkgutil package provider

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 132 1 files changed, 132 insertions(+), 0 deletions(-) create mode 100755 spec/unit/provider/package/pkgutil_spec.rb diff --git

[Puppet-dev] [PATCH/puppet 14/26] Removing blastwave references and unused PAGER

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 18 -- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 15/26] Fixing indentation

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 11/26] Fixing wget verbose regex

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 10/26] These regular expressions will not match anything. pkgutil doesn't output anything that can be matched.

2011-04-04 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 1d699f3

[Puppet-dev] [PATCH/puppet 12/26] Single package queries made more robust when dealing with pkgutil noise

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 08/26] Ignoring lines from use_gpg and catalog fetching

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 09/26] Add comments that explain what we are ignoring in the package and remove legacy output

2011-04-04 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 05/26] pkgutil provider: The path to the admin file is /var/opt/csw/pkgutil/admin

2011-04-04 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 06/26] pkgutil provider: Using the --single option which speeds up execution.

2011-04-04 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 07/26] Adding patch from Rudy Gevaert to fix not installed detection

2011-04-04 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 04/26] pkgutil provider: Correcting a typo in a message.

2011-04-04 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 01/26] Fixed #4258 - Added pkgutil package provider

2011-04-04 Thread Juerg Walz
From: James Turnbull Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 117 1 files changed, 117 insertions(+), 0 deletions(-) create mode 100644 lib/puppet/provider/package/pkgutil.rb diff --git a

[Puppet-dev] [PATCH/puppet 03/26] Using --single in the pkgutil provider.

2011-04-04 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 02/26] Adding pkgutil support.

2011-04-04 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 111 1 files changed, 111 insertions(+), 0 deletions(-) create mode 100755 lib/puppet/provider/package/pkgutil.rb diff --git

[Puppet-dev]

2011-04-04 Thread Juerg Walz
This patch adds a check to the pkgutil provider to make sure the wget options (in pkgutil.conf) include -nv to give us the optimal level of verbosity from wget. I've merged Dominic Cleal's latest changes before working on this patch. https://github.com/gw42/puppet/tree/tickets/master/4258-dev

[Puppet-dev] Re: [PATCH/puppet 23/23] (#4258) Bug fix: populating instances for aliases

2011-03-22 Thread Juerg Walz
On Mar 23, 6:53 am, Dominic Cleal wrote: > Yep, fixed with pkg.dup. Works for me, too. I've ran a few test with package names and aliases. Juerg -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to puppet

[Puppet-dev] [PATCH/puppet 01/23] Fixed #4258 - Added pkgutil package provider

2011-03-20 Thread Juerg Walz
From: James Turnbull Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 117 1 files changed, 117 insertions(+), 0 deletions(-) create mode 100644 lib/puppet/provider/package/pkgutil.rb diff --git a

[Puppet-dev] [PATCH/puppet 23/23] (#4258) Bug fix: populating instances for aliases

2011-03-20 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 350cacc..97625f2 100755

[Puppet-dev] [PATCH/puppet 10/23] These regular expressions will not match anything. pkgutil doesn't output anything that can be matched.

2011-03-20 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 1d699f3

[Puppet-dev] [PATCH/puppet 21/23] (#4258) Update pkgutil spec for recent impl changes

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Fix test output of pkgutil for commands using --single Fix expected upgrade command to match impl Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 22 +- 1 files changed, 9 insertions

[Puppet-dev] [PATCH/puppet 22/23] (#4258) Use pkgutil -a to reliably determine package common names/aliases

2011-03-20 Thread Juerg Walz
detection. Updated pkgutil_spec test to show various edge cases. Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 86 --- spec/unit/provider/package/pkgutil_spec.rb | 60 +-- 2 files

[Puppet-dev] [PATCH/puppet 19/23] (#4258) pkgutil provider: better handling of short package names

2011-03-20 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 0e2056b..4a87932 100755

[Puppet-dev] [PATCH/puppet 20/23] (#4258) pkgutil: bug fix: if shortname is not equal to package name

2011-03-20 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 4a87932..3a23796 100755

[Puppet-dev] [PATCH/puppet 18/23] (#4258) pkgutil provider: misc enhancements

2011-03-20 Thread Juerg Walz
uot;) - use the "-u" command line switch for updates Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.r

[Puppet-dev] [PATCH/puppet 17/23] Add spec tests for pkgutil package provider

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 132 1 files changed, 132 insertions(+), 0 deletions(-) create mode 100755 spec/unit/provider/package/pkgutil_spec.rb diff --git

[Puppet-dev] [PATCH/puppet 14/23] Removing blastwave references and unused PAGER

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 18 -- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 15/23] Fixing indentation

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 13/23] Changing indentation to 2-spaces as per 2.6+ style

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 212 1 files changed, 106 insertions(+), 106 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 16/23] * Fix exception on parse failure of pkgutil output * Fix exception when querying latest version for unknown package

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 09/23] Add comments that explain what we are ignoring in the package and remove legacy output

2011-03-20 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 07/23] Adding patch from Rudy Gevaert to fix not installed detection

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 12/23] Single package queries made more robust when dealing with pkgutil noise

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 11/23] Fixing wget verbose regex

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 08/23] Ignoring lines from use_gpg and catalog fetching

2011-03-20 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 05/23] pkgutil provider: The path to the admin file is /var/opt/csw/pkgutil/admin

2011-03-20 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 02/23] Adding pkgutil support.

2011-03-20 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 111 1 files changed, 111 insertions(+), 0 deletions(-) create mode 100755 lib/puppet/provider/package/pkgutil.rb diff --git

[Puppet-dev] [PATCH/puppet 06/23] pkgutil provider: Using the --single option which speeds up execution.

2011-03-20 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 04/23] pkgutil provider: Correcting a typo in a message.

2011-03-20 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 03/23] Using --single in the pkgutil provider.

2011-03-20 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev]

2011-03-20 Thread Juerg Walz
Small bug fix for Dominic Cleals latest changes (to populate pkg instances for aliases): the hash "cloning" didn't work for me as expected: notice: /Stage[main]/Base_system::Common_pkgs/Package[screen]/ensure: is ensure4.0.3,REV=2010.06.08providerpkgutilnameCSWscrn, should be 4.0.3,REV=2010.06.

[Puppet-dev] [PATCH/puppet 15/20] Fixing indentation

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 19/20] (#4258) pkgutil provider: better handling of short package names

2011-03-10 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 0e2056b..4a87932 100755

[Puppet-dev] [PATCH/puppet 13/20] Changing indentation to 2-spaces as per 2.6+ style

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 212 1 files changed, 106 insertions(+), 106 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 20/20] (#4258) pkgutil: bug fix: if shortname is not equal to package name

2011-03-10 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 4a87932..3a23796 100755

[Puppet-dev] [PATCH/puppet 03/20] Using --single in the pkgutil provider.

2011-03-10 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 16/20] * Fix exception on parse failure of pkgutil output * Fix exception when querying latest version for unknown package

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 04/20] pkgutil provider: Correcting a typo in a message.

2011-03-10 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 02/20] Adding pkgutil support.

2011-03-10 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 111 1 files changed, 111 insertions(+), 0 deletions(-) create mode 100755 lib/puppet/provider/package/pkgutil.rb diff --git

[Puppet-dev] [PATCH/puppet 17/20] Add spec tests for pkgutil package provider

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 132 1 files changed, 132 insertions(+), 0 deletions(-) create mode 100755 spec/unit/provider/package/pkgutil_spec.rb diff --git

[Puppet-dev] [PATCH/puppet 09/20] Add comments that explain what we are ignoring in the package and remove legacy output

2011-03-10 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 18/20] (#4258) pkgutil provider: misc enhancements

2011-03-10 Thread Juerg Walz
uot;) - use the "-u" command line switch for updates Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.r

[Puppet-dev] [PATCH/puppet 08/20] Ignoring lines from use_gpg and catalog fetching

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 10/20] These regular expressions will not match anything. pkgutil doesn't output anything that can be matched.

2011-03-10 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 1d699f3

[Puppet-dev] [PATCH/puppet 12/20] Single package queries made more robust when dealing with pkgutil noise

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 14/20] Removing blastwave references and unused PAGER

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 18 -- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 06/20] pkgutil provider: Using the --single option which speeds up execution.

2011-03-10 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev]

2011-03-10 Thread Juerg Walz
2nd update to pkgutil changes (aka v3): Bug-fix to handle OpenCSW packages where the package "shortname" is not equal to the Solaris package name (eg. "screen" -> Solaris pkg CSWscrn). --- a/lib/puppet/provider/package/pkgutil.rb +++ b/lib/puppet/provider/package/pkgutil.rb @@ -80,8 +80,8 @@ Pup

[Puppet-dev] [PATCH/puppet 11/20] Fixing wget verbose regex

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 01/20] Fixed #4258 - Added pkgutil package provider

2011-03-10 Thread Juerg Walz
From: James Turnbull Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 117 1 files changed, 117 insertions(+), 0 deletions(-) create mode 100644 lib/puppet/provider/package/pkgutil.rb diff --git a

[Puppet-dev] [PATCH/puppet 07/20] Adding patch from Rudy Gevaert to fix not installed detection

2011-03-10 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] Re: [PATCH/puppet 19/19] (#4258) pkgutil provider: better handling of short package names

2011-03-10 Thread Juerg Walz
that bit of code to handle the case where the shortname is different to the package filename. Juerg On Mar 10, 5:11 am, Dominic Cleal wrote: > Hi Juerg, > > Thanks for the extra patches, comment below. > > On 08/03/11 06:10, Juerg Walz wrote: > > > Local-branch: tickets/mast

[Puppet-dev] [PATCH/puppet 05/20] pkgutil provider: The path to the admin file is /var/opt/csw/pkgutil/admin

2011-03-10 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 16/19] * Fix exception on parse failure of pkgutil output * Fix exception when querying latest version for unknown package

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 14/19] Removing blastwave references and unused PAGER

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 18 -- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 11/19] Fixing wget verbose regex

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev]

2011-03-07 Thread Juerg Walz
This is an update (i.e. version 2) of my proposed changes for ticket #4258. Here's the diff: --- a/lib/puppet/provider/package/pkgutil.rb +++ b/lib/puppet/provider/package/pkgutil.rb @@ -56,7 +56,7 @@ Puppet::Type.type(:package).provide :pkgutil, :parent => :sun, :source => :sun d if hash

[Puppet-dev] [PATCH/puppet 01/19] Fixed #4258 - Added pkgutil package provider

2011-03-07 Thread Juerg Walz
From: James Turnbull Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 117 1 files changed, 117 insertions(+), 0 deletions(-) create mode 100644 lib/puppet/provider/package/pkgutil.rb diff --git a

[Puppet-dev] [PATCH/puppet 15/19] Fixing indentation

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 18/19] (#4258) pkgutil provider: misc enhancements

2011-03-07 Thread Juerg Walz
uot;) - use the "-u" command line switch for updates Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.r

[Puppet-dev] [PATCH/puppet 10/19] These regular expressions will not match anything. pkgutil doesn't output anything that can be matched.

2011-03-07 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 1d699f3

[Puppet-dev] [PATCH/puppet 09/19] Add comments that explain what we are ignoring in the package and remove legacy output

2011-03-07 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 05/19] pkgutil provider: The path to the admin file is /var/opt/csw/pkgutil/admin

2011-03-07 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 06/19] pkgutil provider: Using the --single option which speeds up execution.

2011-03-07 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 13/19] Changing indentation to 2-spaces as per 2.6+ style

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 212 1 files changed, 106 insertions(+), 106 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 19/19] (#4258) pkgutil provider: better handling of short package names

2011-03-07 Thread Juerg Walz
Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 0e2056b..4a87932 100755

[Puppet-dev] [PATCH/puppet 08/19] Ignoring lines from use_gpg and catalog fetching

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 17/19] Add spec tests for pkgutil package provider

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 132 1 files changed, 132 insertions(+), 0 deletions(-) create mode 100755 spec/unit/provider/package/pkgutil_spec.rb diff --git

[Puppet-dev] [PATCH/puppet 12/19] Single package queries made more robust when dealing with pkgutil noise

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb

[Puppet-dev] [PATCH/puppet 03/19] Using --single in the pkgutil provider.

2011-03-07 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 07/19] Adding patch from Rudy Gevaert to fix not installed detection

2011-03-07 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 02/19] Adding pkgutil support.

2011-03-07 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 111 1 files changed, 111 insertions(+), 0 deletions(-) create mode 100755 lib/puppet/provider/package/pkgutil.rb diff --git

[Puppet-dev] [PATCH/puppet 04/19] pkgutil provider: Correcting a typo in a message.

2011-03-07 Thread Juerg Walz
From: Maciej Blizinski Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 13/18] Changing indentation to 2-spaces as per 2.6+ style

2011-02-22 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 212 1 files changed, 106 insertions(+), 106 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet

[Puppet-dev] [PATCH/puppet 10/18] These regular expressions will not match anything. pkgutil doesn't output anything that can be matched.

2011-02-22 Thread Juerg Walz
From: rgevaert Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index 1d699f3

[Puppet-dev] [PATCH/puppet 16/18] * Fix exception on parse failure of pkgutil output * Fix exception when querying latest version for unknown package

2011-02-22 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package

[Puppet-dev] [PATCH/puppet 18/18] (#4258) pkgutil provider: misc enhancements

2011-02-22 Thread Juerg Walz
uot;) - use the "-u" command line switch for updates Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.r

[Puppet-dev] [PATCH/puppet 11/18] Fixing wget verbose regex

2011-02-22 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev lib/puppet/provider/package/pkgutil.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/puppet/provider/package/pkgutil.rb b/lib/puppet/provider/package/pkgutil.rb index

[Puppet-dev] [PATCH/puppet 17/18] Add spec tests for pkgutil package provider

2011-02-22 Thread Juerg Walz
From: Dominic Cleal Signed-off-by: Juerg Walz --- Local-branch: tickets/master/4258-dev spec/unit/provider/package/pkgutil_spec.rb | 132 1 files changed, 132 insertions(+), 0 deletions(-) create mode 100755 spec/unit/provider/package/pkgutil_spec.rb diff --git

  1   2   >