[PATCH core-updates] guix: self: Use guile with libgc-7.

  • Done
  • quality assurance status badge
Details
2 participants
  • Ludovic Courtès
  • Christopher Baines
Owner
unassigned
Submitted by
Christopher Baines
Severity
normal
C
C
Christopher Baines wrote on 17 Apr 2020 19:21
(address . guix-patches@gnu.org)
20200417172142.23741-1-mail@cbaines.net
Rather than libgc version 8. This should avoid crashes that can occur,
particularly when loading data in to the Guix Data Service [1].


* gnu/packages/guile.scm (guile-3.0/libgc-7): New variable.
* guix/self.scm (specification->package): Use guile-3.0/libgc-7 for guile.
---
gnu/packages/guile.scm | 12 ++++++++++++
guix/self.scm | 2 +-
2 files changed, 13 insertions(+), 1 deletion(-)

Toggle diff (38 lines)
diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
index 8ccbc22f26..6b7bd492ed 100644
--- a/gnu/packages/guile.scm
+++ b/gnu/packages/guile.scm
@@ -309,6 +309,18 @@ without requiring the source code to be rewritten.")
(define-public guile-next guile-3.0)
+(define-public guile-3.0/libgc-7
+ (hidden-package
+ (package
+ (inherit guile-3.0)
+ (propagated-inputs
+ (map (lambda (input)
+ (if (string=? (car input)
+ "bdw-gc")
+ (list "bdw-gc" libgc-7)
+ input))
+ (package-propagated-inputs guile-3.0))))))
+
(define-public guile-3.0/fixed
;; A package of Guile that's rarely changed. It is the one used in the
;; `base' module, and thus changing it entails a full rebuild.
diff --git a/guix/self.scm b/guix/self.scm
index 842161400e..905f931aeb 100644
--- a/guix/self.scm
+++ b/guix/self.scm
@@ -48,7 +48,7 @@
(let ((ref (lambda (module variable)
(module-ref (resolve-interface module) variable))))
(match-lambda
- ("guile" (ref '(gnu packages guile) 'guile-3.0))
+ ("guile" (ref '(gnu packages guile) 'guile-3.0/libgc-7))
("guile-json" (ref '(gnu packages guile) 'guile-json-3))
("guile-ssh" (ref '(gnu packages ssh) 'guile-ssh))
("guile-git" (ref '(gnu packages guile) 'guile-git))
--
2.26.0
L
L
Ludovic Courtès wrote on 17 Apr 2020 23:02
(name . Christopher Baines)(address . mail@cbaines.net)(address . 40684@debbugs.gnu.org)
87mu79akqw.fsf@gnu.org
Hi!

Christopher Baines <mail@cbaines.net> skribis:

Toggle quote (31 lines)
> Rather than libgc version 8. This should avoid crashes that can occur,
> particularly when loading data in to the Guix Data Service [1].
>
> 1: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40525
>
> * gnu/packages/guile.scm (guile-3.0/libgc-7): New variable.
> * guix/self.scm (specification->package): Use guile-3.0/libgc-7 for guile.
> ---
> gnu/packages/guile.scm | 12 ++++++++++++
> guix/self.scm | 2 +-
> 2 files changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
> index 8ccbc22f26..6b7bd492ed 100644
> --- a/gnu/packages/guile.scm
> +++ b/gnu/packages/guile.scm
> @@ -309,6 +309,18 @@ without requiring the source code to be rewritten.")
>
> (define-public guile-next guile-3.0)
>
> +(define-public guile-3.0/libgc-7
> + (hidden-package
> + (package
> + (inherit guile-3.0)
> + (propagated-inputs
> + (map (lambda (input)
> + (if (string=? (car input)
> + "bdw-gc")
> + (list "bdw-gc" libgc-7)
> + input))

Nitpicking: please write it as:

`(("bdw-gc" ,libgc-7)
,@(alist-delete "bdw-gc" (package-propagated-inputs guile-3.0)))

as we do elsewhere.

Also, could you add a comment referencing the bug report, so we
immediately see why this variant exists?

OK with these changes, thanks!

Ludo’.
C
C
Christopher Baines wrote on 18 Apr 2020 10:08
[PATCH v2] guix: self: Use guile with libgc-7.
(address . 40684@debbugs.gnu.org)
20200418080847.24298-1-mail@cbaines.net
Rather than libgc version 8. This should avoid crashes that can occur,
particularly when loading data in to the Guix Data Service [1].


* gnu/packages/guile.scm (guile-3.0/libgc-7): New variable.
* guix/self.scm (specification->package): Use guile-3.0/libgc-7 for guile.
---
gnu/packages/guile.scm | 11 +++++++++++
guix/self.scm | 2 +-
2 files changed, 12 insertions(+), 1 deletion(-)

Toggle diff (37 lines)
diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
index 8ccbc22f26..a262553ba1 100644
--- a/gnu/packages/guile.scm
+++ b/gnu/packages/guile.scm
@@ -309,6 +309,17 @@ without requiring the source code to be rewritten.")
(define-public guile-next guile-3.0)
+(define-public guile-3.0/libgc-7
+ ;; Using libgc-7 avoid crashes that can occur, particularly when loading
+ ;; data in to the Guix Data Service:
+ ;; https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40525
+ (hidden-package
+ (package
+ (inherit guile-3.0)
+ (propagated-inputs
+ `(("bdw-gc" ,libgc-7)
+ ,@(srfi-1:alist-delete "bdw-gc" (package-propagated-inputs guile-3.0)))))))
+
(define-public guile-3.0/fixed
;; A package of Guile that's rarely changed. It is the one used in the
;; `base' module, and thus changing it entails a full rebuild.
diff --git a/guix/self.scm b/guix/self.scm
index 842161400e..905f931aeb 100644
--- a/guix/self.scm
+++ b/guix/self.scm
@@ -48,7 +48,7 @@
(let ((ref (lambda (module variable)
(module-ref (resolve-interface module) variable))))
(match-lambda
- ("guile" (ref '(gnu packages guile) 'guile-3.0))
+ ("guile" (ref '(gnu packages guile) 'guile-3.0/libgc-7))
("guile-json" (ref '(gnu packages guile) 'guile-json-3))
("guile-ssh" (ref '(gnu packages ssh) 'guile-ssh))
("guile-git" (ref '(gnu packages guile) 'guile-git))
--
2.26.0
C
C
Christopher Baines wrote on 18 Apr 2020 10:10
Re: [bug#40684] [PATCH core-updates] guix: self: Use guile with libgc-7.
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 40684@debbugs.gnu.org)
87d0855i58.fsf@cbaines.net
Ludovic Courtès <ludo@gnu.org> writes:

Toggle quote (47 lines)
> Hi!
>
> Christopher Baines <mail@cbaines.net> skribis:
>
>> Rather than libgc version 8. This should avoid crashes that can occur,
>> particularly when loading data in to the Guix Data Service [1].
>>
>> 1: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40525
>>
>> * gnu/packages/guile.scm (guile-3.0/libgc-7): New variable.
>> * guix/self.scm (specification->package): Use guile-3.0/libgc-7 for guile.
>> ---
>> gnu/packages/guile.scm | 12 ++++++++++++
>> guix/self.scm | 2 +-
>> 2 files changed, 13 insertions(+), 1 deletion(-)
>>
>> diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
>> index 8ccbc22f26..6b7bd492ed 100644
>> --- a/gnu/packages/guile.scm
>> +++ b/gnu/packages/guile.scm
>> @@ -309,6 +309,18 @@ without requiring the source code to be rewritten.")
>>
>> (define-public guile-next guile-3.0)
>>
>> +(define-public guile-3.0/libgc-7
>> + (hidden-package
>> + (package
>> + (inherit guile-3.0)
>> + (propagated-inputs
>> + (map (lambda (input)
>> + (if (string=? (car input)
>> + "bdw-gc")
>> + (list "bdw-gc" libgc-7)
>> + input))
>
> Nitpicking: please write it as:
>
> `(("bdw-gc" ,libgc-7)
> ,@(alist-delete "bdw-gc" (package-propagated-inputs guile-3.0)))
>
> as we do elsewhere.
>
> Also, could you add a comment referencing the bug report, so we
> immediately see why this variant exists?
>
> OK with these changes, thanks!

Sure, I've sent an updated patch now.

Thanks,

Chris
-----BEGIN PGP SIGNATURE-----

iQKTBAEBCgB9FiEEPonu50WOcg2XVOCyXiijOwuE9XcFAl6ateNfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNF
ODlFRUU3NDU4RTcyMEQ5NzU0RTBCMjVFMjhBMzNCMEI4NEY1NzcACgkQXiijOwuE
9XdVeg//XEhkCy8YRugHxQgnHr1n+ArW4ivW+USZRyMvVVY+253eVHmblaXYQ+yK
V1yqybidYPPhlqQMODDYVUE/67spv6Rs5pqHR9yt6okefiSlTv9rpBLqjDPseKqk
PeAidog7ykufeXMeRcCwfAf7nGpepuSlnXYCHp2IU8ZQkWo7M4cPD4isuK07bF6X
7/ddeAZYLiIp2FVthl8u84+ed1dtFaiiWtUUoZN2Yyk+3OYGlDWaUBa9qH/CXRnH
SB8L5GVtBBoRtdGGA5UAjMsqKSSx4ogB+8OdFNwr9eBufZQdsKUN5xun0iTXgzSS
1/mbs+txM8P8TMJnNFCeGJwA1iLxdVaKYJh90vpwQlv9ufiaU3ERsYH56BEgDhPn
7BDoucLwhXHYMN/FvBwI6t+fFM1KA9JnkAUp+PTg19c3w35PH2FjwwprYfZuxmNo
/GhqL+rvGv5VxeTWut5dlMpTwvwWgQSX9Or7AXJhe3vXLVjzOj/2GhN95+XdIcAJ
oLFAl/x68xVS4APUTE/XB8cTtUtZwAgx7DH5Dmt3BLM6ZTzN3S7emRnO/Pq6eyEw
S3Cz1bJVdkjXCR132qOStwlSr71NCWv8L1LbKHYCPLGn0RPiimGK3R1UXALNvfzG
fYWwkXWH+vlKfj+XB3dZgdf0RLoruOdAco556AsJ2Etbj0ipbbY=
=pSHl
-----END PGP SIGNATURE-----

C
C
Christopher Baines wrote on 18 Apr 2020 18:51
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 40684-done@debbugs.gnu.org)
878sis68l1.fsf@cbaines.net
Christopher Baines <mail@cbaines.net> writes:

Toggle quote (2 lines)
> Sure, I've sent an updated patch now.

I went ahead and pushed this to core-updates earlier as
cef392f3936922b7b0b74bd59be67e660c10db67.

It looks to have had the intended effect, the Guix Data Service was able
to process the commit [1] :D


I'll go ahead and mark this bug as done now. Thanks for your help!
-----BEGIN PGP SIGNATURE-----

iQKTBAEBCgB9FiEEPonu50WOcg2XVOCyXiijOwuE9XcFAl6bMApfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNF
ODlFRUU3NDU4RTcyMEQ5NzU0RTBCMjVFMjhBMzNCMEI4NEY1NzcACgkQXiijOwuE
9XeU9A//ZKyYUHaD3UHIxYPeXoEMzzT6ZXVxlQd5rHRKjmmPHWgLcCTfqUT3ptjo
4ubqoiYKiDlEYqnOVrmhY1pdKBu6vmEl4zDdPTeWWfR8XIY7UrsD+coPbsSPRlfY
2EwiFXlBUdqZOPR1m50Df7YiwR11+IyJgEZA+Ay8qCiBimYlh4jZJ0dKwthLy6Bx
HmyLwpFJaKO8a3iIGhWTYqnD1G4W2llXpXIWea3KFBzRw8DN+db1lN1mPyKCJrAs
30ysjYBMyO11LFHk1iPe2tmu7+AY794GuM92Wx5264LNS9iHLQTMYzN3PriEgBBo
vjUE5tr7aep3/13Ss5kDLqZjED9+YF6o4f/HSx0ADuKDblvz64klUjjSU6sVq4wW
clsm2OImJnosHgInQPJhdq0d1e+mNhAdDPI3KTZYTAJl21bUUAfQ0uYEaYrFgOwE
WAIcLaLBbRyS6RSTxyAQJAgOyIn8a7zW+Bo+TFgkz+FpiiNAQ43aubbbM/tYn8Z7
VNpnlaM/3ktgPdztuBXb/2kGvNZ44+CEepw84QpOhJ9P/tMwMFzixE40j+UdpUxV
Kr1pc/W0MDOjg6ia8z6VkybTyqWWYN1NKgRguQcpb3BiRuQhokg0az59ukUKhtkV
gbKG4Mx573uucDc9KFhMoDTbjErk4uGDt9LEePv0zsR/0pfNUGs=
=6Uvw
-----END PGP SIGNATURE-----

Closed
?