All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: "Andreas Färber" <afaerber@suse.de>
Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org, rth@twiddle.net
Subject: Re: [Qemu-devel] [PATCH for-2.3? 6/7] tests/tcg: Prettify run-runcom output
Date: Mon, 13 Apr 2015 16:37:53 +0100	[thread overview]
Message-ID: <87twwkjay6.fsf@linaro.org> (raw)
In-Reply-To: <1428777292-24628-7-git-send-email-afaerber@suse.de>


Andreas Färber <afaerber@suse.de> writes:

> Executing the binary pi_10.com terminates without a trailing newline.
>
> As it does not seem to be caused by the runcom program and since we
> can't easily fix pi_10.com, add an echo after executing it. This places
> "make:" output (or that of a target following it) on a new line as expected.
>
> Signed-off-by: Andreas Färber <afaerber@suse.de>
> ---
>  tests/tcg/Makefile | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/tests/tcg/Makefile b/tests/tcg/Makefile
> index db8c510..f853d54 100644
> --- a/tests/tcg/Makefile
> +++ b/tests/tcg/Makefile
> @@ -75,6 +75,7 @@ run-test-mmap: test-mmap
>  
>  run-runcom: runcom
>  	-$(QEMU) ./runcom $(SRC_PATH)/tests/tcg/pi_10.com
> +	@echo

I'm not so sure about this. Shouldn't we be being a lot quieter by
default. At the moment the TCG tests are very noisy with their output
compared to the rest of the tests. On a V=0 run I expect just to see
test name and a PASS/FAIL.

>  
>  run-test_path: test_path
>  	./test_path

-- 
Alex Bennée

  reply	other threads:[~2015-04-13 15:37 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-11 18:34 [Qemu-devel] [PATCH for-2.3? 0/7] tests: Fix TCG make test Andreas Färber
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 1/7] tests/tcg: Fix test_path build Andreas Färber
2015-04-13 11:57   ` Alex Bennée
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 2/7] tests/tcg: Fix linux-test build Andreas Färber
2015-04-13 15:34   ` Alex Bennée
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 3/7] tests/tcg: Disable hanging test-mmap runs Andreas Färber
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 4/7] tests/tcg: Fix running x86_64 tests Andreas Färber
2015-04-13 15:35   ` Alex Bennée
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 5/7] tests/tcg: Fix run-runcom target Andreas Färber
2015-04-13 15:36   ` Alex Bennée
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 6/7] tests/tcg: Prettify run-runcom output Andreas Färber
2015-04-13 15:37   ` Alex Bennée [this message]
2015-04-13 15:38     ` Peter Maydell
2015-04-11 18:34 ` [Qemu-devel] [PATCH for-2.3? 7/7] tests/tcg: Enable runcom test by default Andreas Färber
2015-04-11 20:22   ` Peter Maydell
2015-04-11 20:28     ` Andreas Färber
2015-04-11 21:33       ` Peter Maydell
2015-04-11 21:46         ` Andreas Färber
2015-04-12  8:34           ` Alexander Graf
2015-04-12  8:39             ` Andreas Färber
2015-04-12 11:45             ` Peter Maydell
2015-04-13 11:36 ` [Qemu-devel] [PATCH for-2.3? 0/7] tests: Fix TCG make test Peter Maydell
2015-04-13 14:00   ` Peter Maydell
2015-06-24 14:19 ` Andreas Färber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87twwkjay6.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=afaerber@suse.de \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.