Re: [Qemu-devel] [PATCH v4 26/26] qidl: unit tests and build infrastructure

2012-10-18 Thread Michael Roth
On Tue, Oct 16, 2012 at 09:21:05AM +0200, Paolo Bonzini wrote: > Il 15/10/2012 18:37, Michael Roth ha scritto: > > There is another way to do this (still using a "dummy" target) that is a > > bit less cryptic: > > > > QIDL-PP-%: %.c qidl.h ... > > > > > > %.o: QIDL-PP-% > > > > > > Bu

Re: [Qemu-devel] [PATCH v4 26/26] qidl: unit tests and build infrastructure

2012-10-16 Thread Paolo Bonzini
Il 15/10/2012 18:37, Michael Roth ha scritto: > There is another way to do this (still using a "dummy" target) that is a > bit less cryptic: > > QIDL-PP-%: %.c qidl.h ... > > > %.o: QIDL-PP-% > > > But make detects that QIDL-PP-% is an intermediate target and removes > the *.qidl.c fi

Re: [Qemu-devel] [PATCH v4 26/26] qidl: unit tests and build infrastructure

2012-10-15 Thread Michael Roth
On Mon, Oct 15, 2012 at 12:05:09PM +0200, Paolo Bonzini wrote: > Il 12/10/2012 23:11, Michael Roth ha scritto: > > +%.qidl.c: %.c $(SRC_PATH)/qidl.h $(addprefix $(SRC_PATH)/scripts/,lexer.py > > qidl.py qidl_parser.py qapi.py qapi_visit.py) > > The rule here is wrong, because %.qidl.c is never pr

Re: [Qemu-devel] [PATCH v4 26/26] qidl: unit tests and build infrastructure

2012-10-15 Thread Paolo Bonzini
Il 12/10/2012 23:11, Michael Roth ha scritto: > +%.qidl.c: %.c $(SRC_PATH)/qidl.h $(addprefix $(SRC_PATH)/scripts/,lexer.py > qidl.py qidl_parser.py qapi.py qapi_visit.py) The rule here is wrong, because %.qidl.c is never produced by the commands. The output is in the qidl-generated subdirectory

[Qemu-devel] [PATCH v4 26/26] qidl: unit tests and build infrastructure

2012-10-12 Thread Michael Roth
Signed-off-by: Michael Roth --- Makefile |2 + configure |1 + rules.mak | 25 ++- tests/Makefile |8 +- tests/test-qidl-included.h | 31 tests/test-qidl-linked.c | 93 +++ tests/test-qidl-linked.h