From: swedebugia <swedebugia@riseup.net>
To: 33329@debbugs.gnu.org, rekado@elephly.net
Subject: [bug#33329] [PATCH] gnu: Deprecate linux-module shpchp and tell user to remove it.
Date: Sat, 10 Nov 2018 13:03:29 +0100 [thread overview]
Message-ID: <290b28ec-cd1e-e5e8-275e-133771c7d4f3@riseup.net> (raw)
In-Reply-To: <01fa8c80-c57c-f73a-cec1-af91cacb58bf@riseup.net>
[-- Attachment #1.1: Type: text/plain, Size: 1906 bytes --]
Hi
Updated patch attached with a fix of parens and removed my changes to
linux-modules.scm.
I look forward to the review.
Cheers
On 2018-11-09 23:47, swedebugia wrote:
>
> Hi
>
> Ricardo, you mentioned that this needs to be fixed.
>
> I made a patch to fix this. I did not test the patch as I have no idea
> how to do that.
>
> I have not run the indent script on it.
>
> Se attachment
>
>
> -------- Forwarded Message --------
> Subject: Re: GuixSD system reconfigure error
> Date: Thu, 30 Aug 2018 17:22:53 +0200
> From: Ricardo Wurmus <rekado@elephly.net>
> To: Hebi Li <lihebi.com@gmail.com>
> CC: help-guix@gnu.org
>
>
>
>
> Hi,
>
>> https://github.com/lihebi/dothebi/blob/master/config.scm
>
> I see you use the “shpchp” kernel module. This no longer exists in the
> latest version of the kernel Linux. That’s what this really bad error
> message tries to tell you:
>
>> Backtrace:
>> 8 (primitive-load "/gnu/store/vz7ci9rh483f2zps2cl174rd91b?")
>> In ice-9/eval.scm:
>> 619:8 7 (_ #f)
>> 626:19 6 (_ #<directory (guile-user) 7cf140>)
>> 159:9 5 (_ _)
>> In gnu/build/linux-modules.scm:
>> 184:47 4 (recursive-module-dependencies _ #:lookup-module _)
>> 98:14 3 (module-dependencies _)
>> 85:18 2 (modinfo-section-contents _)
>> In ice-9/ports.scm:
>> 439:11 1 (call-with-input-file #f #<procedure get-bytevector-al?> ?)
>> In unknown file:
>> 0 (open-file #f "r" #:encoding #f #:guess-encoding #f)
>>
>> ERROR: In procedure open-file:
>> Wrong type (expecting string): #f
>> builder for `/gnu/store/0ahsvp7wx52zzh1rywbdbq78llcwb7id-linux-modules.drv' failed with exit code 1
>
> (I only know this because I had the same problem.) Obviously, this
> error message should be changed.
>
> As a workaround you could switch to the LTS kernel where that module
> still exists.
>
> --
> Ricardo
>
>
--
Cheers
Swedebugia
[-- Attachment #1.2: Type: text/html, Size: 4292 bytes --]
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-V2-gnu-Check-for-linux-module-shpchp-and-tell-user-t.patch --]
[-- Type: text/x-patch; name="0001-V2-gnu-Check-for-linux-module-shpchp-and-tell-user-t.patch", Size: 3062 bytes --]
From 4d70dda8c2f119fc6ff9d221eae6f060ff1fcd98 Mon Sep 17 00:00:00 2001
From: swedebugia <swedebugia@riseup.net>
Date: Fri, 9 Nov 2018 22:52:12 +0100
Subject: [PATCH] [V2] gnu: Check for linux-module shpchp and tell user to
remove it.
* gnu/system/mapped-devices.scm (check-device-initrd-modules): New if
statement raising a condition if shpchp is found.
---
gnu/packages/cran.go.3Q3wbP | 0
gnu/system/mapped-devices.scm | 28 +++++++++++++++++++---------
2 files changed, 19 insertions(+), 9 deletions(-)
create mode 100644 gnu/packages/cran.go.3Q3wbP
diff --git a/gnu/packages/cran.go.3Q3wbP b/gnu/packages/cran.go.3Q3wbP
new file mode 100644
index 000000000..e69de29bb
diff --git a/gnu/system/mapped-devices.scm b/gnu/system/mapped-devices.scm
index a87466646..483f952c6 100644
--- a/gnu/system/mapped-devices.scm
+++ b/gnu/system/mapped-devices.scm
@@ -2,6 +2,7 @@
;;; Copyright © 2014, 2015, 2016, 2017, 2018 Ludovic Courtès <ludo@gnu.org>
;;; Copyright © 2016 Andreas Enge <andreas@enge.fr>
;;; Copyright © 2017, 2018 Mark H Weaver <mhw@netris.org>
+;;; Copyright © 2018 swedebugia <swedebugia@riseup.net>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -142,13 +143,22 @@ DEVICE must be a \"/dev\" file name."
;; "usb_storage"), not file names (e.g., "usb-storage.ko"). This is
;; OK because we have machinery that accepts both the hyphen and the
;; underscore version.
- (raise (condition
- (&message
- (message (format #f (G_ "you may need these modules \
+ (if (eqv? (missing 'shpchp))
+ ;; Tell user to remove shpchp from config.scm
+ ;; True
+ (raise (condition
+ (&message
+ (message (format #f (G_ "shpchp is no longer \
+needed because it has been included in linux-libre. Please remove it from \
+your config.scm to continue"))))))
+ ;; Else
+ (raise (condition
+ (&message
+ (message (format #f (G_ "you may need these modules \
in the initrd for ~a:~{ ~a~}")
- device missing)))
- (&fix-hint
- (hint (format #f (G_ "Try adding them to the
+ device missing)))
+ (&fix-hint
+ (hint (format #f (G_ "Try adding them to the
@code{initrd-modules} field of your @code{operating-system} declaration, along
these lines:
@@ -161,9 +171,9 @@ these lines:
If you think this diagnostic is inaccurate, use the @option{--skip-checks}
option of @command{guix system}.\n")
- missing)))
- (&error-location
- (location (source-properties->location location)))))))))
+ missing)))
+ (&error-location
+ (location (source-properties->location location))))))))))
\f
;;;
--
2.18.0
next prev parent reply other threads:[~2018-11-10 12:04 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-29 21:20 GuixSD system reconfigure error Hebi Li
2018-08-30 10:28 ` Pierre Neidhardt
2018-08-30 11:05 ` Tobias Geerinckx-Rice
2018-08-30 15:22 ` Ricardo Wurmus
2018-08-30 19:51 ` Hebi Li
2018-08-30 20:54 ` Ricardo Wurmus
2018-08-30 22:48 ` Hebi Li
2018-11-09 22:47 ` [bug#33329] [PATCH] gnu: Deprecate linux-module shpchp and tell user to remove it swedebugia
2018-11-10 12:03 ` swedebugia [this message]
2018-11-10 22:31 ` Ludovic Courtès
2018-11-11 0:15 ` Brett Gilio
2018-11-11 7:27 ` swedebugia
2018-11-11 11:32 ` Ludovic Courtès
2018-11-11 18:42 ` swedebugia
2018-11-12 8:42 ` bug#33329: " Ludovic Courtès
2018-11-11 11:27 ` [bug#33329] " Ludovic Courtès
2018-11-09 11:55 ` GuixSD system reconfigure error swedebugia
2018-11-12 7:53 ` Hebi Li
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=290b28ec-cd1e-e5e8-275e-133771c7d4f3@riseup.net \
--to=swedebugia@riseup.net \
--cc=33329@debbugs.gnu.org \
--cc=rekado@elephly.net \
/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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.