Author: lslezak Date: Tue Feb 24 13:32:34 2009 New Revision: 55712 URL: http://svn.opensuse.org/viewcvs/yast?rev=55712&view=rev Log: - merged rev. 55643 from SLE11 branch: Removed fragile test for service being modified. Leave it up to libzypp to decide wich actions must be performed when saving a service. (bnc#476418) Modified: trunk/pkg-bindings/package/yast2-pkg-bindings.changes trunk/pkg-bindings/src/ServiceManager.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=55712&r1=55711&r2=55712&view=diff ============================================================================== --- trunk/pkg-bindings/package/yast2-pkg-bindings.changes (original) +++ trunk/pkg-bindings/package/yast2-pkg-bindings.changes Tue Feb 24 13:32:34 2009 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Fri Feb 20 13:16:39 CET 2009 - ma@suse.de + +- Removed fragile test for service being modified. Leave it up to libzypp + to decide wich actions must be performed when saving a service. (bnc#476418) + +------------------------------------------------------------------- Wed Jan 28 14:11:58 CET 2009 - lslezak@suse.cz - Fixed Pkg::SetSolverFlags() function - properly set Modified: trunk/pkg-bindings/src/ServiceManager.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/src/ServiceManager.cc?rev=55712&r1=55711&r2=55712&view=diff ============================================================================== --- trunk/pkg-bindings/src/ServiceManager.cc (original) +++ trunk/pkg-bindings/src/ServiceManager.cc Tue Feb 24 13:32:34 2009 @@ -285,34 +285,21 @@ DBG << "Known Service: " << s_known << std::endl; DBG << "Stored Service: " << s_stored << std::endl; - bool modified = (s_stored.url() != s_known.url() - || s_stored.name() != s_known.name() - || s_stored.enabled() != s_known.enabled() - || s_stored.autorefresh() != s_known.autorefresh() - ); + std::string orig_alias(s_known.origAlias()); - if (modified) - { - std::string orig_alias(s_known.origAlias()); - - y2internal("orig_alias: %s", orig_alias.c_str()); + y2internal("orig_alias: %s", orig_alias.c_str()); - // the old alias is empty for new services - if (orig_alias.empty()) - { - y2milestone("Adding new service %s", alias.c_str()); - // add the service - repomgr.addService(s_known); - } - else - { - y2milestone("Saving service %s", alias.c_str()); - // use the old alias - repomgr.modifyService(orig_alias, s_known); - } + // the old alias is empty for new services + if (orig_alias.empty()) + { + y2milestone("Adding new service %s", alias.c_str()); + // add the service + repomgr.addService(s_known); } else { - y2milestone("Service %s has not been modified, not saving", alias.c_str()); + y2milestone("Saving service %s", alias.c_str()); + // use the old alias + repomgr.modifyService(orig_alias, s_known); } } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org