[Kimchi-devel] [PATCH 1/2] pep8 cleanup for featuretests.py
Rodrigo Trujillo
rodrigo.trujillo at linux.vnet.ibm.com
Mon Dec 30 16:35:37 UTC 2013
Reviewed-by: Rodrigo Trujillo <rodrigo.trujillo at linux.vnet.ibm.com>
On 12/30/2013 10:53 AM, Aline Manera wrote:
> From: Aline Manera <alinefm at br.ibm.com>
>
> This patch cleans up pep8 style issue in featuretests.py
>
> Signed-off-by: Aline Manera <alinefm at 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)
>
More information about the Kimchi-devel
mailing list