* fix: duplicate packages in order_inst

pisi/cli/__init__.py

@@ -52,7 +52,8 @@ class CLI(UI):
     def close(self):
         util.xterm_title_reset()
 
-    def output(self, msg, err = False):
+    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:
@@ -69,16 +70,13 @@ class CLI(UI):
             msgend = ''
         else:
             msgend = '\n'
-        if verbose and self.show_verbose:
+        self.output(msg + msgend, verbose=verbose)
-            self.output(msg + msgend)
-        elif not verbose:
-            self.output(msg + msgend)
 
-    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):
         if ctx.get_option('no_color'):
@@ -86,7 +84,7 @@ class CLI(UI):
         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'))
 

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():