[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 1/4] qapi: [trivial] Break long command line
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v10 1/4] qapi: [trivial] Break long command lines |
Date: |
Thu, 01 May 2014 06:34:24 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 04/30/2014 01:09 PM, Lluís Vilanova wrote:
> Signed-off-by: Lluís Vilanova <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---
> Makefile | 24 ++++++++++++++++++------
> tests/Makefile | 20 ++++++++++++++++----
> 2 files changed, 34 insertions(+), 10 deletions(-)
>
> @@ -362,7 +368,13 @@ check-tests/test-qapi.py: tests/test-qapi.py
>
> .PHONY: $(patsubst %, check-%, $(check-qapi-schema-y))
> $(patsubst %, check-%, $(check-qapi-schema-y)): check-%.json:
> $(SRC_PATH)/%.json
> - $(call quiet-command, PYTHONPATH=$(SRC_PATH)/scripts $(PYTHON)
> $(SRC_PATH)/tests/qapi-schema/test-qapi.py <$^ >$*.test.out 2>$*.test.err;
> echo $$? >$*.test.exit, " TEST $*.out")
> + $(call quiet-command, PYTHONPATH=$(SRC_PATH)/scripts \
> + $(PYTHON) $(SRC_PATH)/tests/qapi-schema/test-qapi.py \
> + <$^ \
> + >$*.test.out \
> + 2>$*.test.err; \
I've already made the comment several times that I would have put these
three lines as one instead of excessively wrapping into three; I'm a bit
surprised you haven't picked up on the hint when rebasing for other
reasons. But the patch as-is is still valid, so my Reviewed-by is still
okay whether or not you have a reason to spin a v11.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v10 1/4] qapi: [trivial] Break long command lines,
Eric Blake <=