Am 28.11.2013 um 15:34 hat Fam Zheng geschrieben: > On Thu, Nov 28, 2013 at 7:56 PM, Kevin Wolf <kw...@redhat.com> wrote: > > Am 27.11.2013 um 03:01 hat Fam Zheng geschrieben: > >> On 2013年11月20日 10:25, Fam Zheng wrote: > >> >The second patch is too large as a mail, you can pull from: > >> > > >> > https://github.com/famz/qemu.git 035-filter-qemu-io > >> > > >> >v2: Rebase to current master. > >> > [01] Add Reviewed-by for Eric. > >> > [02] Update comment to describe the command for mechanical > >> > substitution. (Eric) > >> > >> Kevin, would you like to take this one? > > > > I would, but this way it simply doesn't work. > > > > The '035-filter-qemu-io' branch in your repo is something, but certainly > > not this series. It may be some early RFC where you only fixed one case. > > I then found the 'iotests-filter-qemu-io' (which is already not what you > > said) branch and I almost would have applied it, but I noticed it's not > > exactly the same as this series; I suppose it's version 1. > > > > Eric reviewed _something_, but the two branches I found aren't what you > > sent, and he didn't specify the commit ID he reviewed, so I can't put > > his Reviewed-by tag anywhere, because I simply don't know what he looked > > at. > > > > Fam, can you push the _current_ version somewhere and tell me the > > correct branch name and commit ID? Eric, can you then tell me if this > > newly pushed code matches what you reviewed? > > Sorry, I've pushed again to: > > https://github.com/famz/qemu.git for-kevin > > Please fetch again.
Thanks, cherry-picked the patches from that branch (commit ID of the top is 0974950e05137c7751ec1d3c5d1b0fa5387eead9). I didn't add Eric's Reviewed-by yet, but I'll do so when he confirms that this is what he checked. Kevin
