Re: [Qemu-devel] [PATCH v1 0/4] Updates for Travis testing

[Stefan Hajnoczi]

On Thu, Feb 20, 2014 at 03:37:12PM +0000, address@hidden wrote:
> From: Alex Bennée <address@hidden>
> 
> Hi,
> 
> I've now enabled Travis testing on the "official" QEMU GitHub mirror.
> 
> Of the following patches two expand the testing, one is a cosmetic
> whitespace fix and the final one enabled IRC notification when the
> build fails. This is potentially controversial but on balance I think
> it would be worth while letting #qemu know as soon as master breaks.
> If lots of people start enabling Travis on there own repos then maybe
> we would reconsider the notification idea.
> 
> As it is build failures should automatically email the author of
> ${HEAD} as well as the owner of the repo.
> 
> There has been interest expressed in running tests on all PULL
> requests. One option would be to have some sort of bot (via
> patchwork?) push PULL request heads to another GitHub mirror
> (qemu-pull-requests.git?) to trigger builds.
> 
> 
> Alex Bennée (4):
>   .travis.yml: add a new build target with non-core devlibs
>   .travis.yml: re-enable lttng user space trace test
>   .travis.yml: trivial whitespace fixup
>   .travis.yml: add IRC notifications for build failures
> 
>  .travis.yml | 53 +++++++++++++++++++++++++++++++----------------------
>  1 file changed, 31 insertions(+), 22 deletions(-)

Reviewed-by: Stefan Hajnoczi <address@hidden>

Regarding IRC spamming, I think this is okay for now.  If it becomes
more annoying than helpful (e.g. too many people pushing broken builds
onto GitHub) then we can always change .travis.yml.