[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Maxim Cournoyer]

Hi,

This was fully built for x86_64 as some insurance that it didn't
introduce any problem:
https://ci.guix.gnu.org/jobset/meson-parallel-tests

It's ready to me merged.

-- 
Thanks,
Maxim

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 388 bytes --]

Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

> Hi,
>
> This was fully built for x86_64 as some insurance that it didn't
> introduce any problem:
> https://ci.guix.gnu.org/jobset/meson-parallel-tests
>
> It's ready to me merged.

The branch name in the issue title doesn't quite match the actual name
of the branch. Could you retitle the issue to make it match up?

Thanks,

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Maxim Cournoyer]

Hi Chris,

Christopher Baines <mail@cbaines.net> writes:

> Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
>
>> Hi,
>>
>> This was fully built for x86_64 as some insurance that it didn't
>> introduce any problem:
>> https://ci.guix.gnu.org/jobset/meson-parallel-tests
>>
>> It's ready to me merged.
>
> The branch name in the issue title doesn't quite match the actual name
> of the branch. Could you retitle the issue to make it match up?

Thanks for the heads-up.  I believe that's now fixed.

PS: Your email went to my spam folder, for some reasons.

-- 
Thanks,
Maxim

[bug#73765] Rebased

[Andreas Enge]

Hello,

this is the next branch in the QA queue. I have rebased it on master;
hopefully it will be picked up by QA. Or you could have it built again
on CI, as QA has trouble keeping up recently.

Andreas

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Ludovic Courtès]

Hi,

Andreas Enge <andreas@enge.fr> skribis:

> this is the next branch in the QA queue. I have rebased it on master;
> hopefully it will be picked up by QA. Or you could have it built again
> on CI, as QA has trouble keeping up recently.

It currently fails to evaluate (commit 0363f28):

--8<---------------cut here---------------start------------->8---
   376:14  3 (arguments #<package mesa@24.2.5 gnu/packages/gl.scm:30?>)
In ice-9/boot-9.scm:
  1685:16  2 (raise-exception _ #:continuable? _)
  1780:13  1 (_ #<&compound-exception components: (#<&undefined-vari?>)
In unknown file:
           0 (backtrace #<undefined>)

(exception unbound-variable (value #f) (value "Unbound variable: ~S") (value (meson-1.5)) (value #f))
--8<---------------cut here---------------end--------------->8---

(From <https://ci.guix.gnu.org/eval/1837912/log/raw>.)

Ludo’.

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Andreas Enge]

Am Fri, Nov 22, 2024 at 03:45:26PM +0100 schrieb Ludovic Courtès:
> It currently fails to evaluate (commit 0363f28):
> --8<---------------cut here---------------start------------->8---
>    376:14  3 (arguments #<package mesa@24.2.5 gnu/packages/gl.scm:30?>)
> In ice-9/boot-9.scm:
>   1685:16  2 (raise-exception _ #:continuable? _)
>   1780:13  1 (_ #<&compound-exception components: (#<&undefined-vari?>)
> In unknown file:
>            0 (backtrace #<undefined>)
> 
> (exception unbound-variable (value #f) (value "Unbound variable: ~S") (value (meson-1.5)) (value #f))
> --8<---------------cut here---------------end--------------->8---

That is strange - I did a "make" in my branch before pushing.
How can this be explained?

Andreas

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Andreas Enge]

Am Fri, Nov 22, 2024 at 04:03:11PM +0100 schrieb Andreas Enge:
> That is strange - I did a "make" in my branch before pushing.
> How can this be explained?

"guix build mesa" fails due to a missing (and now superfluous) variable.

I have pushed a fix, which I could not test due to lack of space.
If it turns out not to work on CI, and since we rebase and thus force
push branches anyway, please feel free to remove the commit again.

Andreas

[bug#73765] Results

[Andreas Enge]

The branch has been built on CI, with 36 new failures:
   https://ci.guix.gnu.org/eval/1838237?status=newly-failed

Maybe something could be done about them before merging?

Andreas

[bug#73765] Request for merging "meson-enable-parallel-tests" branch

[Ludovic Courtès]

Hi!

Andreas Enge <andreas@enge.fr> skribis:

> The branch has been built on CI, with 36 new failures:
>    https://ci.guix.gnu.org/eval/1838237?status=newly-failed
>
> Maybe something could be done about them before merging?

BTW, I noticed that the jobset was configured to build for x86_64-linux
only (with 95% success currently¹).  I’ve added i686-linux now, though
it’ll only take effect on the next evaluation.

Feel free to add aarch64-linux as well once you know this is likely the
last evaluation before merge.

Ludo’.

¹ https://ci.guix.gnu.org/jobset/meson-parallel-tests

bug#73765: Request for merging "meson-enable-parallel-tests" branch

[Andreas Enge]

Hello!

I have gone back to the original merge request and noticed, following
Maxim's remark, that indeed no build failures occurred. Updating the meson
version resulted in build failures, but which looked more like random test
failures in somewhat brittle packages that fail on and off.

So I have taken the liberty and rebased on master and pushed.

I had forgotten that the branch was not yet built on i686, sorry.

Apart from that, I think it is good to clear the space for the following
branches waiting in line.

Andreas