diff mbox series

[v2,1/4] scripts/ci/setup: ninja missing from build-environment

Message ID 20220914124153.61017-2-lucas.araujo@eldorado.org.br (mailing list archive)
State New, archived
Headers show
Series Patch series to set up a ppc64le CI | expand

Commit Message

Lucas Mateus Martins Araujo e Castro Sept. 14, 2022, 12:41 p.m. UTC
From: "Lucas Mateus Castro (alqotel)" <lucas.araujo@eldorado.org.br>

ninja-build is missing from the RHEL environment, so a system prepared
with that script would still fail to compile QEMU.
Tested on a Fedora 36

Signed-off-by: Lucas Mateus Castro (alqotel) <lucas.araujo@eldorado.org.br>
---
 scripts/ci/setup/build-environment.yml | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Huth Sept. 15, 2022, 3:54 p.m. UTC | #1
On 14/09/2022 13.41, Lucas Mateus Castro(alqotel) wrote:
> From: "Lucas Mateus Castro (alqotel)" <lucas.araujo@eldorado.org.br>
> 
> ninja-build is missing from the RHEL environment, so a system prepared
> with that script would still fail to compile QEMU.
> Tested on a Fedora 36
> 
> Signed-off-by: Lucas Mateus Castro (alqotel) <lucas.araujo@eldorado.org.br>
> ---
>   scripts/ci/setup/build-environment.yml | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/ci/setup/build-environment.yml b/scripts/ci/setup/build-environment.yml
> index 232525b91d..6df3e61d94 100644
> --- a/scripts/ci/setup/build-environment.yml
> +++ b/scripts/ci/setup/build-environment.yml
> @@ -150,6 +150,7 @@
>             - libepoxy-devel
>             - libgcrypt-devel
>             - lzo-devel
> +          - ninja-build
>             - make
>             - mesa-libEGL-devel
>             - nettle-devel

Looks like the list used to be sorted alphabetically, so maybe you should 
sort it in some lines later?

And I wonder why this hasn't been noticed before ... Cleber, was this in use 
after all?

  Thomas
Lucas Mateus Martins Araujo e Castro Sept. 21, 2022, 12:09 p.m. UTC | #2
On 15/09/2022 12:54, Thomas Huth wrote:

> On 14/09/2022 13.41, Lucas Mateus Castro(alqotel) wrote:
>> From: "Lucas Mateus Castro (alqotel)" <lucas.araujo@eldorado.org.br>
>>
>> ninja-build is missing from the RHEL environment, so a system prepared
>> with that script would still fail to compile QEMU.
>> Tested on a Fedora 36
>>
>> Signed-off-by: Lucas Mateus Castro (alqotel) 
>> <lucas.araujo@eldorado.org.br>
>> ---
>>   scripts/ci/setup/build-environment.yml | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/scripts/ci/setup/build-environment.yml 
>> b/scripts/ci/setup/build-environment.yml
>> index 232525b91d..6df3e61d94 100644
>> --- a/scripts/ci/setup/build-environment.yml
>> +++ b/scripts/ci/setup/build-environment.yml
>> @@ -150,6 +150,7 @@
>>             - libepoxy-devel
>>             - libgcrypt-devel
>>             - lzo-devel
>> +          - ninja-build
>>             - make
>>             - mesa-libEGL-devel
>>             - nettle-devel
> 
> Looks like the list used to be sorted alphabetically, so maybe you should
> sort it in some lines later?
D'oh, I'll send a v3 fixing that.
> 
> And I wonder why this hasn't been noticed before ... Cleber, was this in 
> use
> after all?
> 
>   Thomas
>
diff mbox series

Patch

diff --git a/scripts/ci/setup/build-environment.yml b/scripts/ci/setup/build-environment.yml
index 232525b91d..6df3e61d94 100644
--- a/scripts/ci/setup/build-environment.yml
+++ b/scripts/ci/setup/build-environment.yml
@@ -150,6 +150,7 @@ 
           - libepoxy-devel
           - libgcrypt-devel
           - lzo-devel
+          - ninja-build
           - make
           - mesa-libEGL-devel
           - nettle-devel