diff mbox series

.travis.yml: split MacOSX builds and reduce target list

Message ID 20181008152059.17503-1-alex.bennee@linaro.org (mailing list archive)
State New, archived
Headers show
Series .travis.yml: split MacOSX builds and reduce target list | expand

Commit Message

Alex Bennée Oct. 8, 2018, 3:20 p.m. UTC
We have reached the point where the MacOSX build was regularly timing
out. So as before I've reduced the target list to "major"
architectures to try and bring the build time down. I've added an
additional MacOSX build with the latest XCode with a minimal list of
"most likely" targets on MacOS.

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

Comments

Philippe Mathieu-Daudé Oct. 8, 2018, 4:40 p.m. UTC | #1
Hi Alex,

On Mon, Oct 8, 2018 at 5:21 PM Alex Bennée <alex.bennee@linaro.org> wrote:
> We have reached the point where the MacOSX build was regularly timing
> out. So as before I've reduced the target list to "major"
> architectures to try and bring the build time down. I've added an
> additional MacOSX build with the latest XCode with a minimal list of
> "most likely" targets on MacOS.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  .travis.yml | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 95be6ec59f..13a09facd3 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -107,8 +107,14 @@ matrix:
>      - env: CONFIG="--disable-tcg"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG=""
> +    # MacOSX builds
> +    - env: CONFIG="--target-list=aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>        os: osx
> +      osx_image: xcode9.4
> +      compiler: clang
> +    - env: CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
> +      os: osx
> +      osx_image: xcode10

I'd also add another entry with --enable-user.

>        compiler: clang
>      # Python builds
>      - env: CONFIG="--target-list=x86_64-softmmu"
> --
> 2.17.1
>
Alex Bennée Oct. 9, 2018, 12:24 p.m. UTC | #2
Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> Hi Alex,
>
> On Mon, Oct 8, 2018 at 5:21 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>> We have reached the point where the MacOSX build was regularly timing
>> out. So as before I've reduced the target list to "major"
>> architectures to try and bring the build time down. I've added an
>> additional MacOSX build with the latest XCode with a minimal list of
>> "most likely" targets on MacOS.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>>  .travis.yml | 8 +++++++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index 95be6ec59f..13a09facd3 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -107,8 +107,14 @@ matrix:
>>      - env: CONFIG="--disable-tcg"
>>             TEST_CMD=""
>>        compiler: gcc
>> -    - env: CONFIG=""
>> +    # MacOSX builds
>> +    - env: CONFIG="--target-list=aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>>        os: osx
>> +      osx_image: xcode9.4
>> +      compiler: clang
>> +    - env: CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
>> +      os: osx
>> +      osx_image: xcode10
>
> I'd also add another entry with --enable-user.

Ohh, does MacOSX use the bsd-user machinery?

>
>>        compiler: clang
>>      # Python builds
>>      - env: CONFIG="--target-list=x86_64-softmmu"
>> --
>> 2.17.1
>>


--
Alex Bennée
Peter Maydell Oct. 9, 2018, 12:29 p.m. UTC | #3
On 9 October 2018 at 13:24, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Philippe Mathieu-Daudé <f4bug@amsat.org> writes:
>> I'd also add another entry with --enable-user.
>
> Ohh, does MacOSX use the bsd-user machinery?

No. It used to have its own darwin-user, but we removed
that some years ago because it was unmaintained and broken.

thanks
-- PMM
diff mbox series

Patch

diff --git a/.travis.yml b/.travis.yml
index 95be6ec59f..13a09facd3 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -107,8 +107,14 @@  matrix:
     - env: CONFIG="--disable-tcg"
            TEST_CMD=""
       compiler: gcc
-    - env: CONFIG=""
+    # MacOSX builds
+    - env: CONFIG="--target-list=aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
       os: osx
+      osx_image: xcode9.4
+      compiler: clang
+    - env: CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
+      os: osx
+      osx_image: xcode10
       compiler: clang
     # Python builds
     - env: CONFIG="--target-list=x86_64-softmmu"