GNU bug report logs - #37519
[PATCH v2] gnu: add iwd.

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 37519 in the body.
You can then email your comments to 37519 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: Brice Waegeneire <brice <at> waegenei.re>
To: guix-patches <at> gnu.org
Subject: [PATCH v2] gnu: add iwd.
Date: Thu, 26 Sep 2019 16:23:40 +0200

* gnu/packages/networking.scm (iwd): New variable.
---
This version of the patch split the commit in two, remove the commented
code and replace the patch with a post-unpack phase.

 gnu/packages/networking.scm | 56 +++++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 6bdf7f9e2f..87dd015320 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -30,6 +30,7 @@
 ;;; Copyright © 2019 Vasile Dumitrascu <va511e <at> yahoo.com>
 ;;; Copyright © 2019 Julien Lepiller <julien <at> lepiller.eu>
 ;;; Copyright © 2019 Timotej Lazar <timotej.lazar <at> araneo.si>
+;;; Copyright © 2019 Brice Waegeneire <brice <at> waegenei.re>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2578,3 +2579,58 @@ communication.")
     (description "FRRouting (FRR) is an IP routing protocol suite which includes
 protocol daemons for BGP, IS-IS, LDP, OSPF, PIM, and RIP. ")
     (license license:gpl2+)))
+
+(define-public iwd
+  (package
+    (name "iwd")
+    (version "0.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://git.kernel.org/pub/scm/network/wireless/iwd.git")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "001dikinsa6kshwscjbvwipavzwpqnpvx9fpshcn63gbvbhyd393"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("dbus" ,dbus)
+       ("libtool" ,libtool)
+       ("ell" ,ell)
+       ("readline" ,readline)))
+    (native-inputs
+     `(("asciidoc" ,asciidoc)
+       ("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("pkgconfig" ,pkg-config)
+       ("python" ,python)
+       ("openssl" ,openssl)))
+    (arguments
+     `(#:configure-flags
+       (let ((dbus (assoc-ref %outputs "out")))
+         (list "--disable-systemd-service"
+               "--enable-external-ell"
+               "--enable-hwsim"
+               "--enable-tools"
+               "--enable-wired"
+               "--enable-docs"
+               "--localstatedir=/var"
+               (string-append "--with-dbus-datadir=" dbus "/share/")
+               (string-append "--with-dbus-busdir="
+                              dbus "/share/dbus-1/system-services")))
+       #:phases
+       (modify-phases %standard-phases
+         (add-before 'configure 'pre-configure
+           (lambda _
+             ;; Don't try to 'mkdir /var'.
+             (substitute* "Makefile.in"
+               (("\\$\\(MKDIR_P\\) -m 700")
+                "true")))))))
+    (home-page "https://git.kernel.org/cgit/network/wireless/iwd.git/")
+    (synopsis "Internet Wireless Daemon")
+    (description "iwd is a wireless daemon for Linux written by Intel that
+aims to replace WPA supplicant. It optimize resource utilization by not
+depending on any external libraries and instead utilizing features provided by
+the Linux Kernel to the maximum extent possible.")
+    (license license:lgpl2.1+)))
-- 
2.19.2

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brice Waegeneire <brice <at> waegenei.re>
Cc: 37519 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2] gnu: add iwd.
Date: Thu, 26 Sep 2019 23:05:52 +0200

Hi Brice,

Brice Waegeneire <brice <at> waegenei.re> skribis:

> * gnu/packages/networking.scm (iwd): New variable.
> ---
> This version of the patch split the commit in two, remove the commented
> code and replace the patch with a post-unpack phase.

This patch looks good, but could you also resend the ell patch?

Thanks in advance,
Ludo’.

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

From: Brice Waegeneire <brice <at> waegenei.re>
To: 37519 <at> debbugs.gnu.org
Subject: [PATCH v2 1/2] gnu: Add ell.
Date: Thu, 26 Sep 2019 23:44:30 +0200

* gnu/packages/linux.scm (ell): New variable.
---
 gnu/packages/linux.scm | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 59874b097c..84b3a148f2 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -36,6 +36,7 @@
 ;;; Copyright © 2019 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
 ;;; Copyright © 2019 Stefan Stefanović <stefanx2ovic <at> gmail.com>
 ;;; Copyright © 2019 Pierre Langlois <pierre.langlois <at> gmx.com>
+;;; Copyright © 2019 Brice Waegeneire <brice <at> waegenei.re>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -5968,3 +5969,42 @@ have to construct the archives directly, without using the archiver.")
     (description "inputattach dispatches input events from several device
 types and interfaces and translates so that the X server can use them.")
     (license license:gpl2+)))
+
+(define-public ell
+  (package
+    (name "ell")
+    (version "0.23")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://git.kernel.org/pub/scm/libs/ell/ell.git")
+                    (commit version)))
+              (sha256
+               (base32
+                "1qhlcwhn0gj877yss2ymx1aczghlddzb5v9mm1dgp2zliii3jy10"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'fix-dbus-tests
+           (lambda _
+             (substitute* '("unit/test-dbus-message-fds.c"
+                            "unit/test-dbus-properties.c"
+                            "unit/test-dbus.c")
+               (("/usr/bin/dbus-daemon") (which "dbus-daemon")))
+             #t)))))
+    (inputs
+     `(("dbus" ,dbus)
+       ("libtool" ,libtool)))
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("pkgconfig" ,pkg-config)
+       ("automake" ,automake)))
+    (home-page "https://01.org/ell")
+    (synopsis "Embedded Linux Library")
+    (description "The Embedded Linux* Library (ELL) provides core, low-level
+functionality for system daemons. It typically has no dependencies other than
+the Linux kernel, C standard library, and libdl (for dynamic linking). While
+ELL is designed to be efficient and compact enough for use on embedded Linux
+platforms, it is not limited to resource-constrained systems.")
+    (license license:lgpl2.1+)))
-- 
2.19.2

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

From: Brice Waegeneire <brice <at> waegenei.re>
To: 37519 <at> debbugs.gnu.org
Subject: [PATCH v2 2/2] gnu: add iwd.
Date: Thu, 26 Sep 2019 23:44:31 +0200

* gnu/packages/networking.scm (iwd): New variable.
---
 gnu/packages/networking.scm | 56 +++++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 6bdf7f9e2f..87dd015320 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -30,6 +30,7 @@
 ;;; Copyright © 2019 Vasile Dumitrascu <va511e <at> yahoo.com>
 ;;; Copyright © 2019 Julien Lepiller <julien <at> lepiller.eu>
 ;;; Copyright © 2019 Timotej Lazar <timotej.lazar <at> araneo.si>
+;;; Copyright © 2019 Brice Waegeneire <brice <at> waegenei.re>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2578,3 +2579,58 @@ communication.")
     (description "FRRouting (FRR) is an IP routing protocol suite which includes
 protocol daemons for BGP, IS-IS, LDP, OSPF, PIM, and RIP. ")
     (license license:gpl2+)))
+
+(define-public iwd
+  (package
+    (name "iwd")
+    (version "0.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://git.kernel.org/pub/scm/network/wireless/iwd.git")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "001dikinsa6kshwscjbvwipavzwpqnpvx9fpshcn63gbvbhyd393"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("dbus" ,dbus)
+       ("libtool" ,libtool)
+       ("ell" ,ell)
+       ("readline" ,readline)))
+    (native-inputs
+     `(("asciidoc" ,asciidoc)
+       ("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("pkgconfig" ,pkg-config)
+       ("python" ,python)
+       ("openssl" ,openssl)))
+    (arguments
+     `(#:configure-flags
+       (let ((dbus (assoc-ref %outputs "out")))
+         (list "--disable-systemd-service"
+               "--enable-external-ell"
+               "--enable-hwsim"
+               "--enable-tools"
+               "--enable-wired"
+               "--enable-docs"
+               "--localstatedir=/var"
+               (string-append "--with-dbus-datadir=" dbus "/share/")
+               (string-append "--with-dbus-busdir="
+                              dbus "/share/dbus-1/system-services")))
+       #:phases
+       (modify-phases %standard-phases
+         (add-before 'configure 'pre-configure
+           (lambda _
+             ;; Don't try to 'mkdir /var'.
+             (substitute* "Makefile.in"
+               (("\\$\\(MKDIR_P\\) -m 700")
+                "true")))))))
+    (home-page "https://git.kernel.org/cgit/network/wireless/iwd.git/")
+    (synopsis "Internet Wireless Daemon")
+    (description "iwd is a wireless daemon for Linux written by Intel that
+aims to replace WPA supplicant. It optimize resource utilization by not
+depending on any external libraries and instead utilizing features provided by
+the Linux Kernel to the maximum extent possible.")
+    (license license:lgpl2.1+)))
-- 
2.19.2

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brice Waegeneire <brice <at> waegenei.re>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2 1/2] gnu: Add ell.
Date: Sat, 28 Sep 2019 22:49:56 +0200

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

Hi,

Brice Waegeneire <brice <at> waegenei.re> skribis:

> * gnu/packages/linux.scm (ell): New variable.

Applied with the changes below to address ‘guix lint’ warnings.

Ludo’.

[Message part 2 (text/x-patch, inline)]

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index ddbc5cd305..876eb005af 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -5980,6 +5980,7 @@ types and interfaces and translates so that the X server can use them.")
               (uri (git-reference
                     (url "https://git.kernel.org/pub/scm/libs/ell/ell.git")
                     (commit version)))
+              (file-name (git-file-name name version))
               (sha256
                (base32
                 "1qhlcwhn0gj877yss2ymx1aczghlddzb5v9mm1dgp2zliii3jy10"))))
@@ -6004,8 +6005,8 @@ types and interfaces and translates so that the X server can use them.")
     (home-page "https://01.org/ell")
     (synopsis "Embedded Linux Library")
     (description "The Embedded Linux* Library (ELL) provides core, low-level
-functionality for system daemons. It typically has no dependencies other than
-the Linux kernel, C standard library, and libdl (for dynamic linking). While
+functionality for system daemons.  It typically has no dependencies other than
+the Linux kernel, C standard library, and libdl (for dynamic linking).  While
 ELL is designed to be efficient and compact enough for use on embedded Linux
 platforms, it is not limited to resource-constrained systems.")
     (license license:lgpl2.1+)))

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brice Waegeneire <brice <at> waegenei.re>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2 2/2] gnu: add iwd.
Date: Sat, 28 Sep 2019 22:56:11 +0200

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

Brice Waegeneire <brice <at> waegenei.re> skribis:

> * gnu/packages/networking.scm (iwd): New variable.

On my machine I see one test failure:

--8<---------------cut here---------------start------------->8---
FAIL: unit/test-eapol
=====================

test-eapol: unit/test-eapol.c:2959: eapol_sm_test_tls: Assertion `l_tls_set_auth_data(s->tls, CERTDIR "cert-server.pem", CERTDIR "cert-server-key-pkcs8.pem", NULL)' failed.
FAIL unit/test-eapol (exit status: 134)
--8<---------------cut here---------------end--------------->8---

Could you take a look?

Also, I had applied the cosmetic changes below.

We’re almost there, thank you!  :-)

Ludo’.

[Message part 2 (text/x-patch, inline)]

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 5a9340c3bd..213ac4cfc1 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -2626,11 +2626,12 @@ protocol daemons for BGP, IS-IS, LDP, OSPF, PIM, and RIP. ")
              ;; Don't try to 'mkdir /var'.
              (substitute* "Makefile.in"
                (("\\$\\(MKDIR_P\\) -m 700")
-                "true")))))))
-    (home-page "https://git.kernel.org/cgit/network/wireless/iwd.git/")
+                "true"))
+             #t)))))
+    (home-page "https://git.kernel.org/pub/scm/network/wireless/iwd.git/")
     (synopsis "Internet Wireless Daemon")
-    (description "iwd is a wireless daemon for Linux written by Intel that
-aims to replace WPA supplicant. It optimize resource utilization by not
-depending on any external libraries and instead utilizing features provided by
-the Linux Kernel to the maximum extent possible.")
+    (description "iwd is a wireless daemon for Linux that aims to replace WPA
+Supplicant.  It optimizes resource utilization by not depending on any external
+libraries and instead utilizing features provided by the Linux kernel to the
+maximum extent possible.")
     (license license:lgpl2.1+)))

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

From: brice <at> waegenei.re <brice <at> waegenei.re>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37493] [bug#37519] [PATCH v2 2/2] gnu: add 
 iwd.
Date: Sat, 28 Sep 2019 23:49:58 +0200

On 28 September, 2019 22:56 CEST, Ludovic Courtès <ludo <at> gnu.org> wrote:

> On my machine I see one test failure:
>
> --8<---------------cut here---------------start------------->8---
> FAIL: unit/test-eapol
> =====================
>
> test-eapol: unit/test-eapol.c:2959: eapol_sm_test_tls: Assertion `l_tls_set_auth_data(s->tls, CERTDIR "cert-server.pem", CERTDIR "cert-server-key-pkcs8.pem", NULL)' failed.
> FAIL unit/test-eapol (exit status: 134)
> --8<---------------cut here---------------end--------------->8---

Unfortunately I can't reproduce this test failure and I didn't recall stumbling upon it during the trial and error packaging process.
--8<---------------cut here---------------start------------->8---
# TOTAL: 19
# PASS:  19
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
--8<---------------cut here---------------end--------------->8---
Where do we go from here?

--
bricewge ~ 8929 BBC5 73CD 9206 3DDD  979D 3D36 CAA0 116F 0F99

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

From: brice <at> waegenei.re <brice <at> waegenei.re>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2
 2/2] gnu: add iwd.
Date: Sun, 29 Sep 2019 19:18:04 +0200

I managed to reproduce the issue in a VM, previously I was building in NixOS. Unfortunately I wasn't successful in fixing it.

Alpine disabled the test that fail for Guix: https://git.alpinelinux.org/aports/tree/community/iwd/disable-eapol-test.patch



--
bricewge ~ 8929 BBC5 73CD 9206 3DDD  979D 3D36 CAA0 116F 0F99

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: "brice\@waegenei.re" <brice <at> waegenei.re>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2  2/2] gnu: add iwd.
Date: Sun, 29 Sep 2019 23:01:09 +0200

Hi,

"brice <at> waegenei.re" <brice <at> waegenei.re> skribis:

> I managed to reproduce the issue in a VM, previously I was building in NixOS. Unfortunately I wasn't successful in fixing it. 
>
> Alpine disabled the test that fail for Guix: https://git.alpinelinux.org/aports/tree/community/iwd/disable-eapol-test.patch

I straced ‘test-eapol’ and got this:

--8<---------------cut here---------------start------------->8---
8701  add_key("keyring", "ell-internal", NULL, 0, KEY_SPEC_THREAD_KEYRING) = 618338105
8701  add_key("asymmetric", "ell-key-0", "0"..., 1219, 618338105) = -1 EBADMSG (Malbona mesaĝo)
8701  write(2, "test-eapol: unit/test-eapol.c:2959: eapol_sm_test_tls: Assertion `l_tls_set_auth_data(s->tls, CERTDIR \"cert-server.pem\", CERTDIR \"cert-server-key-pkcs8.pem\", NULL)' failed.\n", 173) = 173
--8<---------------cut here---------------end--------------->8---

I didn’t know about the add_key(2) syscall, but looking at the man page,
it seems that the “asymmetric” type is not supported (but EBADMSG is not
documented either…).

Could you check what the intertubes have to say on this topic, or
whether changes have been made in ell (where ‘l_tls_set_auth_data’ is
defined) in this area?

Thanks in advance!

Ludo’.

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

From: Brice Waegeneire <brice <at> waegenei.re>
To: 37519 <at> debbugs.gnu.org
Subject: [PATCH v3] gnu: add iwd.
Date: Mon, 30 Sep 2019 16:43:21 +0200

* gnu/packages/networking.scm (iwd): New variable.
---
 gnu/packages/networking.scm | 60 +++++++++++++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a93e2cb9fc..bc768c6697 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -30,6 +30,7 @@
 ;;; Copyright © 2019 Vasile Dumitrascu <va511e <at> yahoo.com>
 ;;; Copyright © 2019 Julien Lepiller <julien <at> lepiller.eu>
 ;;; Copyright © 2019 Timotej Lazar <timotej.lazar <at> araneo.si>
+;;; Copyright © 2019 Brice Waegeneire <brice <at> waegenei.re>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2578,3 +2579,62 @@ communication.")
     (description "FRRouting (FRR) is an IP routing protocol suite which includes
 protocol daemons for BGP, IS-IS, LDP, OSPF, PIM, and RIP. ")
     (license license:gpl2+)))
+
+(define-public iwd
+  (package
+    (name "iwd")
+    (version "0.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://git.kernel.org/pub/scm/network/wireless/iwd.git")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "001dikinsa6kshwscjbvwipavzwpqnpvx9fpshcn63gbvbhyd393"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("dbus" ,dbus)
+       ("libtool" ,libtool)
+       ("ell" ,ell)
+       ("readline" ,readline)))
+    (native-inputs
+     `(("asciidoc" ,asciidoc)
+       ("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("pkgconfig" ,pkg-config)
+       ("python" ,python)
+       ("openssl" ,openssl)))
+    (arguments
+     `(#:configure-flags
+       (let ((dbus (assoc-ref %outputs "out")))
+         (list "--disable-systemd-service"
+               "--enable-external-ell"
+               "--enable-hwsim"
+               "--enable-tools"
+               "--enable-wired"
+               "--enable-docs"
+               "--localstatedir=/var"
+               (string-append "--with-dbus-datadir=" dbus "/share/")
+               (string-append "--with-dbus-busdir="
+                              dbus "/share/dbus-1/system-services")))
+       #:phases
+       (modify-phases %standard-phases
+         (add-before 'configure 'pre-configure
+           (lambda _
+             ;; Test disabled because it need the kernel module
+             ;; pkcs8_key_parser loaded.
+             (substitute* "Makefile.am"
+               (("unit\\/test-eapol.*? ") ""))
+             ;; Don't try to 'mkdir /var'.
+             (substitute* "Makefile.in"
+               (("\\$\\(MKDIR_P\\) -m 700") "true"))
+             #t)))))
+    (home-page "https://git.kernel.org/pub/scm/network/wireless/iwd.git/")
+    (synopsis "Internet Wireless Daemon")
+    (description "iwd is a wireless daemon for Linux that aims to replace WPA
+Supplicant.  It optimizes resource utilization by not depending on any external
+libraries and instead utilizing features provided by the Linux kernel to the
+maximum extent possible.")
+    (license license:lgpl2.1+)))
-- 
2.19.2

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

From: brice <at> waegenei.re <brice <at> waegenei.re>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2
 2/2] gnu: add iwd.
Date: Mon, 30 Sep 2019 16:50:55 +0200

On 29 September, 2019 23:01 CEST, Ludovic Courtès <ludo <at> gnu.org> wrote:

> I didn’t know about the add_key(2) syscall, but looking at the man page,
> it seems that the “asymmetric” type is not supported (but EBADMSG is not
> documented either…).

It was missing the kernel module pkcs8_key_parser, as explained in src/pkcs8.conf. Since we can't load it at build time, I disabled the test. Following is the content of src/pkcs8.conf:

--8<---------------cut here---------------start------------->8---
 # When distributions use CONFIG_PKCS8_PRIVATE_KEY_PARSER=m kernel option,
# using keyctl(2) will fail for loading PKCS#8 private keys since there is
# no automatic module loading for key type parsers.  This entry ensures
# that the kernel module pkcs8_key_parser.ko is loaded at boot time.
--8<---------------cut here---------------end--------------->8---
--
bricewge ~ 8929 BBC5 73CD 9206 3DDD  979D 3D36 CAA0 116F 0F99

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: "brice\@waegenei.re" <brice <at> waegenei.re>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2  2/2] gnu: add iwd.
Date: Tue, 01 Oct 2019 10:27:03 +0200

Hi Brice,

"brice <at> waegenei.re" <brice <at> waegenei.re> skribis:

> On 29 September, 2019 23:01 CEST, Ludovic Courtès <ludo <at> gnu.org> wrote: 
>  
>> I didn’t know about the add_key(2) syscall, but looking at the man page,
>> it seems that the “asymmetric” type is not supported (but EBADMSG is not
>> documented either…).
>
> It was missing the kernel module pkcs8_key_parser, as explained in src/pkcs8.conf. Since we can't load it at build time, I disabled the test. Following is the content of src/pkcs8.conf:
>
>  # When distributions use CONFIG_PKCS8_PRIVATE_KEY_PARSER=m kernel option,
> # using keyctl(2) will fail for loading PKCS#8 private keys since there is
> # no automatic module loading for key type parsers.  This entry ensures
> # that the kernel module pkcs8_key_parser.ko is loaded at boot time.

Oh, I see, thanks for explaining.

So what about skipping the test altogether (because we cannot guarantee
that the kernel will have that kernel module), along the lines of the
Alpine patch you showed, but with a comment explaining why we skip the
test?

Thank you,
Ludo’.

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

From: brice <at> waegenei.re <brice <at> waegenei.re>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 37519 <at> debbugs.gnu.org, 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2
 2/2] gnu: add iwd.
Date: Tue, 01 Oct 2019 13:15:33 +0200

Hello Ludo',

I have already send a patch with the test disabled, you may have missed it.

Have a good day.

--
bricewge ~ 8929 BBC5 73CD 9206 3DDD  979D 3D36 CAA0 116F 0F99

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brice Waegeneire <brice <at> waegenei.re>
Cc: 37519-done <at> debbugs.gnu.org, 37493-done <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v3] gnu: add iwd.
Date: Tue, 01 Oct 2019 23:13:07 +0200

Hello,

Brice Waegeneire <brice <at> waegenei.re> skribis:

> * gnu/packages/networking.scm (iwd): New variable.

I changed it as follows so that the test is really disabled and
committed.

Thanks!

Ludo’.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brice Waegeneire <brice <at> waegenei.re>
Cc: 37519-done <at> debbugs.gnu.org, 37493-done <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v3] gnu: add iwd.
Date: Tue, 01 Oct 2019 23:14:12 +0200

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

Hello,

(Now with the patch…)

Brice Waegeneire <brice <at> waegenei.re> skribis:

> * gnu/packages/networking.scm (iwd): New variable.

I changed it as follows so that the test is really disabled and
committed.

Thanks!

Ludo’.

[Message part 2 (text/x-patch, inline)]

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index bc768c6697..07b056adab 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -2621,14 +2621,13 @@ protocol daemons for BGP, IS-IS, LDP, OSPF, PIM, and RIP. ")
                               dbus "/share/dbus-1/system-services")))
        #:phases
        (modify-phases %standard-phases
-         (add-before 'configure 'pre-configure
+         (add-before 'bootstrap 'pre-bootstrap
            (lambda _
-             ;; Test disabled because it need the kernel module
-             ;; pkcs8_key_parser loaded.
              (substitute* "Makefile.am"
-               (("unit\\/test-eapol.*? ") ""))
-             ;; Don't try to 'mkdir /var'.
-             (substitute* "Makefile.in"
+               ;; Test disabled because it needs the kernel module
+               ;; 'pkcs8_key_parser' loaded.
+               (("unit\\/test-eapol.*? ") "")
+               ;; Don't try to 'mkdir /var'.
                (("\\$\\(MKDIR_P\\) -m 700") "true"))
              #t)))))
     (home-page "https://git.kernel.org/pub/scm/network/wireless/iwd.git/")

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

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 37519 <at> debbugs.gnu.org, "brice <at> waegenei.re" <brice <at> waegenei.re>,
 37493 <at> debbugs.gnu.org
Subject: Re: [bug#37519] [PATCH v2  2/2] gnu: add iwd.
Date: Wed, 2 Oct 2019 00:30:21 +0200

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

Hi,

> > It was missing the kernel module pkcs8_key_parser, as explained in src/pkcs8.conf. Since we can't load it at build time, I disabled the test.

It's fine to do it like that in that case, but just some food for thought:

We often disable tests in cases like that because writing system tests for something like that is annoying if one has to do it manually.

Would it be possible to discover packages which need those kinds of tests (if necessary just specify an "argument" in the package record) and automatically create&run system tests for them?  Guix would then run all the tests in a qemu-system container.

Something like

  $ make TESTS=packages check-system

which would:

* Traverse all the packages with (#:run-tests-as-system-tests? #t)
* Automatically set up a system test to run the tests of all of these packages.  Basically we could just provide guix-daemon inside a qemu guest and build the same derivation again in there, letting it load Linux kernel modules however it wants.

Especially Linux kernel-requiring (or worse, -modifying) tests are otherwise impossible to do.

Long term, I don't feel good excluding all those tests just because they are low-level.

Docs:

>there is no automatic module loading for key type parsers

Why not?  Weird... there's require_module, was it not thought of--or is it inapplicable somehow?

[Message part 2 (application/pgp-signature, inline)]

Previous Next

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