On Thursday 26 February 2009 00:21:38 Duncan Mac-Vicar P. wrote:
Duncan Mac-Vicar P. wrote:
7 days ago Michael Andres remove unused 'DueToObsolete' from ResStatus
Looking at Michael's commit, would it just be ok to remove the .isToBeUninstalledDueToObsolete () case?
if (it->status ().isToBeUninstalled ()) { status = PK_INFO_ENUM_REMOVING; hit = TRUE; }else if (it->status ().isToBeInstalled ()) { status = PK_INFO_ENUM_INSTALLING; hit = TRUE; }else if (it->status ().isToBeUninstalledDueToUpgrade ()) { status = PK_INFO_ENUM_UPDATING; hit = TRUE; }else if (it->status ().isToBeUninstalledDueToObsolete ()) { status = PK_INFO_ENUM_OBSOLETING; hit = TRUE; }
Yes, but this bit is never set. If the info is really needed we have to provide some method that computes it? In that case we should revert the change, untill we know how. (Duncan: We need to bring PkgKit to Hudson AND ALSO libzypp-testsuite! ) -- cu, Michael Andres +------------------------------------------------------------------+ Key fingerprint = 2DFA 5D73 18B1 E7EF A862 27AC 3FB8 9E3A 27C6 B0E4 +------------------------------------------------------------------+ Michael Andres YaST Development ma@novell.com SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg) Maxfeldstrasse 5, D-90409 Nuernberg, Germany, ++49 (0)911 - 740 53-0 +------------------------------------------------------------------+ -- To unsubscribe, e-mail: zypp-devel+unsubscribe@opensuse.org For additional commands, e-mail: zypp-devel+help@opensuse.org