unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: "Thompson, David" <dthompson2@worcester.edu>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 72183@debbugs.gnu.org
Subject: [bug#72183] [PATCH] gnu: guile: Update to 3.0.10.
Date: Fri, 19 Jul 2024 18:07:17 -0400	[thread overview]
Message-ID: <CAJ=RwfZ9WWpJ6DCPVKR_A_maVsuiWzxuiDFspRwzJK1qp5ULKg@mail.gmail.com> (raw)
In-Reply-To: <874j8lru82.fsf@gnu.org>

Hey Ludo,

On Fri, Jul 19, 2024 at 4:40 PM Ludovic Courtès <ludo@gnu.org> wrote:
>
> Hey,
>
> I temporarily reverted the commits that upgrade Guile and adjust Hoot in
> 31244f5cefae4c14a1a5d441dc3b3626b5f32abc so we can investigate the issue
> below (which broke ‘guix pull’) without pressure.

Oh no, sorry! I ran 'guix pull' after pushing these commits and didn't
experience issues so I thought all was well. :(

> Turns out ‘guile-3.0-latest’ is used to build Guix itself (via ‘guix
> pull’, but also the ‘guix’ package) and this cause a failure of
> guix-cli-core.drv:

How about using guile-3.0 for Guix so that future Guile updates can be
done without fear?

> The expressions leading to this internal compiler error are:
>
>   (bytevector->hash-data (sha256 (string->utf8 s))
>                          #:key-type (key-type public-key))
>
> and:
>
>   (bytevector->hash-data sha256
>                          #:key-type (key-type public-key))
>
> This sounds like a compiler bug, possibly related to Guile commit
> f95bf6921e13799abca6a0a13087609c42baba6b.
>
> Note that ‘bytevector->hash-data’ comes from Guile-Gcrypt, which was
> itself still compiled with 3.0.9.  So there’s a possibility that the bug
> comes with this particular combination as is exhibited by cross-module
> inlining.

Yup, that certainly sounds like what is happening here. Cross-module
inlining + the new keyword args optimization.

Sorry for breaking 'guix pull'. I thought I had scoped the changes
down to a safe level. :(

- Dave




  reply	other threads:[~2024-07-19 22:09 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-18 20:16 [bug#72183] [PATCH] gnu: guile: Update to 3.0.10 Thompson, David
2024-07-18 20:49 ` [bug#72183] [PATCH] gnu: shepherd 0.9: Switch from guile-3.0-latest to guile-3.0 Thompson, David
2024-07-19 16:32 ` [bug#72183] [PATCH] gnu: guile: Update to 3.0.10 Ludovic Courtès
2024-07-19 17:09   ` bug#72183: " Thompson, David
2024-07-19 20:40   ` [bug#72183] " Ludovic Courtès
2024-07-19 22:07     ` Thompson, David [this message]
2024-09-01 17:31       ` Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAJ=RwfZ9WWpJ6DCPVKR_A_maVsuiWzxuiDFspRwzJK1qp5ULKg@mail.gmail.com' \
    --to=dthompson2@worcester.edu \
    --cc=72183@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).