Author: kmachalkova Date: Tue Oct 21 17:38:35 2008 New Revision: 52406 URL: http://svn.opensuse.org/viewcvs/yast?rev=52406&view=rev Log: AddPackageList from Storage is no longer needed - now y2-storage handles package selection in installation by itself (bnc#433001) Modified: trunk/packager/package/yast2-packager.changes trunk/packager/src/clients/wrapper_storage.ycp trunk/packager/src/modules/Packages.ycp Modified: trunk/packager/package/yast2-packager.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/package/yast2-packager.changes?rev=52406&r1=52405&r2=52406&view=diff ============================================================================== --- trunk/packager/package/yast2-packager.changes (original) +++ trunk/packager/package/yast2-packager.changes Tue Oct 21 17:38:35 2008 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Tue Oct 21 16:54:31 CEST 2008 - kmachalkova@suse.cz + +- AddPackageList from Storage is no longer needed - now y2-storage + handles package selection in installation by itself (bnc#433001) + +------------------------------------------------------------------- Tue Oct 21 16:49:06 CEST 2008 - lslezak@suse.cz - repositories.ycp - pad the priority number with spaces so the Modified: trunk/packager/src/clients/wrapper_storage.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/clients/wrapper_storage.ycp?rev=52406&r1=52405&r2=52406&view=diff ============================================================================== --- trunk/packager/src/clients/wrapper_storage.ycp (original) +++ trunk/packager/src/clients/wrapper_storage.ycp Tue Oct 21 17:38:35 2008 @@ -29,10 +29,6 @@ { ret = Storage::GetTargetMap(); } - else if (func == "AddPackageList") - { - ret = Storage::AddPackageList(); - } else if (func == "GetTargetChangeTime") { ret = Storage::GetTargetChangeTime(); Modified: trunk/packager/src/modules/Packages.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/modules/Packages.ycp?rev=52406&r1=52405&r2=52406&view=diff ============================================================================== --- trunk/packager/src/modules/Packages.ycp (original) +++ trunk/packager/src/modules/Packages.ycp Tue Oct 21 17:38:35 2008 @@ -860,17 +860,9 @@ install_list = (list<string>) union (install_list, modePackages ()); - // add storage packages if yast2-storage installed - list<string> storage_packages = (list<string>)WFM::call("wrapper_storage", ["AddPackageList"]); - if (storage_packages == nil) - { - y2warning("storage_packages is nil, module Storage:: is probably missing"); - } - else - { - y2milestone ("Storage packages: %1", storage_packages); - install_list = (list<string>) union (install_list, storage_packages); - } + // No longer needed - partitions_proposal uses PackagesProposal now + // to gather the list of pkgs needed by y2-storage (#433001) + //list<string> storage_packages = (list<string>)WFM::call("wrapper_storage", ["AddPackageList"]); if (size (additional_packages) > 0) { y2warning ("Additional packages are still in use, please, change it to use PackagesProposal API"); -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org