[Aptitude-svn-commit] r3929 - in branches/aptitude-0.3/aptitude: . src

[Daniel Burrows]

Author: dburrows
Date: Fri Aug 19 20:52:55 2005
New Revision: 3929

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/broken_indicator.cc
Log:
Don't suppress updates when only the completed-state of the
backend resolver changed.

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Fri Aug 19 20:52:55 2005
@@ -1,5 +1,11 @@
 2005-08-19  Daniel Burrows  <dburrows at debian.org>
 
+	* src/broken_indicator.cc:
+
+	  Base the decision on whether to really update the broken
+	  indicator both on the actual generated solution and on whether
+	  the completed-state of the backend resolver changed.
+
 	* src/generic/aptcache.cc:
 
 	  Call selected_solution_changed() when we run out of solutions.

Modified: branches/aptitude-0.3/aptitude/src/broken_indicator.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/broken_indicator.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/broken_indicator.cc	Fri Aug 19 20:52:55 2005
@@ -54,6 +54,11 @@
 {
   aptitude_solution last_sol;
 
+  /** Records whether we had generated all solutions at the time of
+   *  the last update.
+   */
+  bool last_complete;
+
   void handle_cache_reload()
   {
     if(apt_cache_file)
@@ -108,10 +113,11 @@
       {
 	aptitude_solution sol=(*apt_cache_file)->get_current_solution();
 
-	if(sol == last_sol)
+	if(sol == last_sol && (*apt_cache_file)->solution_generation_complete() == last_complete)
 	  return;
 
-	last_sol=sol;
+	last_sol = sol;
+	last_complete = (*apt_cache_file)->solution_generation_complete();
 
 	if(sol.get_actions().empty())
 	  {