GNU bug report logs -
#37493
[PATCH] 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 37493 in the body.
You can then email your comments to 37493 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Mon, 23 Sep 2019 18:42:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Brice Waegeneire <brice <at> waegenei.re>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Mon, 23 Sep 2019 18:42:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/linux.scm (ell): New variable.
* gnu/packages/networking.scm (iwd): New variable.
* gnu/packages/patches/ell-fix-dbus-tests.patch: New file.
---
gnu/packages/linux.scm | 37 +++++++++++
gnu/packages/networking.scm | 56 ++++++++++++++++
gnu/packages/patches/ell-fix-dbus-tests.patch | 65 +++++++++++++++++++
3 files changed, 158 insertions(+)
create mode 100644 gnu/packages/patches/ell-fix-dbus-tests.patch
diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 59874b097c..11f2503841 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,39 @@ 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"))
+ (patches (search-patches "ell-fix-dbus-tests.patch"))))
+ (build-system gnu-build-system)
+ ;; (arguments
+ ;; `(#:tests? #f ;kernel/user integration tests are in package "xfstests"
+ ;; #:phases (modify-phases %standard-phases
+ ;; (add-after 'install 'install-headers
+ ;; (lambda _
+ ;; (invoke "make" "install-dev"))))))
+ (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+)))
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+)))
diff --git a/gnu/packages/patches/ell-fix-dbus-tests.patch b/gnu/packages/patches/ell-fix-dbus-tests.patch
new file mode 100644
index 0000000000..b494ba8b43
--- /dev/null
+++ b/gnu/packages/patches/ell-fix-dbus-tests.patch
@@ -0,0 +1,65 @@
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -140,6 +140,7 @@
+ ell_libell_private_la_SOURCES = $(ell_libell_la_SOURCES)
+
+ AM_CFLAGS = -fvisibility=hidden -DUNITDIR=\""$(top_srcdir)/unit/"\" \
++ -DDBUS_DAEMON=\""$(DBUS_DAEMONDIR)/dbus-daemon"\" \
+ -DCERTDIR=\""$(top_builddir)/unit/"\"
+
+ pkgconfigdir = $(libdir)/pkgconfig
+--- a/configure.ac
++++ b/configure.ac
+@@ -14,6 +14,8 @@
+
+ AC_PREFIX_DEFAULT(/usr/local)
+
++PKG_PROG_PKG_CONFIG
++
+ COMPILER_FLAGS
+
+ AC_LANG_C
+@@ -131,6 +133,10 @@
+ AC_CHECK_PROG(have_xxd, [xxd], [yes], [no])
+ fi
+
++PKG_CHECK_MODULES(DBUS, dbus-1, dummy=yes,
++ AC_MSG_ERROR(D-Bus is required for running tests))
++PKG_CHECK_VAR(DBUS_DAEMONDIR, dbus-1, daemondir)
++
+ AM_CONDITIONAL(DBUS_TESTS, test "${little_endian}" = "yes")
+ AM_CONDITIONAL(CERT_TESTS, test "${have_openssl}" = "yes")
+
+--- a/unit/test-dbus-message-fds.c
++++ b/unit/test-dbus-message-fds.c
+@@ -51,7 +51,7 @@
+ char *prg_envp[1];
+ pid_t pid;
+
+- prg_argv[0] = "/usr/bin/dbus-daemon";
++ prg_argv[0] = DBUS_DAEMON;
+ prg_argv[1] = "--nopidfile";
+ prg_argv[2] = "--nofork";
+ prg_argv[3] = "--config-file=" UNITDIR "dbus.conf";
+--- a/unit/test-dbus-properties.c
++++ b/unit/test-dbus-properties.c
+@@ -48,7 +48,7 @@
+ char *prg_envp[1];
+ pid_t pid;
+
+- prg_argv[0] = "/usr/bin/dbus-daemon";
++ prg_argv[0] = DBUS_DAEMON;
+ prg_argv[1] = "--nopidfile";
+ prg_argv[2] = "--nofork";
+ prg_argv[3] = "--config-file=" UNITDIR "dbus.conf";
+--- a/unit/test-dbus.c
++++ b/unit/test-dbus.c
+@@ -45,7 +45,7 @@
+ char *prg_envp[1];
+ pid_t pid;
+
+- prg_argv[0] = "/usr/bin/dbus-daemon";
++ prg_argv[0] = DBUS_DAEMON;
+ prg_argv[1] = "--nopidfile";
+ prg_argv[2] = "--nofork";
+ prg_argv[3] = "--config-file=" UNITDIR "dbus.conf";
--
2.19.2
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Wed, 25 Sep 2019 13:25:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 37493 <at> debbugs.gnu.org (full text, mbox):
Hi Brice,
Brice Waegeneire <brice <at> waegenei.re> skribis:
> * gnu/packages/linux.scm (ell): New variable.
> * gnu/packages/networking.scm (iwd): New variable.
> * gnu/packages/patches/ell-fix-dbus-tests.patch: New file.
We normally do one patch per package. I can split this patch on your
behalf if you want, what do you prefer?
[...]
> + ;; (arguments
> + ;; `(#:tests? #f ;kernel/user integration tests are in package "xfstests"
> + ;; #:phases (modify-phases %standard-phases
> + ;; (add-after 'install 'install-headers
> + ;; (lambda _
> + ;; (invoke "make" "install-dev"))))))
Should we remove this comment?
> --- /dev/null
> +++ b/gnu/packages/patches/ell-fix-dbus-tests.patch
It’s good to add one or two describing the patch and its upstream
status.
Thank you,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Thu, 26 Sep 2019 16:22:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 37493 <at> debbugs.gnu.org (full text, mbox):
Hello Ludovic,
Thank you for your review.
I'm new to the mailing list workflow , so I screw up... When posting the second version of the patch, It ended up as a new bug: #37519. Next time I'll used the specific debbugs email instead.
Regards,
--
bricewge ~ 8929 BBC5 73CD 9206 3DDD 979D 3D36 CAA0 116F 0F99
Merged 37493 37519.
Request was from
Ludovic Courtès <ludo <at> gnu.org>
to
control <at> debbugs.gnu.org
.
(Thu, 26 Sep 2019 21:05:01 GMT)
Full text and
rfc822 format available.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Sat, 28 Sep 2019 20:51:01 GMT)
Full text and
rfc822 format available.
Message #16 received at 37493 <at> debbugs.gnu.org (full text, mbox):
[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+)))
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Sat, 28 Sep 2019 20:57:01 GMT)
Full text and
rfc822 format available.
Message #19 received at 37493 <at> debbugs.gnu.org (full text, mbox):
[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+)))
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Sat, 28 Sep 2019 21:51:01 GMT)
Full text and
rfc822 format available.
Message #22 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Sun, 29 Sep 2019 17:19:01 GMT)
Full text and
rfc822 format available.
Message #25 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Sun, 29 Sep 2019 21:02:01 GMT)
Full text and
rfc822 format available.
Message #28 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Mon, 30 Sep 2019 14:51:01 GMT)
Full text and
rfc822 format available.
Message #31 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Tue, 01 Oct 2019 08:28:01 GMT)
Full text and
rfc822 format available.
Message #34 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Tue, 01 Oct 2019 11:16:02 GMT)
Full text and
rfc822 format available.
Message #37 received at 37493 <at> debbugs.gnu.org (full text, mbox):
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
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Tue, 01 Oct 2019 21:14:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Brice Waegeneire <brice <at> waegenei.re>
:
bug acknowledged by developer.
(Tue, 01 Oct 2019 21:14:02 GMT)
Full text and
rfc822 format available.
Message #42 received at 37493-done <at> debbugs.gnu.org (full text, mbox):
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’.
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Tue, 01 Oct 2019 21:14:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Brice Waegeneire <brice <at> waegenei.re>
:
bug acknowledged by developer.
(Tue, 01 Oct 2019 21:14:03 GMT)
Full text and
rfc822 format available.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Tue, 01 Oct 2019 21:15:02 GMT)
Full text and
rfc822 format available.
Message #50 received at 37493-done <at> debbugs.gnu.org (full text, mbox):
[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/")
Information forwarded
to
guix-patches <at> gnu.org
:
bug#37493
; Package
guix-patches
.
(Tue, 01 Oct 2019 22:31:02 GMT)
Full text and
rfc822 format available.
Message #53 received at 37493 <at> debbugs.gnu.org (full text, mbox):
[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)]
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Wed, 30 Oct 2019 11:24:06 GMT)
Full text and
rfc822 format available.
This bug report was last modified 4 years and 180 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.