GNU bug report logs - #61873
[PATCH core-updates] gnu: libgc: Update to 8.2.2 and remove '--disable-munmap'.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 61873 in the body.
You can then email your comments to 61873 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: guix-patches <at> gnu.org
Cc: Ludovic Courtès <ludo <at> gnu.org>
Subject: [PATCH core-updates] gnu: libgc: Update to 8.2.2 and remove
 '--disable-munmap'.
Date: Tue, 28 Feb 2023 18:38:54 +0100

* gnu/packages/bdw-gc.scm (libgc): Update to 8.2.2.
[arguments]: Remove "--disable-munmap" from #:configure-flags.
* gnu/packages/guile.scm (guile-3.0)[source]: Leave 'test-out-of-memory'
test untouched.
(guile-next)[arguments]: Likewise, in 'skip-failing-tests' phase.
(guile-for-guile-emacs)[arguments]: Likewise, in 'bootstrap' phase.
---
 gnu/packages/bdw-gc.scm |  9 +++------
 gnu/packages/guile.scm  | 16 +++-------------
 2 files changed, 6 insertions(+), 19 deletions(-)

Hello Guix!

This is another proposal for a world-rebuild change, hopefully the latest
(I swear!).  It’s quite important because it fixes a libgc bug that led
us to build libgc with ‘--disable-munmap’, meaning it would not return
memory to the OS:

  https://github.com/ivmai/bdwgc/issues/353

Thanks to Andrew Whatson for tackling this and for letting me know!

I tested on x86_64-linux on top of 4895846197bd445dd68c832364c667dea4337f8c
like so:

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env  guix build guix guile <at> 2.2 guile <at> 2.0 -nguix build: warning: ambiguous package specification `guile <at> 2.2'
guix build: warning: choosing guile <at> 2.2.7 from gnu/packages/guile.scm:253:2
/gnu/store/4nhq6kmqcfk2qnzhi4xifbb0zjav0f01-guile-2.0.14-debug
/gnu/store/4p1l5bdxxbyyqc3wh0d07jv9rp1pdcy7-guile-2.0.14
/gnu/store/k47958rhcrk176i9sabxda2rgmnlig4c-guile-2.2.7-debug
/gnu/store/ka0v3fk21lqrnwj4fzbf8z154mpbnjvx-guile-2.2.7
/gnu/store/agrv086cfa0a5f9cmj94k9zgwi1a08q6-guix-1.4.0-3.d5fece6
--8<---------------cut here---------------end--------------->8---

Given that the test suite of Guix passed and that everything in between
went fine, I’m rather confident.

Thoughts?

Ludo’.

diff --git a/gnu/packages/bdw-gc.scm b/gnu/packages/bdw-gc.scm
index cfa037dec0..86caa62d7e 100644
--- a/gnu/packages/bdw-gc.scm
+++ b/gnu/packages/bdw-gc.scm
@@ -1,5 +1,5 @@
 ;;; GNU Guix --- Functional package management for GNU
-;;; Copyright © 2012, 2013, 2014, 2016, 2017, 2020, 2021 Ludovic Courtès <ludo <at> gnu.org>
+;;; Copyright © 2012, 2013, 2014, 2016, 2017, 2020, 2021, 2023 Ludovic Courtès <ludo <at> gnu.org>
 ;;; Copyright © 2014 Mark H Weaver <mhw <at> netris.org>
 ;;; Copyright © 2016, 2018 Leo Famulari <leo <at> famulari.name>
 ;;; Copyright © 2017 Rene Saavedra <rennes <at> openmailbox.org>
@@ -34,14 +34,14 @@ (define-module (gnu packages bdw-gc)
 (define-public libgc
   (package
    (name "libgc")
-   (version "8.0.6")
+   (version "8.2.2")
    (source (origin
             (method url-fetch)
             (uri (string-append "https://github.com/ivmai/bdwgc/releases"
                                 "/download/v" version "/gc-" version ".tar.gz"))
             (sha256
              (base32
-              "04ga3c95w5az5sznzm73j19lvvfpf6k4sgkpjqsmjxpsr6mi8j9v"))))
+              "181ni2rn9qjbl4ilqdjrd1a4h8siv5bgmzqgg4595q32n2y0f0gk"))))
    (build-system gnu-build-system)
    (arguments
     (list
@@ -50,9 +50,6 @@ (define-public libgc
         ;; Install gc_cpp.h et al.
         "--enable-cplusplus"
 
-        ;; Work around <https://github.com/ivmai/bdwgc/issues/353>.
-        "--disable-munmap"
-
         ;; In GNU/Hurd systems during the 'check' phase,
         ;; there is a deadlock caused by the 'gctest' test.
         ;; To disable the error set "--disable-gcj-support"
diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
index c2ffbefeec..64a815dbd4 100644
--- a/gnu/packages/guile.scm
+++ b/gnu/packages/guile.scm
@@ -325,11 +325,8 @@ (define-public guile-3.0
               (patches '())
               ;; Replace the snippet because the oom-test still
               ;; fails on some 32-bit architectures.
-              (snippet '(begin
-                          (substitute* "test-suite/standalone/Makefile.in"
-                            (("test-out-of-memory") ""))
-                          (for-each delete-file
-                                    (find-files "prebuilt" "\\.go$"))))))
+              (snippet '(for-each delete-file
+                                  (find-files "prebuilt" "\\.go$")))))
 
     ;; Build with the bundled mini-GMP to avoid interference with GnuTLS' own
     ;; use of GMP via Nettle: <https://issues.guix.gnu.org/46330>.
@@ -444,10 +441,7 @@ (define-public guile-next
                      (display ,version port)))))
              (add-before 'check 'skip-failing-tests
                (lambda _
-                 (substitute* "test-suite/standalone/test-out-of-memory"
-                   (("!#") "!#\n\n(exit 77)\n"))
-                 (delete-file "test-suite/tests/version.test")
-                 #t))))))
+                 (delete-file "test-suite/tests/version.test")))))))
       (native-inputs
        (modify-inputs (package-native-inputs guile-3.0)
          (prepend autoconf
@@ -571,10 +565,6 @@ (define-public guile-for-guile-emacs
                  (substitute* "test-suite/tests/version.test"
                    (("\\(pass-if \"version reporting works\"" m)
                     (string-append "#;" m)))
-                 ;; Warning: Unwind-only `out-of-memory' exception; skipping pre-unwind handler.
-                 ;; FAIL: test-out-of-memory
-                 (substitute* "test-suite/standalone/Makefile.am"
-                   (("(check_SCRIPTS|TESTS) \\+= test-out-of-memory") ""))
 
                  (patch-shebang "build-aux/git-version-gen")
                  (invoke "sh" "autogen.sh")

base-commit: 4895846197bd445dd68c832364c667dea4337f8c
-- 
2.39.1

Message #8 received at 61873 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Enge <andreas <at> enge.fr>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, Josselin Poiret <dev <at> jpoiret.xyz>,
 Tobias Geerinckx-Rice <me <at> tobias.gr>,
 Simon Tournier <zimon.toutoune <at> gmail.com>, Mathieu Othacehe <othacehe <at> gnu.org>,
 Christopher Baines <mail <at> cbaines.net>, 61873 <at> debbugs.gnu.org,
 Andrew Whatson <whatson <at> tailcall.au>
Subject: Re: [bug#61873] [PATCH core-updates] gnu: libgc: Update to 8.2.2 and
 remove '--disable-munmap'.
Date: Tue, 28 Feb 2023 18:47:09 +0100

Am Tue, Feb 28, 2023 at 06:38:54PM +0100 schrieb Ludovic Courtès:
> This is another proposal for a world-rebuild change, hopefully the latest
> (I swear!).  It’s quite important because it fixes a libgc bug that led
> us to build libgc with ‘--disable-munmap’, meaning it would not return
> memory to the OS:
>   https://github.com/ivmai/bdwgc/issues/353
> Thanks to Andrew Whatson for tackling this and for letting me know!
> 
> Given that the test suite of Guix passed and that everything in between
> went fine, I’m rather confident.
> Thoughts?

Indeed, the bug looks important to fix!

Concerning world rebuilding, I do not know if anybody did; I did not ;-)
So maybe after pushing this, we could set up jobs on the build farms to
rebuild the world twice.

Andreas

Message #11 received at 61873 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Enge <andreas <at> enge.fr>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, Josselin Poiret <dev <at> jpoiret.xyz>,
 Tobias Geerinckx-Rice <me <at> tobias.gr>,
 Simon Tournier <zimon.toutoune <at> gmail.com>, Mathieu Othacehe <othacehe <at> gnu.org>,
 Christopher Baines <mail <at> cbaines.net>, 61873 <at> debbugs.gnu.org,
 Andrew Whatson <whatson <at> tailcall.au>
Subject: Re: [bug#61873] [PATCH core-updates] gnu: libgc: Update to 8.2.2 and
 remove '--disable-munmap'.
Date: Thu, 2 Mar 2023 15:36:14 +0100

Could you push it? Before that there is not much point in rebuilding
core-updates, knowing it will have to be rebuilt right after that again.

Andreas

Message #14 received at 61873 <at> debbugs.gnu.org (full text, mbox):

From: Josselin Poiret <dev <at> jpoiret.xyz>
To: Andreas Enge <andreas <at> enge.fr>, Ludovic Courtès
 <ludo <at> gnu.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, Tobias Geerinckx-Rice <me <at> tobias.gr>,
 Simon Tournier <zimon.toutoune <at> gmail.com>, Mathieu Othacehe <othacehe <at> gnu.org>,
 Christopher Baines <mail <at> cbaines.net>, 61873 <at> debbugs.gnu.org,
 Andrew Whatson <whatson <at> tailcall.au>
Subject: Re: [bug#61873] [PATCH core-updates] gnu: libgc: Update to 8.2.2
 and remove '--disable-munmap'.
Date: Thu, 02 Mar 2023 19:29:55 +0100

[Message part 1 (text/plain, inline)]

Hi Ludo and Andreas,

I agree with Andreas here, better push it now so that momentum doesn't
die out.  FWIW, the patch looks good to me, although the important part
will be how the new version fares in real applications :)

Best,
-- 
Josselin Poiret

[signature.asc (application/pgp-signature, inline)]

Message #19 received at 61873-done <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Andreas Enge <andreas <at> enge.fr>
Cc: Josselin Poiret <dev <at> jpoiret.xyz>, Christopher Baines <mail <at> cbaines.net>,
 Simon Tournier <zimon.toutoune <at> gmail.com>, 61873-done <at> debbugs.gnu.org,
 Mathieu Othacehe <othacehe <at> gnu.org>, Tobias Geerinckx-Rice <me <at> tobias.gr>,
 Ricardo Wurmus <rekado <at> elephly.net>, Andrew Whatson <whatson <at> tailcall.au>
Subject: Re: bug#61873: [PATCH core-updates] gnu: libgc: Update to 8.2.2 and
 remove '--disable-munmap'.
Date: Thu, 02 Mar 2023 21:54:55 +0100

Hi,

Andreas Enge <andreas <at> enge.fr> skribis:

> Could you push it? Before that there is not much point in rebuilding
> core-updates, knowing it will have to be rebuilt right after that again.

Pushed as 5222ed8dc364f13a36bca13c151856a3edab795a.  Apologies for the
delay!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.