It's broken so it's no longer helping. The latest Xcode is covered by
Cirrus.

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

diff --git a/.travis.yml b/.travis.yml
index c09b6a00143..445b0646c18 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -237,13 +237,7 @@ matrix:
         - TEST_CMD=""
 
 
-    # MacOSX builds
-    - env:
-        - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS}"
-      os: osx
-      osx_image: xcode9.4
-      compiler: clang
-
+    # MacOSX builds - cirrus.yml also tests some MacOS builds including latest Xcode
 
     - env:
         - CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
-- 
2.20.1

Le mer. 27 nov. 2019 14:51, Alex Bennée <alex.bennee@linaro.org> a écrit :

> It's broken so it's no longer helping. The latest Xcode is covered by
> Cirrus.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

---
>  .travis.yml | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index c09b6a00143..445b0646c18 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -237,13 +237,7 @@ matrix:
>          - TEST_CMD=""
>
>
> -    # MacOSX builds
> -    - env:
> -        - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS}"
> -      os: osx
> -      osx_image: xcode9.4
> -      compiler: clang
> -
> +    # MacOSX builds - cirrus.yml also tests some MacOS builds including
> latest Xcode
>
>      - env:
>          -
> CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
> --
> 2.20.1
>
>
>

On 27/11/2019 14.24, Alex Bennée wrote:
> It's broken so it's no longer helping. The latest Xcode is covered by
> Cirrus.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  .travis.yml | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index c09b6a00143..445b0646c18 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -237,13 +237,7 @@ matrix:
>          - TEST_CMD=""
>  
>  
> -    # MacOSX builds
> -    - env:
> -        - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS}"
> -      os: osx
> -      osx_image: xcode9.4
> -      compiler: clang
> -
> +    # MacOSX builds - cirrus.yml also tests some MacOS builds including latest Xcode

Ack for removing xcode9.4, it's also constantly failing for me, which is
quite annoying.

Additionally, the xcode9.4 runs on macOS 10.13, which we do not
officially support anymore in QEMU (we only support the latest two
releases).

But maybe instead of removing the entry completely, you could replace it
with "xcode11.2" instead if that works? See:

 https://docs.travis-ci.com/user/reference/osx

  Thomas

On Fri, 29 Nov 2019 at 13:52, Thomas Huth <thuth@redhat.com> wrote:
>
> On 27/11/2019 14.24, Alex Bennée wrote:
> > It's broken so it's no longer helping. The latest Xcode is covered by
> > Cirrus.
> >
> > Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> > ---
> >  .travis.yml | 8 +-------
> >  1 file changed, 1 insertion(+), 7 deletions(-)
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index c09b6a00143..445b0646c18 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -237,13 +237,7 @@ matrix:
> >          - TEST_CMD=""
> >
> >
> > -    # MacOSX builds
> > -    - env:
> > -        - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS}"
> > -      os: osx
> > -      osx_image: xcode9.4
> > -      compiler: clang
> > -
> > +    # MacOSX builds - cirrus.yml also tests some MacOS builds including latest Xcode
>
> Ack for removing xcode9.4, it's also constantly failing for me, which is
> quite annoying.
>
> Additionally, the xcode9.4 runs on macOS 10.13, which we do not
> officially support anymore in QEMU (we only support the latest two
> releases).
>
> But maybe instead of removing the entry completely, you could replace it
> with "xcode11.2" instead if that works? See:

For purposes of 4.2 I've applied this patch-as is (since we're
going to get an rc4 anyway, and this is a simple change which
will stop travis complaining about the 4.2 stable branch in
future). We can look at xcode11.2 for 5.0.

thanks
-- PMM