qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Brian Jackson <brian@vten.us>
To: qemu-devel@nongnu.org
Cc: qemu-trivial@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
	agraf@suse.de
Subject: Re: [Qemu-devel] [PATCH v2 4/4] .travis.yml: remove "make check" from main matrix
Date: Wed, 24 Sep 2014 10:13:05 -0500	[thread overview]
Message-ID: <8024367.sEpDvf3ZWp@iggy-ub> (raw)
In-Reply-To: <1411553636-24717-5-git-send-email-alex.bennee@linaro.org>

On Wednesday, September 24, 2014 11:13:56 AM Alex Bennée wrote:
> There are problems with unreliability in "make check" which still need
> to be tracked down. As the tests are broadly the same for all targets if
> added one explicit target to the matrix to run it. However this does
> build all softmmu targets to ensure they at least "run"
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Reviewed-by: Alexander Graf <agraf@suse.de>

Reviewed-by: Brian Jackson <iggy@theiggy.com>

> 
> diff --git a/.travis.yml b/.travis.yml
> index b9025be..ad66e5b 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -12,7 +12,7 @@ notifications:
>      on_failure: always
>  env:
>    global:
> -    - TEST_CMD="make check"
> +    - TEST_CMD=""
>      - EXTRA_CONFIG=""
>      # Development packages, EXTRA_PKGS saved for additional builds
>      - CORE_PKGS="libusb-1.0-0-dev libiscsi-dev librados-dev
> libncurses5-dev" @@ -48,10 +48,23 @@ before_install:
>  before_script:
>    - ./configure --target-list=${TARGETS} --enable-debug-tcg ${EXTRA_CONFIG}
> script:
> -  - make -j2 && {TEST_CMD}
> +  - make -j2 && ${TEST_CMD}
>  matrix:
>    # We manually include a number of additional build for non-standard bits
>    include:
> +    # Make check target (we only do this once)
> +    - env:
> +        - TARGETS=alpha-softmmu,arm-softmmu,aarch64-softmmu,cris-softmmu,
> +                 
> i386-softmmu,x86_64-softmmu,m68k-softmmu,microblaze-softmmu, +             
>     microblazeel-softmmu,mips-softmmu,mips64-softmmu, +                 
> mips64el-softmmu,mipsel-softmmu,or32-softmmu,ppc-softmmu, +                
>  ppc64-softmmu,ppcemb-softmmu,s390x-softmmu,sh4-softmmu, +                 
> sh4eb-softmmu,sparc-softmmu,sparc64-softmmu,
> +                  unicore32-softmmu,unicore32-linux-user,
> +                 
> lm32-softmmu,moxie-softmmu,tricore-softmmu,xtensa-softmmu, +               
>   xtensaeb-softmmu
> +          TEST_CMD="make check"
> +      compiler: gcc
>      # Debug related options
>      - env: TARGETS=i386-softmmu,x86_64-softmmu
>             EXTRA_CONFIG="--enable-debug"
> @@ -80,7 +93,6 @@ matrix:
>        compiler: gcc
>      - env: TARGETS=i386-softmmu,x86_64-softmmu
>             EXTRA_CONFIG="--enable-trace-backends=ftrace"
> -           TEST_CMD=""
>        compiler: gcc
>      - env: TARGETS=i386-softmmu,x86_64-softmmu
>            EXTRA_PKGS="liblttng-ust-dev liburcu-dev"

  reply	other threads:[~2014-09-24 15:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-24 10:13 [Qemu-devel] [PATCH v2 0/4] A number of Travis CI tweaks Alex Bennée
2014-09-24 10:13 ` [Qemu-devel] [PATCH v2 1/4] .travis.yml: add more linux-user to the build matrix Alex Bennée
2014-09-24 15:08   ` Brian Jackson
2014-09-24 10:13 ` [Qemu-devel] [PATCH v2 2/4] .travis.yml: make the make slightly more parallel Alex Bennée
2014-09-24 15:10   ` Brian Jackson
2014-09-24 10:13 ` [Qemu-devel] [PATCH v2 3/4] .travis.yml: pre-seed sub-modules for speed Alex Bennée
2014-09-24 15:11   ` Brian Jackson
2014-09-24 10:13 ` [Qemu-devel] [PATCH v2 4/4] .travis.yml: remove "make check" from main matrix Alex Bennée
2014-09-24 15:13   ` Brian Jackson [this message]
2014-09-26 17:05 ` [Qemu-devel] [Qemu-trivial] [PATCH v2 0/4] A number of Travis CI tweaks Michael Tokarev

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=8024367.sEpDvf3ZWp@iggy-ub \
    --to=brian@vten.us \
    --cc=agraf@suse.de \
    --cc=alex.bennee@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-trivial@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).