All of lore.kernel.org
 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-trivial] [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"



WARNING: multiple messages have this Message-ID (diff)
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 20:47 UTC|newest]

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