qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3] .cirrus.yml: basic compile and test for Free


From: Thomas Huth
Subject: Re: [Qemu-devel] [PATCH v3] .cirrus.yml: basic compile and test for FreeBSD
Date: Mon, 21 Jan 2019 11:18:45 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1

On 2019-01-21 10:13, Thomas Huth wrote:
> On 2019-01-18 15:47, address@hidden wrote:
>> From: Ed Maste <address@hidden>
>>
>> Signed-off-by: Ed Maste <address@hidden>
>> ---
>> Changes since v2:
>>   * remove fontconfig and freetype2, noted by Thomas Huth
>>
>> After this patch is applied someone with appropriate permissions will
>> need to add the Cirrus-CI application in GitHub.
> 
> Maybe even more important: It also works for forked repositories (once
> you've enabled the Cirrus-CI application from the marketplace), so
> people can use this to test their branches before sending PULL requests,
> e.g.:
> 
>  https://cirrus-ci.com/github/huth/qemu
> 
> Thus this sounds like a good idea to me.
> 
> Reviewed-by: Thomas Huth <address@hidden>
> Tested-by: Thomas Huth <address@hidden>

Actually, scratch that Tested-by: My build timed out after 1 hour:

 https://cirrus-ci.com/task/5113243459649536

I think you need to split it up  / only build a subset of the targets
with the --target-list of the configure script, to make sure that the
task always finishes in < 1 h.
Something similar is also done in .travis.yml already, and I am also had
to split it up for my gitlab-ci.yml, see here:

 https://lists.gnu.org/archive/html/qemu-devel/2018-09/msg00226.html

Are you also sure that you can run with 8 CPUs here? ... I somehow doubt
that, otherwise the build should not take more than 1 h, I think.

 Thomas



reply via email to

[Prev in Thread] Current Thread [Next in Thread]