[tuned] another powertop-2.0 compatibility fix

Jaroslav Škarvada jskarvad at fedoraproject.org
Tue Jun 12 15:24:41 UTC 2012


commit 9f67a15b77299b16ebb3abfd9b6236b588c8fd44
Author: Jaroslav Škarvada <jskarvad at redhat.com>
Date:   Tue Jun 12 17:24:37 2012 +0200

    another powertop-2.0 compatibility fix
    
      Resolves: rhbz#830415

 tuned-powertop2.patch |   46 +++++++++++++++++++++++++++++++++++-----------
 tuned.spec            |    6 +++++-
 2 files changed, 40 insertions(+), 12 deletions(-)
---
diff --git a/tuned-powertop2.patch b/tuned-powertop2.patch
index 7d3daf4..3d849a7 100644
--- a/tuned-powertop2.patch
+++ b/tuned-powertop2.patch
@@ -1,8 +1,23 @@
-diff --git a/experiments/powertop2tuned.py b/experiments/powertop2tuned.py
-index 02a5b23..70c6f0b 100755
---- a/experiments/powertop2tuned.py
-+++ b/experiments/powertop2tuned.py
-@@ -74,6 +74,8 @@ class PowertopHTMLParser(HTMLParser):
+diff -up tuned-2.0.1/experiments/powertop2tuned.py.orig tuned-2.0.1/experiments/powertop2tuned.py
+--- tuned-2.0.1/experiments/powertop2tuned.py.orig	2012-04-03 19:25:37.000000000 +0200
++++ tuned-2.0.1/experiments/powertop2tuned.py	2012-06-12 17:19:30.548102052 +0200
+@@ -44,12 +44,14 @@ stop() {
+ process $@
+ """
+ 
+-TUNED_CONF_PROLOG = "# Automatically generated by powertop2tuned tool"
++TUNED_CONF_PROLOG = "# Automatically generated by powertop2tuned tool\n\n"
+ TUNED_CONF_INCLUDE = """[main]
+ %s
++
+ """
+ TUNED_CONF_EPILOG="""[powertop_script]
+ type=script
++replace=1
+ script=script.sh
+ """
+ 
+@@ -74,6 +76,8 @@ class PowertopHTMLParser(HTMLParser):
  
  	def handle_starttag(self, tag, attrs):
  		self.lastStartTag = tag
@@ -11,7 +26,7 @@ index 02a5b23..70c6f0b 100755
  		if self.inProperTable and tag == "td":
  			self.tdCounter += 1
  
-@@ -92,8 +94,6 @@ class PowertopHTMLParser(HTMLParser):
+@@ -92,8 +96,6 @@ class PowertopHTMLParser(HTMLParser):
  
  	def handle_data(self, data):
  		prefix = self.prefix
@@ -20,7 +35,7 @@ index 02a5b23..70c6f0b 100755
  		if self.inProperTable and self.tdCounter == 1:
  			self.lastDesc = data
  			if self.lastDesc.lower().find("autosuspend") != -1 and (self.lastDesc.lower().find("keyboard") != -1 or self.lastDesc.lower().find("mouse") != -1):
-@@ -103,10 +103,10 @@ class PowertopHTMLParser(HTMLParser):
+@@ -103,10 +105,10 @@ class PowertopHTMLParser(HTMLParser):
  			self.tdCounter = 0
  			if not self.inScript:
  				self.data += "\t# " + self.lastDesc + "\n"
@@ -33,7 +48,7 @@ index 02a5b23..70c6f0b 100755
  
  class PowertopProfile:
  	BAD_PRIVS = 100
-@@ -132,16 +132,18 @@ class PowertopProfile:
+@@ -132,16 +134,18 @@ class PowertopProfile:
  		return True
  
  	def generateHTML(self):
@@ -49,15 +64,24 @@ index 02a5b23..70c6f0b 100755
 +		output = proc.communicate()[0]
 +		if proc.returncode != 0:
 +				return ret
-+
+ 
+-		return name;
 +		prefix = "PowerTOP outputing using base filename "
 +		if output.find(prefix) == -1:
 +				return -1
- 
--		return name;
++
 +		name = output[output.find(prefix)+len(prefix):-1]
 +		#print "Parsed filename=", [name]
 +		return name
  
  	def parseHTML(self, enable_tunings):
  		f = open(self.name)
+@@ -163,7 +167,7 @@ class PowertopProfile:
+ 		print "Generating Tuned config file", os.path.join(self.output, "tuned.conf")
+ 		f = open(os.path.join(self.output, "tuned.conf"), "w")
+ 		f.write(TUNED_CONF_PROLOG)
+-		if (new_profile):		
++		if (not new_profile):
+ 			f.write(TUNED_CONF_INCLUDE % ("include=" + profile))
+ 		f.write(TUNED_CONF_EPILOG)
+ 		f.close()
diff --git a/tuned.spec b/tuned.spec
index 8be03c0..db862b1 100644
--- a/tuned.spec
+++ b/tuned.spec
@@ -1,7 +1,7 @@
 Summary: A dynamic adaptive system tuning daemon
 Name: tuned
 Version: 2.0.1
-Release: 2%{?dist}
+Release: 3%{?dist}
 License: GPLv2
 # The source for this package was pulled from upstream git.  Use the
 # following commands to get the corresponding tarball:
@@ -135,6 +135,10 @@ fi
 %{_prefix}/lib/tuned/spindown-disk
 
 %changelog
+* Tue Jun 12 2012 Jaroslav Škarvada <jskarvad at redhat.com> - 2.0.1-3
+- another powertop-2.0 compatibility fix
+  Resolves: rhbz#830415
+
 * Tue Jun 12 2012 Jan Kaluza <jkaluza at redhat.com> - 2.0.1-2
 - fixed powertop2tuned compatibility with powertop-2.0
 


More information about the scm-commits mailing list