qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@redhat.com>
To: Jeff Cody <jcody@redhat.com>
Cc: qemu-devel@nongnu.org, stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH v3 0/3] qemu-iotests with sample images, vhdx test, cleanup
Date: Fri, 27 Sep 2013 11:15:20 +0200	[thread overview]
Message-ID: <20130927091520.GB2440@dhcp-200-207.str.redhat.com> (raw)
In-Reply-To: <cover.1380110346.git.jcody@redhat.com>

Am 25.09.2013 um 14:12 hat Jeff Cody geschrieben:
> Changes in v3:
>     One line in patch 3, for test 043: "$TEST_IMG".[123].base instead of
>     "$TEST_IMG.[123].base".  Thanks Stefan.
> 
>     Added Stefan's "Reviewed-by" to the patches.
> 
> Changes in v2:
> 
> Patch 1: MANIFEST file dropped.  Instead of bunzip2, use bzcat.  Check error
>          return of bzcat.  Don't attempt to remove $SAMPLE_IMG_FILE if the
>          variable is blank.
> 
> Patch 2: MANIFEST file dropped.
> 
> Patch 3: New patch.  This adds some cleanup in the format of quotations around
>          filenames and paths in the io tests.
> 
> This provides:
> 
> Patch 1/3:  A basic framework for using sample image files.  This is intended
>             to be sample images created with the image format native tool; e.g.
>             a VHDX image created with Hyper-V.
> 
> Patch 2/3:  VHDX read test on a sample image created with Hyper-V.
> 
> Patch 3/3:  Quote around usage of $TEST_IMG and $TEST_DIR, so that pathnames
>             and filenames with spaces can safely be used.

Thanks, applied patches 1 and 3.

Kevin

      parent reply	other threads:[~2013-09-27  9:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-25 12:12 [Qemu-devel] [PATCH v3 0/3] qemu-iotests with sample images, vhdx test, cleanup Jeff Cody
2013-09-25 12:12 ` [Qemu-devel] [PATCH v3 1/3] block: qemu-iotests - add basic ability to use binary sample images Jeff Cody
2013-09-25 12:12 ` [Qemu-devel] [PATCH v3 2/3] block: qemu-iotests for vhdx, read sample dynamic image Jeff Cody
2013-09-27  2:31   ` Fam Zheng
2013-09-27  9:14   ` Kevin Wolf
2013-09-27 11:35     ` Jeff Cody
2013-09-25 12:12 ` [Qemu-devel] [PATCH v3 3/3] block: qemu-iotests - quote $TEST_IMG* and $TEST_DIR usage Jeff Cody
2013-09-27  9:15 ` Kevin Wolf [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130927091520.GB2440@dhcp-200-207.str.redhat.com \
    --to=kwolf@redhat.com \
    --cc=jcody@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).