[Qemu-devel] [PULL 0/3] Travis updates

[Qemu-devel] [PULL 0/3] Travis updates

[Alex Bennée]

The following changes since commit 40eeb397c8b8008aa13bca3a8cb25d152eb5ab44:

  Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2016-06-07 15:59:28 +0100)

are available in the git repository at:

  https://github.com/stsquad/qemu.git tags/pull-travis-20160621-1

for you to fetch changes up to 23654d45f8ac08e3725831d812eb8dcf74e5a0a0:

  .travis.yml: disable Sparse testing (2016-06-08 17:47:22 +0100)

----------------------------------------------------------------
This pull request contains:

  - disable sparse testing
  - add trusty build target
  - add libnfs-dev for NFS block driver

These are the same patches posted last week for any last minute review.

----------------------------------------------------------------
Alex Bennée (1):
      .travis.yml: add trusty GCE target

Paolo Bonzini (1):
      .travis.yml: disable Sparse testing

Stefan Hajnoczi (1):
      .travis.yml: add libnfs-dev for NFS block driver

 .travis.yml | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

-- 
2.7.4

[Qemu-devel] [PULL 1/3] .travis.yml: add libnfs-dev for NFS block driver

[Alex Bennée]

From: Stefan Hajnoczi <stefanha@redhat.com>

Let's ensure that block/nfs.o is built in Travis.

This patch depends on the following build fixes:
1. block/nfs: add missing #include "qapi/error.h"
2. block/nfs: add missing #include "qemu/cutils.h"

This patch also depends on Travis adding libnfs-dev to the list of
approved packages.  This patch can be safely committed but will not do
anything until the Travis maintainers allow libnfs-dev to be installed.
Please see the GitHub Issue I raised here:
https://github.com/travis-ci/apt-package-whitelist/issues/2788

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .travis.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.travis.yml b/.travis.yml
index 50ac17f..beb98d3 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -17,6 +17,7 @@ addons:
       - libgtk-3-dev
       - libiscsi-dev
       - liblttng-ust-dev
+      - libnfs-dev
       - libncurses5-dev
       - libnss3-dev
       - libpixman-1-dev
-- 
2.7.4

[Qemu-devel] [PULL 2/3] .travis.yml: add trusty GCE target

[Alex Bennée]

If we want to run our docker based tests we'll need to do them on a
normal VM with docker support. Lets just enable the build on trusty for
now to check against a newer Ubuntu.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .travis.yml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index beb98d3..cde7d1a 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -89,3 +89,13 @@ matrix:
     - env: CONFIG=""
       os: osx
       compiler: clang
+    - env: CONFIG=""
+      sudo: required
+      addons:
+      dist: trusty
+      compiler: gcc
+      before_install:
+        - sudo apt-get update -qq
+        - sudo apt-get build-dep -qq qemu
+        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
+        - git submodule update --init --recursive
-- 
2.7.4

[Qemu-devel] [PULL 3/3] .travis.yml: disable Sparse testing

[Alex Bennée]

From: Paolo Bonzini <pbonzini@redhat.com>

On travis-ci.org, all builds fail with
   /usr/include/features.h:324:11: error: unable to open bits/predefs.h

With "make docker-travis@ubuntu", they fail with
   /usr/include/features.h:374:13: error: unable to open sys/cdefs.h

With "make docker-travis@fedora", finally, they fail due to sparse
not being able to parse some #pragmas in glib headers.  Just kill
the thing from the CI builds.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
[AJB: tweak title for my OCD]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .travis.yml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index cde7d1a..c13881e 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -64,9 +64,6 @@ script:
   - make -j3 && ${TEST_CMD}
 matrix:
   include:
-    # Sparse is GCC only
-    - env: CONFIG="--enable-sparse"
-      compiler: gcc
     # gprof/gcov are GCC features
     - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
       compiler: gcc
-- 
2.7.4

Re: [Qemu-devel] [PULL 0/3] Travis updates

[Peter Maydell]

On 21 June 2016 at 12:32, Alex Bennée <alex.bennee@linaro.org> wrote:
> The following changes since commit 40eeb397c8b8008aa13bca3a8cb25d152eb5ab44:
>
>   Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2016-06-07 15:59:28 +0100)
>
> are available in the git repository at:
>
>   https://github.com/stsquad/qemu.git tags/pull-travis-20160621-1
>
> for you to fetch changes up to 23654d45f8ac08e3725831d812eb8dcf74e5a0a0:
>
>   .travis.yml: disable Sparse testing (2016-06-08 17:47:22 +0100)
>
> ----------------------------------------------------------------
> This pull request contains:
>
>   - disable sparse testing
>   - add trusty build target
>   - add libnfs-dev for NFS block driver
>
> These are the same patches posted last week for any last minute review.

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/3] Travis updates

[Alex Bennée]

The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5:

  Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100)

are available in the Git repository at:

  https://github.com/stsquad/qemu.git tags/pull-travis-updates-140618-1

for you to fetch changes up to f8309de9b7e7de1bfdfaddf8df4130a06e7e31b7:

  travis: reduce time taken for trace-backend testing (2018-06-14 20:24:07 +0100)

----------------------------------------------------------------
Travis updates

  - show config.log when failing
  - reduce time for gprof build
  - reduce time for alternate trace builds

----------------------------------------------------------------
Alex Bennée (2):
      travis: reduce coverage of gprof build
      travis: reduce time taken for trace-backend testing

Daniel P. Berrangé (1):
      travis: display config.log when configure fails

 .travis.yml | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

-- 
2.17.1

Re: [Qemu-devel] [PULL 0/3] Travis updates

[Peter Maydell]

On 15 June 2018 at 12:09, Alex Bennée <alex.bennee@linaro.org> wrote:
> The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/stsquad/qemu.git tags/pull-travis-updates-140618-1
>
> for you to fetch changes up to f8309de9b7e7de1bfdfaddf8df4130a06e7e31b7:
>
>   travis: reduce time taken for trace-backend testing (2018-06-14 20:24:07 +0100)
>
> ----------------------------------------------------------------
> Travis updates
>
>   - show config.log when failing
>   - reduce time for gprof build
>   - reduce time for alternate trace builds

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/3] Travis updates

[Alex Bennée]

The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:

  net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)

are available in the git repository at:

  https://github.com/stsquad/qemu.git tags/travis-pull-04042016

for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:

  .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)

----------------------------------------------------------------
This pull request includes:
  - further collapse of the build matrix
  - enabling MacOSX in the build
  - make -j3 change

Other pending updates are deferred for later in the cycle.

----------------------------------------------------------------
Alex Bennée (3):
      .travis.yml: collapse the test matrix
      .travis.yml: enable OSX builds
      .travis.yml: make -j3

 .travis.yml | 82 ++++++++++++++++++-------------------------------------------
 1 file changed, 24 insertions(+), 58 deletions(-)

-- 
2.7.4

Re: [Qemu-devel] [PULL 0/3] Travis updates

[Peter Maydell]

On 4 April 2016 at 18:45, Alex Bennée <alex.bennee@linaro.org> wrote:
> The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:
>
>   net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)
>
> are available in the git repository at:
>
>   https://github.com/stsquad/qemu.git tags/travis-pull-04042016
>
> for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:
>
>   .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)
>
> ----------------------------------------------------------------
> This pull request includes:
>   - further collapse of the build matrix
>   - enabling MacOSX in the build
>   - make -j3 change
>
> Other pending updates are deferred for later in the cycle.

This looks good, but you've left the '---' and v2 changes notes
in the commit message for patch 2 by mistake. Can I get you to
delete those and resend? (You can just send the new cover letter,
no need to resend the patches themselves.)

thanks
-- PMM

Re: [Qemu-devel] [PULL 0/3] Travis updates

[Alex Bennée]

Peter Maydell <peter.maydell@linaro.org> writes:

> On 4 April 2016 at 18:45, Alex Bennée <alex.bennee@linaro.org> wrote:
>> The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:
>>
>>   net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)
>>
>> are available in the git repository at:
>>
>>   https://github.com/stsquad/qemu.git tags/travis-pull-04042016
>>
>> for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:
>>
>>   .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)
>>
>> ----------------------------------------------------------------
>> This pull request includes:
>>   - further collapse of the build matrix
>>   - enabling MacOSX in the build
>>   - make -j3 change
>>
>> Other pending updates are deferred for later in the cycle.
>
> This looks good, but you've left the '---' and v2 changes notes
> in the commit message for patch 2 by mistake. Can I get you to
> delete those and resend? (You can just send the new cover letter,
> no need to resend the patches themselves.)

Doh, I'll clean that up.

I guess I need to script up my "prepare pull" step to remove those in
future.

>
> thanks
> -- PMM


--
Alex Bennée