[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/0] .travis and minor compile fixes
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/0] .travis and minor compile fixes |
Date: |
Tue, 24 Sep 2013 14:07:07 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Sep 23, 2013 at 05:07:27PM +0100, address@hidden wrote:
> I've updated the .travis.yml following feedback from Stefan to greatly
> increase the number of compiles it does. In the process:
>
> * Disabled ust backend (horribly broken)
> * Found gov/gprof builds don't enforce --disable-pie
> * Fixed a minor compiler warning when stderr/ftrace back-end enabled
>
> I'm sending this before confirming a clean run (which takes around 2
> hours on the Travis infrastructure). However you can watch it's
> progress at:
>
> https://travis-ci.org/stsquad/qemu/builds/11694820
>
> Given I found a couple of issues while doing this I think this is
> useful alongside the existing buildbot efforts. It allows developers
> to run simple smoke-tests without access to the buildbot
> infrastructure (but of course needing github/travis access, free for
> FLOSS projects). I can see further development possibilities in the
> future, one example would be a tree that auto-generates branches with
> testing results for each patch series as they come into the mailing
> list.
>
> Alex Bennée (3):
> .travis.yml: basic compile and check recipes
> .travis.yml: greatly expand the coverage + more builds
> block/stream.c: ensure copy always set
>
> .travis.yml | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> block/stream.c | 2 ++
> 2 files changed, 57 insertions(+)
The last patch can be dropped since Stefan Weil's fix has already been
reviewed on the list.
Reviewed-by: Stefan Hajnoczi <address@hidden>