Subject: bug#40908: closed (Re: [bug#40908] [PATCH core-updates 5/5] gnu: make-bootstrap: Do not export %GUILE-STATIC-STRIPPED.)

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 01 May 2020 20:17:02 +0000

Your bug report

#40908: [PATCH core-updates 0/5] Use Guile 3.0 in the initrd

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
40908: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=40908
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug#40908] [PATCH core-updates 5/5] gnu: make-bootstrap: Do not export %GUILE-STATIC-STRIPPED.
From: Marius Bakke <[email-masked]>
To: Ludovic Courtès <[email-masked]>
Cc: [email-masked]
Date: Fri, 01 May 2020 22:16:14 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Ludovic Courtès <[email-masked]> writes:

> Hi,
>
> Marius Bakke <[email-masked]> skribis:
>
>> * gnu/packages/make-bootstrap.scm (%guile-static-stripped): Remove export.
>> ---
>>  gnu/packages/make-bootstrap.scm | 4 +---
>>  1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/gnu/packages/make-bootstrap.scm b/gnu/packages/make-bootstrap.scm
>> index 443e778824..981c7aa24e 100644
>> --- a/gnu/packages/make-bootstrap.scm
>> +++ b/gnu/packages/make-bootstrap.scm
>> @@ -56,9 +56,7 @@
>>              %guile-bootstrap-tarball
>>              %mescc-tools-bootstrap-tarball
>>              %mes-bootstrap-tarball
>> -            %bootstrap-tarballs
>> -
>> -            %guile-static-stripped))
>
> Are you sure it’s unused?
>
> Actually, what about simply adding ‘%guile-3.0-static-stripped’ right in
> this file?  I think it would work, right?  And we could also keep
> ‘make-guile-static’ private.

Right, that is better.  Done!

> Apart from this, the patch series LGTM!  Pretty exciting to have a JIT
> in the initrd.  ;-)

Heh, my main motivation for these patches was to remove scary warnings
about incompatible bytecode during boot, but we also boot faster as a
side effect!  :-)

Patches pushed!

I think we are pretty close to actually merging this branch now..!

Subject: [PATCH core-updates 0/5] Use Guile 3.0 in the initrd
From: Marius Bakke <[email-masked]>
To: [email-masked]
Date: Mon, 27 Apr 2020 20:20:27 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Guix,

These patches changes the initrd to use Guile 3.0.  By default it uses
%GUILE-STATIC-STRIPPED from (gnu packages make-bootstrap), which on the
core-updates branch refers to Guile 2.0.

It is suboptimal to define this new static Guile variant straight in
(gnu system linux-initrd), but adding it to (gnu packages guile) would
cause a cyclic module reference with (gnu packages make-bootstrap).

WDYT?

Marius Bakke (5):
  gnu: %guile-static: Rewrite in terms of 'make-guile-static'.
  gnu: %guile-static-stripped: Rewrite in terms of
    'make-guile-static-stripped'.
  gnu: make-bootstrap: Export MAKE-GUILE-STATIC and
    MAKE-GUILE-STATIC-STRIPPED.
  linux-initrd: Use Guile 3.0.
  gnu: make-bootstrap: Do not export %GUILE-STATIC-STRIPPED.

 gnu/local.mk                                  |   7 +-
 gnu/packages/make-bootstrap.scm               | 161 ++++----
 .../patches/guile-2.2-default-utf8.patch      |  82 ++++
 .../patches/guile-3.0-linux-syscalls.patch    | 353 ++++++++++++++++++
 .../patches/guile-3.0-relocatable.patch       |  79 ++++
 gnu/system/linux-initrd.scm                   |  10 +-
 6 files changed, 611 insertions(+), 81 deletions(-)
 create mode 100644 gnu/packages/patches/guile-2.2-default-utf8.patch
 create mode 100644 gnu/packages/patches/guile-3.0-linux-syscalls.patch
 create mode 100644 gnu/packages/patches/guile-3.0-relocatable.patch

-- 
2.26.2