]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
(#11413) Update dpkg query used by apt::force
authorMatthias Pigulla <mp@webfactory.de>
Thu, 15 Dec 2011 14:52:21 +0000 (15:52 +0100)
committerDan Bode <dan@puppetlabs.com>
Fri, 13 Jan 2012 19:59:24 +0000 (11:59 -0800)
This patch fixes the query used by apt::force to
determine rather or not the package is installed.

Previously, the expression was not specific enough and
could not lead to false positives in cases where a package
name is contained within another package name (puppet could be
incorrectly determined as being installed if puppet-common is
installed)

This commit resolves that by improving the query expression.

manifests/force.pp

index 0a3007d0e534bf69ff84254b5c3c7dde2c2a3d35..ece532acd0578ac740eba49c324e3b1a559cdbbe 100644 (file)
@@ -8,8 +8,8 @@ define apt::force(
 
   exec { "/usr/bin/aptitude -y -t ${release} install ${name}":
     unless => $version ? {
-      false => "/usr/bin/dpkg -l | grep ${name}",
-      default => "/usr/bin/dpkg -l | grep ${name} | grep ${version}"
+      false => "/usr/bin/dpkg -s ${name} | grep -q 'Status: install'",
+      default => "/usr/bin/dpkg -s ${name} | grep -q 'Version: ${version}'"
     }
   }