Author: lslezak Date: Mon Sep 17 13:15:07 2007 New Revision: 40936 URL: http://svn.opensuse.org/viewcvs/yast?rev=40936&view=rev Log: - inst_rpmcopy.ycp - check for nil result of Pkg::PkgCommit(), display an error message (#293428) - 2.15.80 Modified: trunk/packager/VERSION trunk/packager/package/yast2-packager.changes trunk/packager/src/clients/inst_rpmcopy.ycp Modified: trunk/packager/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/VERSION?rev=40936&r1=40935&r2=40936&view=diff ============================================================================== --- trunk/packager/VERSION (original) +++ trunk/packager/VERSION Mon Sep 17 13:15:07 2007 @@ -1 +1 @@ -2.15.79 +2.15.80 Modified: trunk/packager/package/yast2-packager.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/package/yast2-packager.changes?rev=40936&r1=40935&r2=40936&view=diff ============================================================================== --- trunk/packager/package/yast2-packager.changes (original) +++ trunk/packager/package/yast2-packager.changes Mon Sep 17 13:15:07 2007 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Mon Sep 17 12:53:25 CEST 2007 - lslezak@suse.cz + +- inst_rpmcopy.ycp - check for nil result of Pkg::PkgCommit(), + display an error message (#293428) +- 2.15.80 + +------------------------------------------------------------------- Fri Sep 14 09:12:23 CEST 2007 - locilka@suse.cz - Fixed behavior when the same licence had to be accepted several Modified: trunk/packager/src/clients/inst_rpmcopy.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/clients/inst_rpmcopy.ycp?rev=40936&r1=40935&r2=40936&view=diff ============================================================================== --- trunk/packager/src/clients/inst_rpmcopy.ycp (original) +++ trunk/packager/src/clients/inst_rpmcopy.ycp Mon Sep 17 13:15:07 2007 @@ -724,8 +724,13 @@ // returns [ int successful, list failed, list remaining ] list commit_result = PackageInstallation::CommitPackages( current_cd_no, 0 ); - if (size (commit_result) == 0) + + if (commit_result == nil || size (commit_result) == 0) { + y2error("Commit failed"); + // TODO: use better message, this is a reused string due to text-freeze + Report::Error(_("Installation of Some Packages Failed")); + return `abort; } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org