Author: lslezak Date: Fri Aug 31 12:49:46 2007 New Revision: 40642 URL: http://svn.opensuse.org/viewcvs/yast?rev=40642&view=rev Log: - check exit status and report errors when adding online repositories (#306272) - 2.15.70 Modified: trunk/packager/VERSION trunk/packager/package/yast2-packager.changes trunk/packager/src/clients/inst_productsources.ycp Modified: trunk/packager/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/VERSION?rev=40642&r1=40641&r2=40642&view=diff ============================================================================== --- trunk/packager/VERSION (original) +++ trunk/packager/VERSION Fri Aug 31 12:49:46 2007 @@ -1 +1 @@ -2.15.69 +2.15.70 Modified: trunk/packager/package/yast2-packager.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/package/yast2-packager.changes?rev=40642&r1=40641&r2=40642&view=diff ============================================================================== --- trunk/packager/package/yast2-packager.changes (original) +++ trunk/packager/package/yast2-packager.changes Fri Aug 31 12:49:46 2007 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Fri Aug 31 12:45:59 CEST 2007 - lslezak@suse.cz + +- check exit status and report errors when adding online + repositories (#306272) +- 2.15.70 + +------------------------------------------------------------------- Wed Aug 29 16:35:46 CEST 2007 - lslezak@suse.cz - fully initialize the target system when adding an update Modified: trunk/packager/src/clients/inst_productsources.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/clients/inst_productsources.ycp?rev=40642&r1=40641&r2=40642&view=diff ============================================================================== --- trunk/packager/src/clients/inst_productsources.ycp (original) +++ trunk/packager/src/clients/inst_productsources.ycp Fri Aug 31 12:49:46 2007 @@ -1021,9 +1021,29 @@ return false; } - Pkg::SourceRefreshNow (src_id); + if (!Pkg::SourceRefreshNow (src_id)) + { + Report::Error (sformat ( + // TRANSLATORS: pop-up error message + // %1 is replaced with a repository name or URL + _("Adding repository %1 failed"), + (repo_name != "" ? repo_name : url)) + + "\n" + Pkg::LastError(); + ); + return false; + } - Pkg::SourceSetEnabled (src_id, true); + if (!Pkg::SourceSetEnabled (src_id, true)) + { + Report::Error (sformat ( + // TRANSLATORS: pop-up error message + // %1 is replaced with a repository name or URL + _("Adding repository %1 failed"), + (repo_name != "" ? repo_name : url)) + + "\n" + Pkg::LastError(); + ); + return false; + } if (Stage::initial()) { AddOnProduct::Integrate (src_id); -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org