qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Cleber Rosa <crosa@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Kevin Wolf" <kwolf@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Max Reitz" <mreitz@redhat.com>,
	qemu-block@nongnu.org, "Fam Zheng" <fam@euphon.net>,
	"Cleber Rosa" <crosa@redhat.com>
Subject: [Qemu-devel] [PATCH 2/2] qemu-iotests: add "make check-block" job to Travis
Date: Fri,  8 Feb 2019 23:01:32 -0500	[thread overview]
Message-ID: <20190209040132.25573-3-crosa@redhat.com> (raw)
In-Reply-To: <20190209040132.25573-1-crosa@redhat.com>

I usually keep an eye if I'm breaking stuff by looking at Travis
(too).  So it seems like a good idea to add a job that will do that.

A few things deserve a better explanation:

  * test 233 requires "--enable-gnutls", and fails without it

  * QEMU seems unable to initialize sound devices on the environment
    given by Travis, and that's why it's compiled without any audio
    driver.

  * The syntax for setting an empty list of audio drivers possibly
    deserves some TLC (it's not currently possible to set it to an
    empty list with quotes, resulting in something like "Unknown
    driver '''' selected".

Signed-off-by: Cleber Rosa <crosa@redhat.com>
---
 .travis.yml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index 87d9fa971c..6d60ae9762 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -196,6 +196,14 @@ matrix:
           packages:
             - python3-pip
             - python3.5-venv
+
+
+    # qemu-iotests
+    - env:
+        - CONFIG="--target-list=x86_64-softmmu --enable-gnutls --audio-drv-list="
+        - TEST_CMD="make check-block"
+
+
     # Using newer GCC with sanitizers
     - addons:
         apt:
-- 
2.20.1

  parent reply	other threads:[~2019-02-09  4:02 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-09  4:01 [Qemu-devel] [PATCH 0/2] qemu-iotests: add a Travis job Cleber Rosa
2019-02-09  4:01 ` [Qemu-devel] [PATCH 1/2] qemu-iotests: only enable kvm if the host seems to support it Cleber Rosa
2019-02-09  4:01 ` Cleber Rosa [this message]
2019-02-18 18:05 ` [Qemu-devel] [PATCH 0/2] qemu-iotests: add a Travis job Cleber Rosa

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=20190209040132.25573-3-crosa@redhat.com \
    --to=crosa@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=fam@euphon.net \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=philmd@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /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 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).