File 0012-zypp-less-debug-in-progress-functions.patch of Package PackageKit

Overview Repositories Revisions Requests Users Attributes Meta

File 0012-zypp-less-debug-in-progress-functions.patch of Package PackageKit (Revision 3c66d39c1cbbdf13c84f7454804da6e7)

Currently displaying revision 3c66d39c1cbbdf13c84f7454804da6e7 , Show latest

From 90864427d3384363d82813cf5b69a4f74f4330f4 Mon Sep 17 00:00:00 2001
From: Stephan Kulow <coolo@suse.de>
Date: Sun, 27 Jan 2013 08:34:46 +0100
Subject: [PATCH 12/12] zypp: less debug in progress functions

Also remove the hacks trying to work around progress bugs - we need to
fix that in rpm/zypp and don't try to hide it, creating new bugs
---
 backends/zypp/pk-backend-zypp.cpp | 23 +++++------------------
 1 file changed, 5 insertions(+), 18 deletions(-)

diff --git a/backends/zypp/pk-backend-zypp.cpp b/backends/zypp/pk-backend-zypp.cpp
index 0f32c18..f492a1c 100644
--- a/backends/zypp/pk-backend-zypp.cpp
+++ b/backends/zypp/pk-backend-zypp.cpp
@@ -205,8 +205,8 @@ public:
 		// fact that libzypp may skip over a "divisible by ten"
 		// value (i.e., 28, 29, 31, 32).
 
-		MIL << percentage << " " << _sub_percentage << std::endl;
-		if (percentage <= _sub_percentage)
+		//MIL << percentage << " " << _sub_percentage << std::endl;
+		if (percentage == _sub_percentage)
 			return;
 
 		if (!_package_id) {
@@ -236,8 +236,6 @@ protected:
 struct InstallResolvableReportReceiver : public zypp::callback::ReceiveReport<zypp::target::rpm::InstallResolvableReport>, ZyppBackendReceiver
 {
 	zypp::Resolvable::constPtr _resolvable;
-	bool preparing;
-	int last_value;
 
 	virtual void start (zypp::Resolvable::constPtr resolvable) {
 		clear_package_id ();
@@ -249,25 +247,14 @@ struct InstallResolvableReportReceiver : public zypp::callback::ReceiveReport<zy
 			pk_backend_job_package (_job, PK_INFO_ENUM_INSTALLING, _package_id, summary);
 			reset_sub_percentage ();
 		}
-		// first we prepare then we install
-		preparing = true;
-		last_value = 0;
 		g_free (summary);
 	}
 
 	virtual bool progress (int value, zypp::Resolvable::constPtr resolvable) {
 		// we need to have extra logic here as progress is reported twice
 		// and PackageKit does not like percentages going back
-		if (preparing && value < last_value) 
-			preparing = false;
-		last_value = value;
-		MIL << preparing << " " << value << " " << _package_id << std::endl;
-		int perc = 0;
-		if (preparing)
-			perc = value * 30 / 100;
-		else
-			perc = 30 + value * 70 / 100;
-		update_sub_percentage (perc);
+		//MIL << value << " " << _package_id << std::endl;
+		update_sub_percentage (value);
 		return true;
 	}
 
@@ -380,7 +367,7 @@ struct DownloadProgressReportReceiver : public zypp::callback::ReceiveReport<zyp
 
 	virtual bool progress (int value, zypp::Resolvable::constPtr resolvable)
 	{
-		MIL << resolvable << " " << value << " " << _package_id << std::endl;
+		//MIL << resolvable << " " << value << " " << _package_id << std::endl;
 		update_sub_percentage (value);
 		//pk_backend_job_set_speed (_job, static_cast<guint>(dbps_current));
 		return true;
-- 
1.8.1.1

Places

File 0012-zypp-less-debug-in-progress-functions.patch of Package PackageKit (Revision 3c66d39c1cbbdf13c84f7454804da6e7)

Places