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 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 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.