unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: bug#74204: [bug#74112] Guix is not reproducible
       [not found]     ` <87ed3pzora.wl-hako@ultrarare.space>
@ 2024-11-07 17:54       ` Simon Tournier
  2024-11-10  9:11         ` Maxim Cournoyer
  0 siblings, 1 reply; 4+ messages in thread
From: Simon Tournier @ 2024-11-07 17:54 UTC (permalink / raw)
  To: Hilton Chain, Jakob Kirsch; +Cc: Guix Devel, 74112-done, 74204-done

Hi,

On Tue, 05 Nov 2024 at 23:25, Hilton Chain via Bug reports for GNU Guix <bug-guix@gnu.org> wrote:

>> I can confirm the reproducibility issue.
>>
>> I have two x86_64-linux machines building guix to verify the fix, I'll apply
>> your patch once they produce matching outputs.
>
> Took me quite a while to build 5 rounds. :)
>
> --8<---------------cut here---------------start------------->8---
> $ guix hash --serializer=nar /gnu/store/fs7x07jfn7igpkwv3alrs9by21q70y13-guix-1.4.0-26.5ab3c4c
> 0kh87wb4qn97kwzrf4igal71cjvv143j6jr2y3dwfzcy1madj1ll
> --8<---------------cut here---------------end--------------->8---
>
> Applied #74112 as 4c56d0cccdc44e12484b26332715f54768738c5f, thanks!

Maybe I am missing something.  To my knowledge, .go files produced by
Guile are not always reproducible, see bug#20272 [1].  And, from my
understanding, Guix cannot be reproducible until this bug had been
fixed.  Therefore, I am not convinced that this patch is worth under
this frame considering the build-time penalty it brings.

That’s said, maybe it’s better than nothing and the package ’guix’ is
barely built after all.  I do not know.

What people think?

Cheers,
simon

1: https://issues.guix.gnu.org/issue/20272


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

* Re: bug#74204: [bug#74112] Guix is not reproducible
  2024-11-07 17:54       ` bug#74204: [bug#74112] Guix is not reproducible Simon Tournier
@ 2024-11-10  9:11         ` Maxim Cournoyer
  2024-11-14 10:01           ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Maxim Cournoyer @ 2024-11-10  9:11 UTC (permalink / raw)
  To: Simon Tournier
  Cc: Hilton Chain, Jakob Kirsch, Guix Devel, 74112-done, 74204-done

Hi Simon,

Simon Tournier <zimon.toutoune@gmail.com> writes:

> Hi,
>
> On Tue, 05 Nov 2024 at 23:25, Hilton Chain via Bug reports for GNU Guix <bug-guix@gnu.org> wrote:
>
>>> I can confirm the reproducibility issue.
>>>
>>> I have two x86_64-linux machines building guix to verify the fix, I'll apply
>>> your patch once they produce matching outputs.
>>
>> Took me quite a while to build 5 rounds. :)
>>
>> --8<---------------cut here---------------start------------->8---
>> $ guix hash --serializer=nar /gnu/store/fs7x07jfn7igpkwv3alrs9by21q70y13-guix-1.4.0-26.5ab3c4c
>> 0kh87wb4qn97kwzrf4igal71cjvv143j6jr2y3dwfzcy1madj1ll
>> --8<---------------cut here---------------end--------------->8---
>>
>> Applied #74112 as 4c56d0cccdc44e12484b26332715f54768738c5f, thanks!
>
> Maybe I am missing something.  To my knowledge, .go files produced by
> Guile are not always reproducible, see bug#20272 [1].  And, from my
> understanding, Guix cannot be reproducible until this bug had been
> fixed.  Therefore, I am not convinced that this patch is worth under
> this frame considering the build-time penalty it brings.
>
> That’s said, maybe it’s better than nothing and the package ’guix’ is
> barely built after all.  I do not know.
>
> What people think?

Perhaps we should set the default parallel-build? to #f in the
guile-build-system at least in the meantime, with a prominent comment as
to why and a reference to the upstream issue?  Many Guile packages use
the gnu-build-system so that wouldn't cover all of them like
'guix'... I'm not sure.

It'd be nicer to fix the underlying guile issue (again?), but I doubt
many people are up to this.

-- 
Thanks,
Maxim


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

* Re: bug#74204: [bug#74112] Guix is not reproducible
  2024-11-10  9:11         ` Maxim Cournoyer
@ 2024-11-14 10:01           ` Ludovic Courtès
  2024-12-18  9:19             ` [bug#74112] " Maxim Cournoyer
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2024-11-14 10:01 UTC (permalink / raw)
  To: Maxim Cournoyer
  Cc: Simon Tournier, Hilton Chain, Jakob Kirsch, Guix Devel,
	74112-done, 74204-done

Hi,

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

> Simon Tournier <zimon.toutoune@gmail.com> writes:

[...]

>> Maybe I am missing something.  To my knowledge, .go files produced by
>> Guile are not always reproducible, see bug#20272 [1].  And, from my
>> understanding, Guix cannot be reproducible until this bug had been
>> fixed.  Therefore, I am not convinced that this patch is worth under
>> this frame considering the build-time penalty it brings.
>>
>> That’s said, maybe it’s better than nothing and the package ’guix’ is
>> barely built after all.  I do not know.
>>
>> What people think?
>
> Perhaps we should set the default parallel-build? to #f in the
> guile-build-system at least in the meantime, with a prominent comment as
> to why and a reference to the upstream issue?  Many Guile packages use
> the gnu-build-system so that wouldn't cover all of them like
> 'guix'... I'm not sure.

Sounds good to me: packages using ‘guile-build-system’ are usually
relatively small so the impact is negligible.

‘guix’ is a little different because it takes so much time to build
sequentially…

Ludo’.


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

* Re: [bug#74112] bug#74204: [bug#74112] Guix is not reproducible
  2024-11-14 10:01           ` Ludovic Courtès
@ 2024-12-18  9:19             ` Maxim Cournoyer
  0 siblings, 0 replies; 4+ messages in thread
From: Maxim Cournoyer @ 2024-12-18  9:19 UTC (permalink / raw)
  To: Ludovic Courtès
  Cc: 74112-done, Jakob Kirsch, Simon Tournier, Hilton Chain,
	Guix Devel

Hi,

Ludovic Courtès <ludo@gnu.org> writes:

[...]

>> Perhaps we should set the default parallel-build? to #f in the
>> guile-build-system at least in the meantime, with a prominent comment as
>> to why and a reference to the upstream issue?  Many Guile packages use
>> the gnu-build-system so that wouldn't cover all of them like
>> 'guix'... I'm not sure.
>
> Sounds good to me: packages using ‘guile-build-system’ are usually
> relatively small so the impact is negligible.

That's now tackled in #74909.

-- 
Thanks,
Maxim


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

end of thread, other threads:[~2024-12-18  9:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <ZykUqu7t4WuUF9kj@kernelpanicroom>
     [not found] ` <ZyJ1wSX6cPIdnLpN@kernelpanicroom>
     [not found]   ` <87fro6yntj.wl-hako@ultrarare.space>
     [not found]     ` <87ed3pzora.wl-hako@ultrarare.space>
2024-11-07 17:54       ` bug#74204: [bug#74112] Guix is not reproducible Simon Tournier
2024-11-10  9:11         ` Maxim Cournoyer
2024-11-14 10:01           ` Ludovic Courtès
2024-12-18  9:19             ` [bug#74112] " Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).