changeset 5432:fd1c309023d4

buildrunner: oops, do not reverse build the deps, only uninstall :-)
author Jan Nieuwenhuizen <janneke@gnu.org>
date Mon, 27 Jul 2009 12:33:33 +0200
parents 92e701bdbf4d
children 9d2d54820803
files gub/buildrunner.py
diffstat 1 files changed, 6 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/gub/buildrunner.py	Mon Jul 27 12:31:02 2009 +0200
+++ b/gub/buildrunner.py	Mon Jul 27 12:33:33 2009 +0200
@@ -284,10 +284,10 @@
                 self.manager (pkg.platform ()).uninstall_package (pkg.name ())
 
     def outdated_names (self, deps):
-        return list (reversed ([name for name in deps
-                                    if (self.is_outdated_spec (name)
-                                        and not (self.options.lax_checksums
-                                                 and self.spec_is_installable (self.specs[name])))]))
+        return [name for name in deps
+                if (self.is_outdated_spec (name)
+                    and not (self.options.lax_checksums
+                             and self.spec_is_installable (self.specs[name])))]
 
     def uninstall_specs (self, lst):
         for name in lst:
@@ -303,7 +303,8 @@
         if not fail_str:
             fail_str = '<nothing to be done>.'
         logging.default_logger.write_log ('must rebuild[%(platform)s]: %(fail_str)s\n' % locals (), 'stage')
-        outdated_installed = [x for x in outdated if self.is_installed_spec (x)]
+        outdated_installed = [x for x in list (reversed (outdated))
+                              if self.is_installed_spec (x)]
         if outdated_installed:
             platform = self.settings.platform
             outdated_str = (' '.join (outdated_installed)