From 2a4716d8726e383291637e8f5ae2dff49248ac9b Mon Sep 17 00:00:00 2001
From: Asaf <asaf@home.localdomain>
Date: Sun, 17 Oct 2010 07:44:56 +0200
Subject: changed package type to 'pacman', if request package version is
 'auto' and its installed dont re-install

---
 src/lib/Client/Tools/Pacman.py     | 21 +++++++++++++--------
 src/lib/Server/Plugins/Packages.py |  2 +-
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/src/lib/Client/Tools/Pacman.py b/src/lib/Client/Tools/Pacman.py
index f155adfd4..a9edc4d65 100644
--- a/src/lib/Client/Tools/Pacman.py
+++ b/src/lib/Client/Tools/Pacman.py
@@ -8,10 +8,10 @@ class Pacman(Bcfg2.Client.Tools.PkgTool):
     '''Archlinux package support'''
     name = 'Pacman'
     __execs__ = ["/usr/bin/pacman"]
-    __handles__ = [('Package', 'txz')]
+    __handles__ = [('Package', 'pacman')]
     __req__ = {'Package': ['name', 'version']}
     pkgtype = 'pacman'
-    pkgtool = ("/usr/bin/pacman --needed --noconfirm --noprogressbar -S %s")
+    pkgtool = ("/usr/bin/pacman --needed --noconfirm --noprogressbar")
 
     def __init__(self, logger, setup, config):
         Bcfg2.Client.Tools.PkgTool.__init__(self, logger, setup, config)
@@ -30,13 +30,18 @@ class Pacman(Bcfg2.Client.Tools.PkgTool):
 
     def VerifyPackage(self, entry, modlist):
         '''Verify Package status for entry'''
+
+        print "VerifyPackage : " + entry.get('name')+ " : " + entry.get('version')
+
         if not 'version' in entry.attrib:
             self.logger.info("Cannot verify unversioned package %s" %
                (entry.attrib['name']))
             return False
 
         if entry.attrib['name'] in self.installed:
-            if self.installed[entry.attrib['name']] == entry.attrib['version']:
+            if entry.attrib['version'] == 'auto':
+                return True
+            elif self.installed[entry.attrib['name']] == entry.attrib['version']:
                 #if not self.setup['quick'] and \
                 #                entry.get('verify', 'true') == 'true':
                 #FIXME: We should be able to check this once
@@ -55,8 +60,8 @@ class Pacman(Bcfg2.Client.Tools.PkgTool):
         '''Remove extra packages'''
         names = [pkg.get('name') for pkg in packages]
         self.logger.info("Removing packages: %s" % " ".join(names))
-        self.cmd.run("/usr/bin/pacman --noconfirm --noprogressbar -R %s" % \
-                     " ".join(names))
+        self.cmd.run("%s --noconfirm --noprogressbar -R %s" % \
+                     (self.pkgtool, " ".join(names)))
         self.RefreshPackages()
         self.extra = self.FindExtraPackages()
 
@@ -72,9 +77,9 @@ class Pacman(Bcfg2.Client.Tools.PkgTool):
 
         try:
             self.logger.debug('Running Pacman.Install()')
-            print "AAAAA"
-            s = self.cmd.run("%s install" % self.pkgtool)
-            print "BBB: " + str(s)
+            print "running : %s -S %s" % (self.pkgtool, pkgline)
+            s = self.cmd.run("%s -S %s" % (self.pkgtool, pkgline))
+            print "pacman : " + str(s)
         except Exception as ex:
             print "error in cmd.run ", ex
 
diff --git a/src/lib/Server/Plugins/Packages.py b/src/lib/Server/Plugins/Packages.py
index b24cd48da..194330723 100644
--- a/src/lib/Server/Plugins/Packages.py
+++ b/src/lib/Server/Plugins/Packages.py
@@ -470,7 +470,7 @@ class APTSource(Source):
 
 class PACSource(Source):
     basegroups = ['arch', 'parabola']
-    ptype = 'txz'
+    ptype = 'pacman'
 
     def __init__(self, basepath, url, version, arches, components, groups,
                  rawurl, blacklist, whitelist, recommended):
-- 
cgit v1.2.3-1-g7c22