On Thu, Feb 20, 2014 at 03:37:12PM +0000, alex.ben...@linaro.org wrote:
> From: Alex Bennée <alex.ben...@linaro.org>
>
> 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 <stefa...@redhat.com>
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.
