diff options
author | luke <luke@980ebf18-57e1-0310-9a29-db15c13687c0> | 2006-11-09 22:02:49 +0000 |
---|---|---|
committer | luke <luke@980ebf18-57e1-0310-9a29-db15c13687c0> | 2006-11-09 22:02:49 +0000 |
commit | cbb4578f054afcbad5b4d7a49601d2207da18d0b (patch) | |
tree | 5ce97d7315e73c433d6a903fd7ea0b80129de590 | |
parent | 87fd075e6ac3876909c9bc2c6eabbed3fd9f1f5d (diff) | |
download | puppet-cbb4578f054afcbad5b4d7a49601d2207da18d0b.tar.gz puppet-cbb4578f054afcbad5b4d7a49601d2207da18d0b.tar.xz puppet-cbb4578f054afcbad5b4d7a49601d2207da18d0b.zip |
fixing #327; debian packages now correctly register their "latest" status
git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1848 980ebf18-57e1-0310-9a29-db15c13687c0
-rwxr-xr-x | lib/puppet/provider/package/apt.rb | 4 | ||||
-rw-r--r-- | lib/puppet/type/package.rb | 2 | ||||
-rwxr-xr-x | test/types/package.rb | 2 |
3 files changed, 5 insertions, 3 deletions
diff --git a/lib/puppet/provider/package/apt.rb b/lib/puppet/provider/package/apt.rb index d4edbb27b..a53490d04 100755 --- a/lib/puppet/provider/package/apt.rb +++ b/lib/puppet/provider/package/apt.rb @@ -92,14 +92,14 @@ Puppet::Type.type(:package).provide :apt, :parent => :dpkg do versioncmp(a,b) } - unless version + unless version.length > 0 self.debug "No latest version" if Puppet[:debug] print output end end - return version + return version.shift else self.err "Could not match string" end diff --git a/lib/puppet/type/package.rb b/lib/puppet/type/package.rb index f88981ffd..0f5ec26c0 100644 --- a/lib/puppet/type/package.rb +++ b/lib/puppet/type/package.rb @@ -137,7 +137,7 @@ module Puppet return true else self.debug "@is is %s, latest %s is %s" % - [@is, @parent.name, @latest] + [@is.inspect, @parent.name, @latest.inspect] end when :absent if @is == :absent diff --git a/test/types/package.rb b/test/types/package.rb index 611d73044..8a542d62c 100755 --- a/test/types/package.rb +++ b/test/types/package.rb @@ -243,6 +243,8 @@ class TestPackages < Test::Unit::TestCase pkg.retrieve assert(pkg.insync?, "After install, package is not insync") + assert_events([], comp, "package") + assert_nothing_raised { pkg[:ensure] = "absent" } |