qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v12 0/4] qapi: Allow modularization of QAPI sche


From: Luiz Capitulino
Subject: Re: [Qemu-devel] [PATCH v12 0/4] qapi: Allow modularization of QAPI schema files
Date: Wed, 7 May 2014 13:23:20 -0400

On Wed, 07 May 2014 19:20:14 +0200
Lluís Vilanova <address@hidden> wrote:

> Luiz Capitulino writes:
> 
> > On Wed, 07 May 2014 19:07:56 +0200
> > Lluís Vilanova <address@hidden> wrote:
> 
> >> Luiz Capitulino writes:
> >> 
> >> > On Mon, 5 May 2014 14:21:26 -0400
> >> > Luiz Capitulino <address@hidden> wrote:
> >> 
> >> >> On Fri,  2 May 2014 15:52:19 +0200
> >> >> Lluís Vilanova <address@hidden> wrote:
> >> >> 
> >> >> > Adds an include primitive to the syntax of QAPI schema files, 
> >> >> > allowing these to
> >> >> > be modularized into multiple per-topic files in the future.
> >> >> > 
> >> >> > Signed-off-by: Lluís Vilanova <address@hidden>
> >> >> > Reviewed-by: Eric Blake <address@hidden>
> >> >> > Reviewed-by: Markus Armbruster <address@hidden>
> >> >> 
> >> >> Applied to the qmp branch, thanks.
> >> 
> >> > make check fails for me:
> >> 
> >> >   TEST  tests/qapi-schema/include-before-err.out
> >> > Files 
> >> > /home/lcapitulino/work/src/upstream/qmp-unstable/tests/qapi-schema/include-before-err.err
> >> >  and - differ
> >> > make: *** [check-tests/qapi-schema/include-before-err.json] Error 1
> >> 
> >> Sorry, it's my fault. I though I did rerun the tests. The change is:
> >> 
> >> sed -i -e 's/after/before' tests/qapi-schema/include-before-err.err
> >> 
> >> Should I resend the series?
> 
> > It would be nice if you can resend only the affected patch.
> 
> You mean resend a fixed version of the original patch, or a patch on top of 
> the
> series?

A fixed version of the original patch. You can do it by replying to the
original patch.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]