Author: lslezak Date: Thu Aug 30 20:04:36 2007 New Revision: 40635 URL: http://svn.opensuse.org/viewcvs/yast?rev=40635&view=rev Log: - report errors when adding a new repository (#306272) Modified: trunk/pkg-bindings/package/yast2-pkg-bindings.changes trunk/pkg-bindings/src/Source.cc Modified: trunk/pkg-bindings/package/yast2-pkg-bindings.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/package/yast2-pkg-bindings.changes?rev=40635&r1=40634&r2=40635&view=diff ============================================================================== --- trunk/pkg-bindings/package/yast2-pkg-bindings.changes (original) +++ trunk/pkg-bindings/package/yast2-pkg-bindings.changes Thu Aug 30 20:04:36 2007 @@ -5,6 +5,7 @@ (#306458) - preliminary fix for source redirection (compile fix only) (#294481) +- report errors when adding a new repository (#306272) ------------------------------------------------------------------- Wed Aug 29 09:36:00 CEST 2007 - lslezak@suse.cz Modified: trunk/pkg-bindings/src/Source.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/src/Source.cc?rev=40635&r1=40634&r2=40635&view=diff ============================================================================== --- trunk/pkg-bindings/src/Source.cc (original) +++ trunk/pkg-bindings/src/Source.cc Thu Aug 30 20:04:36 2007 @@ -2130,12 +2130,12 @@ // load resolvables only when they are missing if (!AnyResolvableFrom(repo->repoInfo().alias())) { - LoadResolvablesFrom(repo->repoInfo()); + success = LoadResolvablesFrom(repo->repoInfo()); } } else { - // the source has been disables, remove resolvables from the pool + // the source has been disabled, remove resolvables from the pool RemoveResolvablesFrom(repo->repoInfo().alias()); } @@ -2216,6 +2216,7 @@ catch ( const zypp::Exception & expt ) { y2error ("Error while refreshing the source: %s", expt.asString().c_str()); + _last_error.setLastError(repo->repoInfo().alias() + ": " + ExceptionAsString(expt)); return YCPBoolean(false); } @@ -2589,7 +2590,9 @@ } catch (const zypp::Exception& excpt) { - y2internal("Caught unknown error"); + std::string alias = repoinfo.alias(); + y2internal("Error: Loading resolvables failed: %s", ExceptionAsString(excpt).c_str()); + _last_error.setLastError("'" + alias + "': " + ExceptionAsString(excpt)); success = false; } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org