Reviewed-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com> On 12/30/2013 10:53 AM, Aline Manera wrote:
From: Aline Manera <alinefm@br.ibm.com>
This patch cleans up pep8 style issue in featuretests.py
Signed-off-by: Aline Manera <alinefm@br.ibm.com> --- Makefile.am | 1 + src/kimchi/featuretests.py | 8 +++++--- 2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/Makefile.am b/Makefile.am index 0fd92c8..a9b2b9c 100644 --- a/Makefile.am +++ b/Makefile.am @@ -43,6 +43,7 @@ PEP8_WHITELIST = \ src/kimchi/cachebust.py \ src/kimchi/config.py.in \ src/kimchi/disks.py \ + src/kimchi/featuretests.py \ src/kimchi/root.py \ src/kimchi/server.py \ plugins/__init__.py \ diff --git a/src/kimchi/featuretests.py b/src/kimchi/featuretests.py index 4cabdc9..6843a26 100644 --- a/src/kimchi/featuretests.py +++ b/src/kimchi/featuretests.py @@ -18,7 +18,7 @@ # # You should have received a copy of the GNU Lesser General Public # License along with this library; if not, write to the Free Software -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
import libvirt import os @@ -67,6 +67,7 @@ ISO_STREAM_XML = """ </devices> </domain>"""
+ class FeatureTests(object):
@staticmethod @@ -85,7 +86,8 @@ class FeatureTests(object):
@staticmethod def qemu_supports_iso_stream(): - cmd = "qemu-io http://127.0.0.1:8000/images/icon-fedora.png -c 'read -v 0 512'" + cmd = "qemu-io http://127.0.0.1:8000/images/icon-fedora.png \ + -c 'read -v 0 512'" proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, shell=True) stdout, stderr = proc.communicate() @@ -98,7 +100,7 @@ class FeatureTests(object): "'read -v 0 512'"] proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE) - thread = threading.Thread(target = proc.communicate) + thread = threading.Thread(target=proc.communicate) thread.start() thread.join(5)