[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