Author: lslezak Date: Thu Sep 22 15:40:34 2011 New Revision: 65932 URL: http://svn.opensuse.org/viewcvs/yast?rev=65932&view=rev Log: - use signed type for RepoId, fixes returning 2^32 - 1 instead of -1 on i586 (bnc#718514) - 2.17.52 Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/VERSION branches/SuSE-Code-11-SP2-Branch/pkg-bindings/package/yast2-pkg-bindings.changes branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Locks.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Package.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/PkgFunctions.h branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Service.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Load.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Misc.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Resolvables.cc branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Set.cc Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/VERSION?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/VERSION (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/VERSION Thu Sep 22 15:40:34 2011 @@ -1 +1 @@ -2.17.51 +2.17.52 Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/package/yast2-pkg-bindings.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/package/yast2-pkg-bindings.changes?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/package/yast2-pkg-bindings.changes (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/package/yast2-pkg-bindings.changes Thu Sep 22 15:40:34 2011 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Thu Sep 22 13:26:20 UTC 2011 - lslezak@suse.cz + +- use signed type for RepoId, fixes returning 2^32 - 1 instead of + -1 on i586 (bnc#718514) +- 2.17.52 + +------------------------------------------------------------------- Wed Aug 10 13:17:36 UTC 2011 - lslezak@suse.cz - Pkg::SourceStartManager() - always display progress, do not Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Locks.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Locks.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Locks.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Locks.cc Thu Sep 22 15:40:34 2011 @@ -172,7 +172,7 @@ } else { - y2error("Repository %zd not found", repo_id); + y2error("Repository %lld not found", repo_id); return YCPBoolean(false); } } @@ -485,7 +485,7 @@ if (locks.size() < idx + 1) { - y2error("Invalid lock index %d, %zd locks defined", idx, locks.size()); + y2error("Invalid lock index %d, %ld locks defined", idx, locks.size()); return YCPBoolean(false); } Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Package.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Package.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Package.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Package.cc Thu Sep 22 15:40:34 2011 @@ -221,7 +221,7 @@ if (repo_name.empty()) { - y2warning("Name of repository '%zd' is empty, using URL", index); + y2warning("Name of repository '%lld' is empty, using URL", index); // use URL as the product name std::string name; Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/PkgFunctions.h URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/PkgFunctions.h?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/PkgFunctions.h (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/PkgFunctions.h Thu Sep 22 15:40:34 2011 @@ -102,7 +102,7 @@ public: // ID type - typedef RepoCont::size_type RepoId; + typedef long long RepoId; private: // source related Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Service.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Service.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Service.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Service.cc Thu Sep 22 15:40:34 2011 @@ -111,7 +111,7 @@ if (repo->repoInfo().service() == service_alias) { std::string repo_alias = repo->repoInfo().alias(); - y2milestone("Removing repository %zd (%s) belonging to service %s", + y2milestone("Removing repository %lld (%s) belonging to service %s", index, repo_alias.c_str(), service_alias.c_str()); repo->setDeleted(); } @@ -294,7 +294,7 @@ { std::string repo_alias = repo->repoInfo().alias(); - y2milestone("%s repository %zd (%s) belonging to service %s", + y2milestone("%s repository %lld (%s) belonging to service %s", enabled ? "Enabling" : "Disabling", index, repo_alias.c_str(), old_alias_str.c_str()); repo->repoInfo().setEnabled(enabled); Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Load.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Load.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Load.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Load.cc Thu Sep 22 15:40:34 2011 @@ -57,7 +57,7 @@ if (repos.size() > 0) { - y2warning("Number of registered repositories: %zd, skipping repository load!", repos.size()); + y2warning("Number of registered repositories: %ld, skipping repository load!", repos.size()); return YCPBoolean(success); } @@ -67,7 +67,7 @@ if (!service_manager.empty()) { - y2warning("Number of known services: %zd, skipping service load!", service_manager.size()); + y2warning("Number of known services: %ld, skipping service load!", service_manager.size()); } else { Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Misc.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Misc.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Misc.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Misc.cc Thu Sep 22 15:40:34 2011 @@ -43,7 +43,7 @@ { try { - if (id < 0 || id >= repos.size()) + if (id < 0 || id >= (long long)repos.size()) { // not found throw(std::exception()); @@ -57,7 +57,7 @@ if (repos[id]->isDeleted()) { - y2error("Source %zd has been deleted, the ID is not valid", id); + y2error("Source %lld has been deleted, the ID is not valid", id); return YRepo_Ptr(); } @@ -65,7 +65,7 @@ } catch (...) { - y2error("Cannot find source with ID: %zd", id); + y2error("Cannot find source with ID: %lld", id); // TODO: improve the error message _last_error.setLastError(_("Cannot find source")); } @@ -157,7 +157,7 @@ while(aliasExists(ret, reps)) { - y2milestone("Alias %s already found: %zd", ret.c_str(), logFindAlias(ret)); + y2milestone("Alias %s already found: %lld", ret.c_str(), logFindAlias(ret)); // the alias already exists - add a counter std::ostringstream ostr; Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Resolvables.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Resolvables.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Resolvables.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Resolvables.cc Thu Sep 22 15:40:34 2011 @@ -116,7 +116,7 @@ repomanager.loadFromCache(repoinfo); repo->setLoaded(); - //y2milestone("Loaded %zd resolvables", store.size()); + //y2milestone("Loaded %lld resolvables", store.size()); } catch(const zypp::repo::RepoNotCachedException &excpt ) { Modified: branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Set.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Set.cc?rev=65932&r1=65931&r2=65932&view=diff ============================================================================== --- branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Set.cc (original) +++ branches/SuSE-Code-11-SP2-Branch/pkg-bindings/src/Source_Set.cc Thu Sep 22 15:40:34 2011 @@ -180,7 +180,7 @@ if (repo->repoInfo().enabled() != enable) { - y2warning("Pkg::SourceEditSet() does not refresh the pool (src: %zd, state: %s)", id, enable ? "disabled -> enabled" : "enabled -> disabled"); + y2warning("Pkg::SourceEditSet() does not refresh the pool (src: %lld, state: %s)", id, enable ? "disabled -> enabled" : "enabled -> disabled"); } y2debug("set enabled: %d", enable); -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org