From: "Alex Bennée" <alex.bennee@linaro.org>
To: luoyonggang@gmail.com
Cc: "Thomas Huth" <thuth@redhat.com>,
"QEMU Trivial" <qemu-trivial@nongnu.org>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
qemu-devel@nongnu.org
Subject: Re: [PATCH v5 1/2] cirrus: Fixing and speedup the msys2/mingw CI
Date: Wed, 07 Oct 2020 19:53:55 +0100 [thread overview]
Message-ID: <87d01tzx30.fsf@linaro.org> (raw)
In-Reply-To: <CAE2XoE89z44nA=N-U_r-C8RA5Vn+C3gqABt85PLFPcSg4ET5gw@mail.gmail.com>
罗勇刚(Yonggang Luo) <luoyonggang@gmail.com> writes:
> On Wed, Oct 7, 2020 at 11:12 PM Thomas Huth <thuth@redhat.com> wrote:
>>
>> On 07/10/2020 16.52, Yonggang Luo wrote:
>> > Use cache of cirrus caching msys2
>> > The install of msys2 are refer to https://github.com/msys2/setup-msys2
>> > The first time install msys2 would be time consuming, so increase
> timeout_in to 90m
>> > according to https://cirrus-ci.org/faq/#instance-timed-out
>> >
>> > Apply patch of
> https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg00072.html
>> >
>> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
>> > ---
>> > .cirrus.yml | 110 +++++++++++++++++++++++++++++++---------------------
>> > 1 file changed, 66 insertions(+), 44 deletions(-)
>> >
>> > diff --git a/.cirrus.yml b/.cirrus.yml
>> > index d58782ce67..a1145d0366 100644
>> > --- a/.cirrus.yml
>> > +++ b/.cirrus.yml
>> > @@ -46,64 +46,86 @@ macos_xcode_task:
>> > - gmake check
>> >
>> > windows_msys2_task:
>> > + timeout_in: 90m
>> > windows_container:
>> > - image: cirrusci/windowsservercore:cmake
>> > + image: cirrusci/windowsservercore:2019
>> > os_version: 2019
>> > cpu: 8
>> > memory: 8G
>> > env:
>> > + CIRRUS_SHELL: powershell
>> > MSYS: winsymlinks:nativestrict
>> > MSYSTEM: MINGW64
>> > CHERE_INVOKING: 1
>> > printenv_script:
>> > - - C:\tools\msys64\usr\bin\bash.exe -lc 'printenv'
>> > + - choco install -y --no-progress 7zip
>>
>> Since you've now removed the 'printenv' line, I think the section should
>> maybe not be called "printenv_script" anymore, should it?
> May I name it install_7zip?
> Cause that's a deps need installed first.
I renamed it setup_script: when I merged it.
>>
>> Thomas
>>
--
Alex Bennée
WARNING: multiple messages have this Message-ID (diff)
From: "Alex Bennée" <alex.bennee@linaro.org>
To: luoyonggang@gmail.com
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"Thomas Huth" <thuth@redhat.com>,
"QEMU Trivial" <qemu-trivial@nongnu.org>,
qemu-devel@nongnu.org, "Philippe Mathieu-Daudé" <f4bug@amsat.org>,
"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: [PATCH v5 1/2] cirrus: Fixing and speedup the msys2/mingw CI
Date: Wed, 07 Oct 2020 19:53:55 +0100 [thread overview]
Message-ID: <87d01tzx30.fsf@linaro.org> (raw)
In-Reply-To: <CAE2XoE89z44nA=N-U_r-C8RA5Vn+C3gqABt85PLFPcSg4ET5gw@mail.gmail.com>
罗勇刚(Yonggang Luo) <luoyonggang@gmail.com> writes:
> On Wed, Oct 7, 2020 at 11:12 PM Thomas Huth <thuth@redhat.com> wrote:
>>
>> On 07/10/2020 16.52, Yonggang Luo wrote:
>> > Use cache of cirrus caching msys2
>> > The install of msys2 are refer to https://github.com/msys2/setup-msys2
>> > The first time install msys2 would be time consuming, so increase
> timeout_in to 90m
>> > according to https://cirrus-ci.org/faq/#instance-timed-out
>> >
>> > Apply patch of
> https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg00072.html
>> >
>> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
>> > ---
>> > .cirrus.yml | 110 +++++++++++++++++++++++++++++++---------------------
>> > 1 file changed, 66 insertions(+), 44 deletions(-)
>> >
>> > diff --git a/.cirrus.yml b/.cirrus.yml
>> > index d58782ce67..a1145d0366 100644
>> > --- a/.cirrus.yml
>> > +++ b/.cirrus.yml
>> > @@ -46,64 +46,86 @@ macos_xcode_task:
>> > - gmake check
>> >
>> > windows_msys2_task:
>> > + timeout_in: 90m
>> > windows_container:
>> > - image: cirrusci/windowsservercore:cmake
>> > + image: cirrusci/windowsservercore:2019
>> > os_version: 2019
>> > cpu: 8
>> > memory: 8G
>> > env:
>> > + CIRRUS_SHELL: powershell
>> > MSYS: winsymlinks:nativestrict
>> > MSYSTEM: MINGW64
>> > CHERE_INVOKING: 1
>> > printenv_script:
>> > - - C:\tools\msys64\usr\bin\bash.exe -lc 'printenv'
>> > + - choco install -y --no-progress 7zip
>>
>> Since you've now removed the 'printenv' line, I think the section should
>> maybe not be called "printenv_script" anymore, should it?
> May I name it install_7zip?
> Cause that's a deps need installed first.
I renamed it setup_script: when I merged it.
>>
>> Thomas
>>
--
Alex Bennée
next prev parent reply other threads:[~2020-10-07 18:54 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-07 14:52 [PATCH v5 0/2] Improve cirrus msys2 Yonggang Luo
2020-10-07 14:52 ` Yonggang Luo
2020-10-07 14:52 ` [PATCH v5 1/2] cirrus: Fixing and speedup the msys2/mingw CI Yonggang Luo
2020-10-07 14:52 ` Yonggang Luo
2020-10-07 15:12 ` Thomas Huth
2020-10-07 15:12 ` Thomas Huth
2020-10-07 15:54 ` 罗勇刚(Yonggang Luo)
2020-10-07 15:54 ` 罗勇刚(Yonggang Luo)
2020-10-07 18:53 ` Alex Bennée [this message]
2020-10-07 18:53 ` Alex Bennée
2020-10-07 14:53 ` [PATCH v5 2/2] cirrus: msys2/mingw speed is up, add excluded target back Yonggang Luo
2020-10-07 14:53 ` Yonggang Luo
2020-10-07 14:56 ` [PATCH v5 0/2] Improve cirrus msys2 罗勇刚(Yonggang Luo)
2020-10-07 14:56 ` 罗勇刚(Yonggang Luo)
2020-10-07 15:58 ` Alex Bennée
2020-10-07 15:58 ` Alex Bennée
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=87d01tzx30.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=f4bug@amsat.org \
--cc=luoyonggang@gmail.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@nongnu.org \
--cc=thuth@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.