From 61e7bfdf1ee007dd85666f5be36754684b7236d3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Eray=20=C3=96zkural?= <eray@pardus.org.tr>
Date: Thu, 9 Mar 2006 09:07:36 +0000
Subject: [PATCH] * fix: duplicate packages in order_inst

---
 pisi/cli/__init__.py | 34 ++++++++++++++++------------------
 pisi/operations.py   |  4 ++--
 2 files changed, 18 insertions(+), 20 deletions(-)

diff --git a/pisi/cli/__init__.py b/pisi/cli/__init__.py
index ea36fffa..3190b998 100644
--- a/pisi/cli/__init__.py
+++ b/pisi/cli/__init__.py
@@ -52,15 +52,16 @@ class CLI(UI):
     def close(self):
         util.xterm_title_reset()
 
-    def output(self, msg, err = False):
-        if type(msg)==type(unicode()):
-            msg = msg.encode('utf-8')
-        if err:
-            out = sys.stderr
-        else:
-            out = sys.stdout
-        out.write(msg)
-        out.flush()
+    def output(self, msg, err = False, verbose = False):
+        if (verbose and self.show_verbose) or (not verbose):                
+            if type(msg)==type(unicode()):
+                msg = msg.encode('utf-8')
+            if err:
+                out = sys.stderr
+            else:
+                out = sys.stdout
+            out.write(msg)
+            out.flush()
 
     def info(self, msg, verbose = False, noln = False):
         # TODO: need to look at more kinds of info messages
@@ -69,24 +70,21 @@ class CLI(UI):
             msgend = ''
         else:
             msgend = '\n'
-        if verbose and self.show_verbose:
-            self.output(msg + msgend)
-        elif not verbose:
-            self.output(msg + msgend)
+        self.output(msg + msgend, verbose=verbose)
 
-    def warning(self,msg):
+    def warning(self, msg, verbose = False):
         if ctx.get_option('no_color'):
-            self.output(_('Warning: ') + msg + '\n', err=True)
+            self.output(_('Warning: ') + msg + '\n', err=True, verbose=verbose)
         else:
-            self.output(colorize(msg + '\n', 'purple'), err=True)
+            self.output(colorize(msg + '\n', 'purple'), err=True, verbose=verbose)
 
-    def error(self,msg):
+    def error(self, msg):
         if ctx.get_option('no_color'):
             self.output(_('Error: ') + msg + '\n', err=True)
         else:
             self.output(colorize(msg + '\n', 'red'), err=True)
 
-    def action(self,msg):
+    def action(self, msg, verbose = False):
         #TODO: this seems quite redundant?
         self.output(colorize(msg + '\n', 'green'))
 
diff --git a/pisi/operations.py b/pisi/operations.py
index 77872bd7..5bd2c8b3 100644
--- a/pisi/operations.py
+++ b/pisi/operations.py
@@ -646,7 +646,7 @@ def plan_emerge(A, rebuild_all):
     #pkgtosrc = {}
     B = A
 
-    install_list = []
+    install_list = set()
     
     while len(B) > 0:
         Bp = set()
@@ -660,7 +660,7 @@ def plan_emerge(A, rebuild_all):
             def process_dep(dep):
                 if not dependency.installed_satisfies_dep(dep):
                     if dependency.repo_satisfies_dep(dep):
-                        install_list.append(dep.package)
+                        install_list.add(dep.package)
                         return
                     srcdep = pkgtosrc(dep.package)
                     if not srcdep in G_f.vertices():