[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v2 3/5] tests: Make check-block a phony targ
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH RFC v2 3/5] tests: Make check-block a phony target |
Date: |
Mon, 27 Jun 2016 08:35:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 06/24/2016 08:19 AM, Markus Armbruster wrote:
>> Forgotten in commit b8c6f29.
>
> Umm, really? That commit starts with:
>
> tests/Makefile
> -.PHONY: check
> +
> +.PHONY: check check-block
>
> Rather, I think it was b93b63f5 that accidentally lost it during an
> overhaul.
Oops. Will fix.
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> tests/Makefile.include | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/Makefile.include b/tests/Makefile.include
>> index 7b0446b..233cb9c 100644
>> --- a/tests/Makefile.include
>> +++ b/tests/Makefile.include
>> @@ -752,7 +752,7 @@ tests/headers/%.c: tests/header-test-template.c
>>
>> # Consolidated targets
>>
>> -.PHONY: check-qapi-schema check-qtest check-unit check check-source
>> check-clean
>> +.PHONY: check-qapi-schema check-qtest check-unit check check-block
>> check-source check-clean
>
> Long line; worth splitting?
Yes.
> With commit message improved,
> Reviewed-by: Eric Blake <address@hidden>
Thanks!
- [Qemu-devel] [PATCH RFC v2 1/5] Use #include "..." exactly for our own headers, (continued)
[Qemu-devel] [PATCH RFC v2 3/5] tests: Make check-block a phony target, Markus Armbruster, 2016/06/24
[Qemu-devel] [PATCH RFC v2 5/5] include: Include exec/hwaddr.h where hwaddr is used, Markus Armbruster, 2016/06/24
[Qemu-devel] [PATCH RFC v2 2/5] tests: New make target check-source, Markus Armbruster, 2016/06/24
[Qemu-devel] [PATCH RFC] fixup! tests: New make target check-source, Markus Armbruster, 2016/06/30