GNU bug report logs - #58697
[bug] guix refresh nftables crashes

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 58697 in the body.
You can then email your comments to 58697 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: kiasoc5 <kiasoc5 <at> disroot.org>
To: bug-guix <at> gnu.org
Subject: [bug] guix refresh nftables crashes
Date: Sat, 22 Oct 2022 01:55:32 +0000

% guix refresh nftables
```
Backtrace:
          13 (primitive-load "/home/kiasoc5/.config/guix/current/…")
In guix/ui.scm:
   2263:7 12 (run-guix . _)
  2226:10 11 (run-guix-command _ . _)
In ice-9/boot-9.scm:
  1752:10 10 (with-exception-handler _ _ #:unwind? _ # _)
  1752:10  9 (with-exception-handler _ _ #:unwind? _ # _)
In guix/store.scm:
   656:37  8 (thunk)
  2165:25  7 (run-with-store #<store-connection 256.99 6cf50eb53e60> …)
In guix/scripts/refresh.scm:
   567:14  6 (_ _)
In srfi/srfi-1.scm:
    634:9  5 (for-each #<procedure 6cf50ec23ba0 at guix/scripts/ref…> …)
In guix/scripts/refresh.scm:
   367:10  4 (check-for-package-update #<package nftables <at> 1.0.5 gnu…> …)
In srfi/srfi-1.scm:
   858:15  3 (any1 #<procedure 6cf50ec23b40 at guix/upstream.scm:28…> …)
In guix/gnu-maintenance.scm:
   834:10  2 (_ _ . _)
In ice-9/boot-9.scm:
  1685:16  1 (raise-exception _ #:continuable? _)
  1685:16  0 (raise-exception _ #:continuable? _)

ice-9/boot-9.scm:1685:16: In procedure raise-exception:
In procedure getaddrinfo: Servname not supported for ai_socktype
```
-- 

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

From: Maxime Devos <maximedevos <at> telenet.be>
To: kiasoc5 <kiasoc5 <at> disroot.org>, 58697 <at> debbugs.gnu.org
Subject: Re: bug#58697: [bug] guix refresh nftables crashes
Date: Sat, 22 Oct 2022 11:56:35 +0200

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

On 22-10-2022 03:55, kiasoc5 via Bug reports for GNU Guix wrote:
> % guix refresh nftables
> [...]
> ice-9/boot-9.scm:1685:16: In procedure raise-exception:
> In procedure getaddrinfo: Servname not supported for ai_socktype
> ```

I can reproduce this locally and don't know the cause.

Greetings,
Maxime

[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]

[OpenPGP_signature (application/pgp-signature, attachment)]

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

From: 宋文武 <iyzsong <at> envs.net>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: kiasoc5 <kiasoc5 <at> disroot.org>, 58697 <at> debbugs.gnu.org
Subject: Re: bug#58697: [bug] guix refresh nftables crashes
Date: Mon, 24 Oct 2022 16:44:02 +0800

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

Maxime Devos <maximedevos <at> telenet.be> writes:

> On 22-10-2022 03:55, kiasoc5 via Bug reports for GNU Guix wrote:
>> % guix refresh nftables
>> [...]
>> ice-9/boot-9.scm:1685:16: In procedure raise-exception:
>> In procedure getaddrinfo: Servname not supported for ai_socktype
>> ```
>
> I can reproduce this locally and don't know the cause.
>

It happens when the origin have both 'mirror://' and 'http://' urls,
current the html updater check for any url match 'http' or 'https', but
when updating it will just pick the first url, so when the first is
'mirror://' this error will come.

This patch should fix it:

[0001-gnu-maintenance-Don-t-try-html-updater-on-mirror-url.patch (text/x-patch, attachment)]

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: 宋文武 <iyzsong <at> envs.net>
Cc: kiasoc5 <kiasoc5 <at> disroot.org>, 58697-done <at> debbugs.gnu.org,
 Maxime Devos <maximedevos <at> telenet.be>
Subject: Re: bug#58697: [bug] guix refresh nftables crashes
Date: Mon, 11 Sep 2023 23:40:39 -0400

Hi,

宋文武 <iyzsong <at> envs.net> writes:

> Maxime Devos <maximedevos <at> telenet.be> writes:
>
>> On 22-10-2022 03:55, kiasoc5 via Bug reports for GNU Guix wrote:
>>> % guix refresh nftables
>>> [...]
>>> ice-9/boot-9.scm:1685:16: In procedure raise-exception:
>>> In procedure getaddrinfo: Servname not supported for ai_socktype
>>> ```
>>
>> I can reproduce this locally and don't know the cause.
>>
>
> It happens when the origin have both 'mirror://' and 'http://' urls,
> current the html updater check for any url match 'http' or 'https', but
> when updating it will just pick the first url, so when the first is
> 'mirror://' this error will come.

I ended up with honoring the first element of the URI, since support for
mirror:// URIs in (guix gnu-maintenance) was added recently (see commit
bdaef69556f68595e5ec0db1710bf8ad208abe20, "gnu-maintenance: Allow mirror
URLs to fallback to the generic HTML updater.").

The fix is pushed with commit 2a7f031ca9.  Let me know if anything is
still causing problems!

-- 
Thanks,
Maxim

Previous Next

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