* [Qemu-devel] [PATCH v2] w32: Support tests (make check)
@ 2012-02-22 19:48 Stefan Weil
2012-02-25 16:31 ` Andreas Färber
0 siblings, 1 reply; 4+ messages in thread
From: Stefan Weil @ 2012-02-22 19:48 UTC (permalink / raw)
To: Anthony Liguori; +Cc: Stefan Weil, qemu-devel
Adding $(EXESUF) is needed to make those tests work on w32 hosts, too.
v2:
Rebased, added new tests, tests sorted alphabetically.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
tests/Makefile | 38 +++++++++++++++++++++++---------------
1 files changed, 23 insertions(+), 15 deletions(-)
diff --git a/tests/Makefile b/tests/Makefile
index 74b29dc..09f2b13 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -1,16 +1,24 @@
-CHECKS = check-qdict check-qfloat check-qint check-qstring check-qlist
-CHECKS += check-qjson test-qmp-output-visitor test-qmp-input-visitor
-CHECKS += test-string-input-visitor test-string-output-visitor test-coroutine
+CHECKS = check-qdict$(EXESUF)
+CHECKS += check-qfloat$(EXESUF)
+CHECKS += check-qint$(EXESUF)
+CHECKS += check-qjson$(EXESUF)
+CHECKS += check-qlist$(EXESUF)
+CHECKS += check-qstring$(EXESUF)
+CHECKS += test-coroutine$(EXESUF)
+CHECKS += test-qmp-input-visitor$(EXESUF)
+CHECKS += test-qmp-output-visitor$(EXESUF)
+CHECKS += test-string-input-visitor$(EXESUF)
+CHECKS += test-string-output-visitor$(EXESUF)
check-qint.o check-qstring.o check-qdict.o check-qlist.o check-qfloat.o check-qjson.o test-coroutine.o: $(GENERATED_HEADERS)
-check-qint: check-qint.o qint.o $(tools-obj-y)
-check-qstring: check-qstring.o qstring.o $(tools-obj-y)
-check-qdict: check-qdict.o qdict.o qfloat.o qint.o qstring.o qbool.o qlist.o $(tools-obj-y)
-check-qlist: check-qlist.o qlist.o qint.o $(tools-obj-y)
-check-qfloat: check-qfloat.o qfloat.o $(tools-obj-y)
-check-qjson: check-qjson.o $(qobject-obj-y) $(tools-obj-y)
-test-coroutine: test-coroutine.o qemu-timer-common.o async.o $(coroutine-obj-y) $(tools-obj-y)
+check-qint$(EXESUF): check-qint.o qint.o $(tools-obj-y)
+check-qstring$(EXESUF): check-qstring.o qstring.o $(tools-obj-y)
+check-qdict$(EXESUF): check-qdict.o qdict.o qfloat.o qint.o qstring.o qbool.o qlist.o $(tools-obj-y)
+check-qlist$(EXESUF): check-qlist.o qlist.o qint.o $(tools-obj-y)
+check-qfloat$(EXESUF): check-qfloat.o qfloat.o $(tools-obj-y)
+check-qjson$(EXESUF): check-qjson.o $(qobject-obj-y) $(tools-obj-y)
+test-coroutine$(EXESUF): test-coroutine.o qemu-timer-common.o async.o $(coroutine-obj-y) $(tools-obj-y)
test-qmp-input-visitor.o test-qmp-output-visitor.o \
test-string-input-visitor.o test-string-output-visitor.o \
@@ -28,19 +36,19 @@ $(SRC_PATH)/qapi-schema-test.json $(SRC_PATH)/scripts/qapi-commands.py
test-string-output-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
-test-string-output-visitor: test-string-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
+test-string-output-visitor$(EXESUF): test-string-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
test-string-input-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
-test-string-input-visitor: test-string-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
+test-string-input-visitor$(EXESUF): test-string-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
test-qmp-output-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
-test-qmp-output-visitor: test-qmp-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
+test-qmp-output-visitor$(EXESUF): test-qmp-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
test-qmp-input-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
-test-qmp-input-visitor: test-qmp-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
+test-qmp-input-visitor$(EXESUF): test-qmp-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
test-qmp-commands.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h test-qmp-marshal.c test-qmp-commands.h) $(qapi-obj-y)
-test-qmp-commands: test-qmp-commands.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o $(qapi-dir)/test-qmp-marshal.o module.o
+test-qmp-commands$(EXESUF): test-qmp-commands.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o $(qapi-dir)/test-qmp-marshal.o module.o
.PHONY: check
check: $(CHECKS)
--
1.7.9
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH v2] w32: Support tests (make check)
2012-02-22 19:48 [Qemu-devel] [PATCH v2] w32: Support tests (make check) Stefan Weil
@ 2012-02-25 16:31 ` Andreas Färber
2012-02-25 16:39 ` Stefan Weil
0 siblings, 1 reply; 4+ messages in thread
From: Andreas Färber @ 2012-02-25 16:31 UTC (permalink / raw)
To: Stefan Weil; +Cc: Anthony Liguori, qemu-devel
Am 22.02.2012 20:48, schrieb Stefan Weil:
> Adding $(EXESUF) is needed to make those tests work on w32 hosts, too.
>
> v2:
> Rebased, added new tests, tests sorted alphabetically.
>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> ---
> tests/Makefile | 38 +++++++++++++++++++++++---------------
> 1 files changed, 23 insertions(+), 15 deletions(-)
>
> diff --git a/tests/Makefile b/tests/Makefile
> index 74b29dc..09f2b13 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -1,16 +1,24 @@
> -CHECKS = check-qdict check-qfloat check-qint check-qstring check-qlist
> -CHECKS += check-qjson test-qmp-output-visitor test-qmp-input-visitor
> -CHECKS += test-string-input-visitor test-string-output-visitor test-coroutine
> +CHECKS = check-qdict$(EXESUF)
> +CHECKS += check-qfloat$(EXESUF)
> +CHECKS += check-qint$(EXESUF)
> +CHECKS += check-qjson$(EXESUF)
> +CHECKS += check-qlist$(EXESUF)
> +CHECKS += check-qstring$(EXESUF)
> +CHECKS += test-coroutine$(EXESUF)
> +CHECKS += test-qmp-input-visitor$(EXESUF)
> +CHECKS += test-qmp-output-visitor$(EXESUF)
> +CHECKS += test-string-input-visitor$(EXESUF)
> +CHECKS += test-string-output-visitor$(EXESUF)
Cute. (one-per-line allows to easily comment individual ones out)
Some lines below were already way over 80 chars, so
Reviewed-by: Andreas Färber <afaerber@suse.de>
Andreas
>
> check-qint.o check-qstring.o check-qdict.o check-qlist.o check-qfloat.o check-qjson.o test-coroutine.o: $(GENERATED_HEADERS)
>
> -check-qint: check-qint.o qint.o $(tools-obj-y)
> -check-qstring: check-qstring.o qstring.o $(tools-obj-y)
> -check-qdict: check-qdict.o qdict.o qfloat.o qint.o qstring.o qbool.o qlist.o $(tools-obj-y)
> -check-qlist: check-qlist.o qlist.o qint.o $(tools-obj-y)
> -check-qfloat: check-qfloat.o qfloat.o $(tools-obj-y)
> -check-qjson: check-qjson.o $(qobject-obj-y) $(tools-obj-y)
> -test-coroutine: test-coroutine.o qemu-timer-common.o async.o $(coroutine-obj-y) $(tools-obj-y)
> +check-qint$(EXESUF): check-qint.o qint.o $(tools-obj-y)
> +check-qstring$(EXESUF): check-qstring.o qstring.o $(tools-obj-y)
> +check-qdict$(EXESUF): check-qdict.o qdict.o qfloat.o qint.o qstring.o qbool.o qlist.o $(tools-obj-y)
> +check-qlist$(EXESUF): check-qlist.o qlist.o qint.o $(tools-obj-y)
> +check-qfloat$(EXESUF): check-qfloat.o qfloat.o $(tools-obj-y)
> +check-qjson$(EXESUF): check-qjson.o $(qobject-obj-y) $(tools-obj-y)
> +test-coroutine$(EXESUF): test-coroutine.o qemu-timer-common.o async.o $(coroutine-obj-y) $(tools-obj-y)
>
> test-qmp-input-visitor.o test-qmp-output-visitor.o \
> test-string-input-visitor.o test-string-output-visitor.o \
> @@ -28,19 +36,19 @@ $(SRC_PATH)/qapi-schema-test.json $(SRC_PATH)/scripts/qapi-commands.py
>
>
> test-string-output-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
> -test-string-output-visitor: test-string-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
> +test-string-output-visitor$(EXESUF): test-string-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
>
> test-string-input-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
> -test-string-input-visitor: test-string-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
> +test-string-input-visitor$(EXESUF): test-string-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
>
> test-qmp-output-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
> -test-qmp-output-visitor: test-qmp-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
> +test-qmp-output-visitor$(EXESUF): test-qmp-output-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
>
> test-qmp-input-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
> -test-qmp-input-visitor: test-qmp-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
> +test-qmp-input-visitor$(EXESUF): test-qmp-input-visitor.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
>
> test-qmp-commands.o: $(addprefix $(qapi-dir)/, test-qapi-types.c test-qapi-types.h test-qapi-visit.c test-qapi-visit.h test-qmp-marshal.c test-qmp-commands.h) $(qapi-obj-y)
> -test-qmp-commands: test-qmp-commands.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o $(qapi-dir)/test-qmp-marshal.o module.o
> +test-qmp-commands$(EXESUF): test-qmp-commands.o $(qobject-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o $(qapi-dir)/test-qmp-marshal.o module.o
>
> .PHONY: check
> check: $(CHECKS)
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH v2] w32: Support tests (make check)
2012-02-25 16:31 ` Andreas Färber
@ 2012-02-25 16:39 ` Stefan Weil
2012-02-25 16:52 ` Andreas Färber
0 siblings, 1 reply; 4+ messages in thread
From: Stefan Weil @ 2012-02-25 16:39 UTC (permalink / raw)
To: Andreas Färber; +Cc: Anthony Liguori, qemu-devel
Am 25.02.2012 17:31, schrieb Andreas Färber:
> Am 22.02.2012 20:48, schrieb Stefan Weil:
>> Adding $(EXESUF) is needed to make those tests work on w32 hosts, too.
>>
>> v2:
>> Rebased, added new tests, tests sorted alphabetically.
>>
>> Signed-off-by: Stefan Weil <sw@weilnetz.de>
>> ---
>> tests/Makefile | 38 +++++++++++++++++++++++---------------
>> 1 files changed, 23 insertions(+), 15 deletions(-)
>>
>> diff --git a/tests/Makefile b/tests/Makefile
>> index 74b29dc..09f2b13 100644
>> --- a/tests/Makefile
>> +++ b/tests/Makefile
>> @@ -1,16 +1,24 @@
>> -CHECKS = check-qdict check-qfloat check-qint check-qstring check-qlist
>> -CHECKS += check-qjson test-qmp-output-visitor test-qmp-input-visitor
>> -CHECKS += test-string-input-visitor test-string-output-visitor
>> test-coroutine
>> +CHECKS = check-qdict$(EXESUF)
>> +CHECKS += check-qfloat$(EXESUF)
>> +CHECKS += check-qint$(EXESUF)
>> +CHECKS += check-qjson$(EXESUF)
>> +CHECKS += check-qlist$(EXESUF)
>> +CHECKS += check-qstring$(EXESUF)
>> +CHECKS += test-coroutine$(EXESUF)
>> +CHECKS += test-qmp-input-visitor$(EXESUF)
>> +CHECKS += test-qmp-output-visitor$(EXESUF)
>> +CHECKS += test-string-input-visitor$(EXESUF)
>> +CHECKS += test-string-output-visitor$(EXESUF)
>
> Cute. (one-per-line allows to easily comment individual ones out)
>
> Some lines below were already way over 80 chars, so
>
> Reviewed-by: Andreas Färber <afaerber@suse.de>
>
> Andreas
And as long as they remain sorted, they also reduce the risk
of merge conflicts:
When everybody just adds to the end,two different commits
will always conflict.
They won't conflictif both commits add new lines at different
locations.
Regards,
Stefan
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH v2] w32: Support tests (make check)
2012-02-25 16:39 ` Stefan Weil
@ 2012-02-25 16:52 ` Andreas Färber
0 siblings, 0 replies; 4+ messages in thread
From: Andreas Färber @ 2012-02-25 16:52 UTC (permalink / raw)
To: Stefan Weil; +Cc: Anthony Liguori, qemu-devel
Am 25.02.2012 17:39, schrieb Stefan Weil:
> Am 25.02.2012 17:31, schrieb Andreas Färber:
>> Am 22.02.2012 20:48, schrieb Stefan Weil:
>>> Adding $(EXESUF) is needed to make those tests work on w32 hosts, too.
>>>
>>> v2:
>>> Rebased, added new tests, tests sorted alphabetically.
>>>
>>> Signed-off-by: Stefan Weil <sw@weilnetz.de>
>>> ---
>>> tests/Makefile | 38 +++++++++++++++++++++++---------------
>>> 1 files changed, 23 insertions(+), 15 deletions(-)
>>>
>>> diff --git a/tests/Makefile b/tests/Makefile
>>> index 74b29dc..09f2b13 100644
>>> --- a/tests/Makefile
>>> +++ b/tests/Makefile
>>> @@ -1,16 +1,24 @@
>>> -CHECKS = check-qdict check-qfloat check-qint check-qstring check-qlist
>>> -CHECKS += check-qjson test-qmp-output-visitor test-qmp-input-visitor
>>> -CHECKS += test-string-input-visitor test-string-output-visitor
>>> test-coroutine
>>> +CHECKS = check-qdict$(EXESUF)
>>> +CHECKS += check-qfloat$(EXESUF)
>>> +CHECKS += check-qint$(EXESUF)
>>> +CHECKS += check-qjson$(EXESUF)
>>> +CHECKS += check-qlist$(EXESUF)
>>> +CHECKS += check-qstring$(EXESUF)
>>> +CHECKS += test-coroutine$(EXESUF)
>>> +CHECKS += test-qmp-input-visitor$(EXESUF)
>>> +CHECKS += test-qmp-output-visitor$(EXESUF)
>>> +CHECKS += test-string-input-visitor$(EXESUF)
>>> +CHECKS += test-string-output-visitor$(EXESUF)
>>
>> Cute. (one-per-line allows to easily comment individual ones out)
>>
>> Some lines below were already way over 80 chars, so
>>
>> Reviewed-by: Andreas Färber <afaerber@suse.de>
>>
>> Andreas
>
> And as long as they remain sorted, they also reduce the risk
> of merge conflicts:
>
> When everybody just adds to the end,two different commits
> will always conflict.
>
> They won't conflictif both commits add new lines at different
> locations.
I'd rather not strictly keep it in alphabetical order since having one
test pass might be a prerequisite for another test to be meaningful
(e.g., qfloat before pi). IMO we should have groups of tests, in which
we can order them alphabetically.
Anyway, my line of thinking was that no such interdependency is
documented here so it should be permissible to rearrange the order now.
The q* and the visitors are still together, input before output, so I
like it. We could add lines before test-coroutine and
test-qmp-input-visitors to group them by topic if we wanted, but the
list is still small, so no real need.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-02-25 16:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-22 19:48 [Qemu-devel] [PATCH v2] w32: Support tests (make check) Stefan Weil
2012-02-25 16:31 ` Andreas Färber
2012-02-25 16:39 ` Stefan Weil
2012-02-25 16:52 ` Andreas Färber
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).