All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Graf <agraf@suse.de>
To: "Alex Bennée" <alex.bennee@linaro.org>, qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org
Subject: Re: [Qemu-devel] [PATCH 0/4] A number of Travis CI tweaks
Date: Tue, 23 Sep 2014 12:38:22 +0200	[thread overview]
Message-ID: <54214D9E.8090602@suse.de> (raw)
In-Reply-To: <1410799691-1226-1-git-send-email-alex.bennee@linaro.org>



On 15.09.14 18:48, Alex Bennée wrote:
> Hi,
> 
> While I was in-between kernel builds last week I attempted to improve
> the Travis build a little. Alexander Graf pointed out we were missing
> a number of the linux-user targets. To avoid exploding the matrix too
> much I've grouped builds together where they hopefully benefit from
> sharing some objtect files. The biggest win however was using a
> tarball to pre-seed the sub-module checkouts. I'm not sure if this is
> because hammering our git server slows down or just because it's a lot
> of data but it was adding up to around half the execution time of the
> build.
> 
> Finally I removed "make check" from every build. It still gets run
> once in the matrix but this means the current instability will bite
> less often.
> 
> I failed in the task of adding mingw builds because it's hard on the
> Travis Ubuntu hosts to install the headers/libs for a cross compile.
> If someone can come up with a nice solid script that hand pulls in
> these dependancies then we can add a call to that for the mingw
> builds. Any volenteers?
> 
> If there are no objections/negative reviews I'll push these at the
> Trivial tree later this week. Or is it time we create a specific
> testing tree for these particular type of patches?

I like it :). Except for the small nit on 3/4.

Reviewed-by: Alexander Graf <agraf@suse.de>


Alex

      parent reply	other threads:[~2014-09-23 10:38 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-15 16:48 [Qemu-devel] [PATCH 0/4] A number of Travis CI tweaks Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 1/4] .travis.yml: add more linux-user to the build matrix Alex Bennée
2014-09-17 18:36   ` Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 2/4] .travis.yml: make the make slightly more parallel Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 3/4] .travis.yml: pre-seed sub-modules for speed Alex Bennée
2014-09-23 10:37   ` Alexander Graf
2014-09-23 14:50     ` Alex Bennée
2014-09-23 15:37       ` Alexander Graf
2014-09-15 16:48 ` [Qemu-devel] [PATCH 4/4] .travis.yml: remove "make check" from main matrix Alex Bennée
2014-09-23 10:38 ` Alexander Graf [this message]

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=54214D9E.8090602@suse.de \
    --to=agraf@suse.de \
    --cc=alex.bennee@linaro.org \
    --cc=peter.maydell@linaro.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 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.