qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: "Alex Bennée" <alex.bennee@linaro.org>, qemu-devel@nongnu.org
Cc: Fam Zheng <fam@euphon.net>, qemu-arm@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v1 7/7] shippable: re-enable the windows cross builds
Date: Sat, 13 Jul 2019 17:01:19 +0200	[thread overview]
Message-ID: <18c04017-7908-c4d1-5414-cae79ba82fc3@redhat.com> (raw)
In-Reply-To: <20190712111849.9006-8-alex.bennee@linaro.org>

On 7/12/19 1:18 PM, Alex Bennée wrote:
> The pkg.mxe.cc repo has been restored.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  .shippable.yml | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/.shippable.yml b/.shippable.yml
> index f2ffef21d11..f74a3de3ffd 100644
> --- a/.shippable.yml
> +++ b/.shippable.yml
> @@ -7,11 +7,10 @@ env:
>    matrix:
>      - IMAGE=debian-amd64
>        TARGET_LIST=x86_64-softmmu,x86_64-linux-user
> -    # currently disabled as the mxe.cc repos are down
> -    # - IMAGE=debian-win32-cross
> -    #   TARGET_LIST=arm-softmmu,i386-softmmu,lm32-softmmu
> -    # - IMAGE=debian-win64-cross
> -    #   TARGET_LIST=aarch64-softmmu,sparc64-softmmu,x86_64-softmmu
> +    - IMAGE=debian-win32-cross
> +      TARGET_LIST=arm-softmmu,i386-softmmu,lm32-softmmu
> +    - IMAGE=debian-win64-cross
> +      TARGET_LIST=aarch64-softmmu,sparc64-softmmu,x86_64-softmmu
>      - IMAGE=debian-armel-cross
>        TARGET_LIST=arm-softmmu,arm-linux-user,armeb-linux-user
>      - IMAGE=debian-armhf-cross
> 

Builds now take ~25min longer. The win32 build is definitively helpful
and should catch most of the errors the win64 could find. I suggest we
build the docker image but not cross-compile on win64. Anyway this can
be improved later.

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


      reply	other threads:[~2019-07-13 15:02 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-12 11:18 [Qemu-devel] [PATCH for 4.1? v1 0/7] testing/next (docker, win-cross) Alex Bennée
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 1/7] tests/docker: add test-misc for building tools & docs Alex Bennée
2019-07-13 15:48   ` Philippe Mathieu-Daudé
2019-07-15 11:08     ` Alex Bennée
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 2/7] tests/docker: Install Sphinx in the Ubuntu images Alex Bennée
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 3/7] tests/docker: Install Sphinx in the Fedora image Alex Bennée
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 4/7] tests/docker: Install Ubuntu images noninteractively Alex Bennée
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 5/7] tests/migration-test: don't spam the logs when we fail Alex Bennée
2019-07-12 11:22   ` Laurent Vivier
2019-07-12 13:35     ` Alex Bennée
2019-07-12 13:49       ` Laurent Vivier
2019-07-12 11:23   ` Thomas Huth
2019-07-12 11:42   ` Dr. David Alan Gilbert
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 6/7] tests/dockerfiles: update the win cross builds to stretch Alex Bennée
2019-07-13 14:56   ` Philippe Mathieu-Daudé
2019-07-12 11:18 ` [Qemu-devel] [PATCH v1 7/7] shippable: re-enable the windows cross builds Alex Bennée
2019-07-13 15:01   ` Philippe Mathieu-Daudé [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18c04017-7908-c4d1-5414-cae79ba82fc3@redhat.com \
    --to=philmd@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=fam@euphon.net \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).