qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] .travis.yml: Install python3-tomli in all build jobs
@ 2024-06-24  9:48 Thomas Huth
  2024-06-24 10:09 ` Alex Bennée
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2024-06-24  9:48 UTC (permalink / raw)
  To: qemu-devel
  Cc: Alex Bennée, Philippe Mathieu-Daudé, Paolo Bonzini,
	Thomas Huth

Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
this package is a hard requirement for compiling QEMU, so install
it now in all Travis jobs, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 .travis.yml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index cef0308952..8fc1ae0cf2 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -106,6 +106,7 @@ jobs:
           - libvdeplug-dev
           - libvte-2.91-dev
           - ninja-build
+          - python3-tomli
           # Tests dependencies
           - genisoimage
       env:
@@ -141,6 +142,7 @@ jobs:
           - libvdeplug-dev
           - libvte-2.91-dev
           - ninja-build
+          - python3-tomli
           # Tests dependencies
           - genisoimage
       env:
@@ -175,6 +177,7 @@ jobs:
           - libvdeplug-dev
           - libvte-2.91-dev
           - ninja-build
+          - python3-tomli
           # Tests dependencies
           - genisoimage
       env:
@@ -215,6 +218,7 @@ jobs:
           - libzstd-dev
           - nettle-dev
           - ninja-build
+          - python3-tomli
           # Tests dependencies
           - genisoimage
       env:
@@ -231,6 +235,7 @@ jobs:
           - ninja-build
           - flex
           - bison
+          - python3-tomli
       env:
         - TEST_CMD="make check check-tcg V=1"
         - CONFIG="--disable-containers --disable-system"
@@ -263,6 +268,7 @@ jobs:
           - libvdeplug-dev
           - libvte-2.91-dev
           - ninja-build
+          - python3-tomli
       env:
         - TEST_CMD="make check-unit"
         - CONFIG="--disable-containers --disable-tcg --enable-kvm --disable-tools
-- 
2.45.2



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] .travis.yml: Install python3-tomli in all build jobs
  2024-06-24  9:48 [PATCH] .travis.yml: Install python3-tomli in all build jobs Thomas Huth
@ 2024-06-24 10:09 ` Alex Bennée
  2024-06-24 10:31   ` Thomas Huth
  0 siblings, 1 reply; 4+ messages in thread
From: Alex Bennée @ 2024-06-24 10:09 UTC (permalink / raw)
  To: Thomas Huth; +Cc: qemu-devel, Philippe Mathieu-Daudé, Paolo Bonzini

Thomas Huth <thuth@redhat.com> writes:

> Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
> this package is a hard requirement for compiling QEMU, so install
> it now in all Travis jobs, too.

AFAICT the only repo currently running these tests is your github
mirror:

  https://app.travis-ci.com/github/huth/qemu/builds?serverType=git

Because both the official github mirror and the gitlab project haven't
run anything for a while:

  https://app.travis-ci.com/gitlab/qemu-project/qemu/branches
  https://app.travis-ci.com/github/qemu/qemu/branches

I've never been able to get anything to build since they transitioned to
TravisCI.com

>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  .travis.yml | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/.travis.yml b/.travis.yml
> index cef0308952..8fc1ae0cf2 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -106,6 +106,7 @@ jobs:
>            - libvdeplug-dev
>            - libvte-2.91-dev
>            - ninja-build
> +          - python3-tomli
>            # Tests dependencies
>            - genisoimage
>        env:
> @@ -141,6 +142,7 @@ jobs:
>            - libvdeplug-dev
>            - libvte-2.91-dev
>            - ninja-build
> +          - python3-tomli
>            # Tests dependencies
>            - genisoimage
>        env:
> @@ -175,6 +177,7 @@ jobs:
>            - libvdeplug-dev
>            - libvte-2.91-dev
>            - ninja-build
> +          - python3-tomli
>            # Tests dependencies
>            - genisoimage
>        env:
> @@ -215,6 +218,7 @@ jobs:
>            - libzstd-dev
>            - nettle-dev
>            - ninja-build
> +          - python3-tomli
>            # Tests dependencies
>            - genisoimage
>        env:
> @@ -231,6 +235,7 @@ jobs:
>            - ninja-build
>            - flex
>            - bison
> +          - python3-tomli
>        env:
>          - TEST_CMD="make check check-tcg V=1"
>          - CONFIG="--disable-containers --disable-system"
> @@ -263,6 +268,7 @@ jobs:
>            - libvdeplug-dev
>            - libvte-2.91-dev
>            - ninja-build
> +          - python3-tomli
>        env:
>          - TEST_CMD="make check-unit"
>          - CONFIG="--disable-containers --disable-tcg --enable-kvm --disable-tools

FWIW the changes themselves look fine:

Acked-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] .travis.yml: Install python3-tomli in all build jobs
  2024-06-24 10:09 ` Alex Bennée
@ 2024-06-24 10:31   ` Thomas Huth
  2024-08-20  8:51     ` Stefan Weil via
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2024-06-24 10:31 UTC (permalink / raw)
  To: Alex Bennée; +Cc: qemu-devel, Philippe Mathieu-Daudé, Paolo Bonzini

On 24/06/2024 12.09, Alex Bennée wrote:
> Thomas Huth <thuth@redhat.com> writes:
> 
>> Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
>> this package is a hard requirement for compiling QEMU, so install
>> it now in all Travis jobs, too.
> 
> AFAICT the only repo currently running these tests is your github
> mirror:
> 
>    https://app.travis-ci.com/github/huth/qemu/builds?serverType=git
> 
> Because both the official github mirror and the gitlab project haven't
> run anything for a while:
> 
>    https://app.travis-ci.com/gitlab/qemu-project/qemu/branches
>    https://app.travis-ci.com/github/qemu/qemu/branches

AFAIK the gitlab integration broke completely at one point in time. But as 
you can see with my account, it still works fine for the github repo - I 
just need to write a short mail to the travis support once a year to ask 
them to extend the open source credits for my repo. So apart from that minor 
effort, it's a very convenient way to test QEMU on non-x86 hosts.

> FWIW the changes themselves look fine:
> 
> Acked-by: Alex Bennée <alex.bennee@linaro.org>

Thanks!

  Thomas




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] .travis.yml: Install python3-tomli in all build jobs
  2024-06-24 10:31   ` Thomas Huth
@ 2024-08-20  8:51     ` Stefan Weil via
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Weil via @ 2024-08-20  8:51 UTC (permalink / raw)
  To: Thomas Huth, Alex Bennée
  Cc: qemu-devel, Philippe Mathieu-Daudé, Paolo Bonzini

Am 24.06.24 um 12:31 schrieb Thomas Huth:
> On 24/06/2024 12.09, Alex Bennée wrote:
>> Thomas Huth <thuth@redhat.com> writes:
>>
>>> Since commit 1f97715c83 ('Revert "python: use vendored tomli"')
>>> this package is a hard requirement for compiling QEMU, so install
>>> it now in all Travis jobs, too.
>>
>> AFAICT the only repo currently running these tests is your github
>> mirror:
>>
>>    https://app.travis-ci.com/github/huth/qemu/builds?serverType=git
>>
>> Because both the official github mirror and the gitlab project haven't
>> run anything for a while:
>>
>>    https://app.travis-ci.com/gitlab/qemu-project/qemu/branches
>>    https://app.travis-ci.com/github/qemu/qemu/branches
> 
> AFAIK the gitlab integration broke completely at one point in time. But 
> as you can see with my account, it still works fine for the github repo 
> - I just need to write a short mail to the travis support once a year to 
> ask them to extend the open source credits for my repo. So apart from 
> that minor effort, it's a very convenient way to test QEMU on non-x86 
> hosts.


https://app.travis-ci.com/github/qemu/qemu/requests shows recent 
requests. But all requests are refused: "Owner qemu is not on a new 
pricing". So whoever is "qemu", he/she might do the same as you did and 
write to the Travis support.

And https://travis-ci.org/qemu/qemu could be replaced by 
https://app.travis-ci.com/github/qemu/qemu in MAINTAINERS.

Stefan


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-08-20  8:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-24  9:48 [PATCH] .travis.yml: Install python3-tomli in all build jobs Thomas Huth
2024-06-24 10:09 ` Alex Bennée
2024-06-24 10:31   ` Thomas Huth
2024-08-20  8:51     ` Stefan Weil via

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).