Script 'mail_helper' called by ro Hello packager, This is just FYI. Your package was checked in in distribution "sle12" by autobuild-member: ro. Here comes the log... ---------------------------%<------------------------------ Hi, here is the log from ci_new_pac /mounts/work_src_done/SLE12/yast2-pkg-bindings -> sle12 ## BNC# 873198 : "yast2-registraion: update existing services" (RESOLVED/FIXED) ## BNC# 873683 : "Build0270: undefined method in registration.rb in get_addon_list" (RESOLVED/FIXED) Changes: -------- --- /work/SRC/SUSE:SLE-12:GA/yast2-pkg-bindings/yast2-pkg-bindings.changes 2014-04-09 17:12:49.000000000 +0200 +++ /mounts/work_src_done/SLE12/yast2-pkg-bindings/yast2-pkg-bindings.changes 2014-04-16 11:40:26.000000000 +0200 @@ -1,0 +2,6 @@ +Tue Apr 15 14:48:45 UTC 2014 - lslezak@suse.cz + +- fixed repeated service save problem (bnc#873198, bnc#873683#c7) +- 3.1.8 + +------------------------------------------------------------------- calling whatdependson for sle12-i586 Packages directly triggered for rebuild: - yast2-pkg-bindings - at least 74 other packages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/SUSE:SLE-12:GA/yast2-pkg-bindings (Old) and /mounts/work_src_done/SLE12/yast2-pkg-bindings (BS:build ID:36427 MAIL:yast-commit@opensuse.org) (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "yast2-pkg-bindings", Maintainer is "yast-commit@opensuse.org" Old: ---- yast2-pkg-bindings-3.1.7.tar.bz2 New: ---- yast2-pkg-bindings-3.1.8.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-pkg-bindings-devel-doc.spec ++++++ --- /var/tmp/diff_new_pack.ifgumS/_old 2014-04-17 14:19:31.000000000 +0200 +++ /var/tmp/diff_new_pack.ifgumS/_new 2014-04-17 14:19:31.000000000 +0200 @@ -16,7 +16,7 @@ # Name: yast2-pkg-bindings-devel-doc -Version: 3.1.7 +Version: 3.1.8 Release: 0 License: GPL-2.0 Group: Documentation/HTML ++++++ yast2-pkg-bindings.spec ++++++ --- /var/tmp/diff_new_pack.ifgumS/_old 2014-04-17 14:19:31.000000000 +0200 +++ /var/tmp/diff_new_pack.ifgumS/_new 2014-04-17 14:19:31.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2-pkg-bindings -Version: 3.1.7 +Version: 3.1.8 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ yast2-pkg-bindings-3.1.7.tar.bz2 -> yast2-pkg-bindings-3.1.8.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings-devel-doc.spec new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings-devel-doc.spec --- old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings-devel-doc.spec 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings-devel-doc.spec 2014-04-16 11:28:29.000000000 +0200 @@ -16,7 +16,7 @@ # Name: yast2-pkg-bindings-devel-doc -Version: 3.1.7 +Version: 3.1.8 Release: 0 License: GPL-2.0 Group: Documentation/HTML diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings.changes new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings.changes --- old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings.changes 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings.changes 2014-04-16 11:28:29.000000000 +0200 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Tue Apr 15 14:48:45 UTC 2014 - lslezak@suse.cz + +- fixed repeated service save problem (bnc#873198, bnc#873683#c7) +- 3.1.8 + +------------------------------------------------------------------- Wed Apr 2 15:11:58 UTC 2014 - lslezak@suse.cz - properly initialize "autorefresh_skipped" attribute (leaving it diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings.spec new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings.spec --- old/yast2-pkg-bindings-3.1.7/package/yast2-pkg-bindings.spec 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/package/yast2-pkg-bindings.spec 2014-04-16 11:28:29.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2-pkg-bindings -Version: 3.1.7 +Version: 3.1.8 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/src/PkgService.cc new/yast2-pkg-bindings-3.1.8/src/PkgService.cc --- old/yast2-pkg-bindings-3.1.7/src/PkgService.cc 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/src/PkgService.cc 2014-04-16 11:28:29.000000000 +0200 @@ -54,3 +54,7 @@ return _old_alias; } +void PkgService::setOrigAlias(const std::string& orig_alias) +{ + _old_alias = orig_alias; +} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/src/PkgService.h new/yast2-pkg-bindings-3.1.8/src/PkgService.h --- old/yast2-pkg-bindings-3.1.7/src/PkgService.h 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/src/PkgService.h 2014-04-16 11:28:29.000000000 +0200 @@ -40,6 +40,7 @@ std::string origAlias() const; + void setOrigAlias(const std::string& orig_alias); private: diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/src/ServiceManager.cc new/yast2-pkg-bindings-3.1.8/src/ServiceManager.cc --- old/yast2-pkg-bindings-3.1.7/src/ServiceManager.cc 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/src/ServiceManager.cc 2014-04-16 11:28:29.000000000 +0200 @@ -53,7 +53,7 @@ } } -void ServiceManager::SaveServices(zypp::RepoManager &repomgr) const +void ServiceManager::SaveServices(zypp::RepoManager &repomgr) { for_ (it, _known_services.begin(), _known_services.end()) { @@ -75,9 +75,9 @@ } } -bool ServiceManager::SaveService(const std::string &alias, zypp::RepoManager &repomgr) const +bool ServiceManager::SaveService(const std::string &alias, zypp::RepoManager &repomgr) { - PkgServices::const_iterator serv_it = _known_services.find(alias); + PkgServices::iterator serv_it = _known_services.find(alias); if (serv_it == _known_services.end()) { @@ -287,7 +287,7 @@ return _known_services.size(); } -void ServiceManager::SavePkgService(const PkgService &s_known, zypp::RepoManager &repomgr) const +void ServiceManager::SavePkgService(PkgService &s_known, zypp::RepoManager &repomgr) const { const std::string alias(s_known.alias()); const zypp::ServiceInfo s_stored = repomgr.getService(alias); @@ -305,6 +305,8 @@ y2milestone("Adding new service %s", alias.c_str()); // add the service repomgr.addService(s_known); + // set the old alias to properly save it next time + s_known.setOrigAlias(alias); } else { diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-pkg-bindings-3.1.7/src/ServiceManager.h new/yast2-pkg-bindings-3.1.8/src/ServiceManager.h --- old/yast2-pkg-bindings-3.1.7/src/ServiceManager.h 2014-04-02 20:23:18.000000000 +0200 +++ new/yast2-pkg-bindings-3.1.8/src/ServiceManager.h 2014-04-16 11:28:29.000000000 +0200 @@ -42,9 +42,9 @@ void LoadServices(const zypp::RepoManager &repomgr); - void SaveServices(zypp::RepoManager &repomgr) const; + void SaveServices(zypp::RepoManager &repomgr); - bool SaveService(const std::string &alias, zypp::RepoManager &repomgr) const; + bool SaveService(const std::string &alias, zypp::RepoManager &repomgr); bool AddService(const std::string &alias, const std::string &url); @@ -70,7 +70,7 @@ private: // internal helper method - void SavePkgService(const PkgService &s_known, zypp::RepoManager &repomgr) const; + void SavePkgService(PkgService &s_known, zypp::RepoManager &repomgr) const; // current alias -> PkgService for convenient search by alias typedef std::map<std::string, PkgService> PkgServices; continue with "q"... Checked in at Thu Apr 17 14:20:12 CEST 2014 by ro Remember to have fun... -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org