http://bugzilla.novell.com/show_bug.cgi?id=544466 User ma@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=544466#c8 Michael Andres <ma@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|tgoettlicher@novell.com | AssignedTo|zypp-maintainers@forge.prov |sreeves@novell.com |o.novell.com | --- Comment #8 from Michael Andres <ma@novell.com> 2009-11-04 07:51:17 MST --- @Scott: I suppose this has to be fixed in PackageKit: backend/zypp-utils.cpp: ... // check if patch is needed if((*it)->candidateObj ().isRelevant() && !((*it)->candidateObj ().isSatisfied())) { 1) Minor: Broken, satisfied and !relevant are mutual exclusive. And you can directly ask the Selectable (no need to pick the candidate first): // check if patch is needed if ( (*it)->isBroken() ) { 2) That's probably also the location where to exclude locked patches: // check if patch is needed if ( (*it)->isBroken() && ! (*it)->status() == zypp::ui::S_Taboo ) { -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.