GNU bug report logs - #30604
[PATCH 0/4] Load Linux module only when supported hardware is present.

Previous Next

Full log

View this message in rfc822 format

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 30604 <at> debbugs.gnu.org
Subject: [bug#30604] [PATCH v8 2/7] linux-modules: Add install-modules.
Date: Sun, 4 Mar 2018 13:45:04 +0100

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

Hi Ludo,

>   (write-module-alias-database modules port)  ;for “modules.alias”
>   (write-module-device-database modules port) ;for “modules.devname”

I'd prefer "install-" because their interface is somewhat like install-file
(caller provides output directory and callee adds "modules."* automatically).

But I used the names "write-module-"* in the patchset v9.

> I’m not sure we need ‘install-module-files’ itself.  Perhaps we can
> inline it at the call site?

Sure.  Did so now.

> For the devname code, please avoid ‘set!’.

I found an even better way to avoid both.

> The indentation of ‘match’ forms is wrong.  Would it be OK for you to
> pass it through ./etc/indent-code.el?  (It’s non interactive, you don’t
> need to actually use Emacs.)

I'm actually trying to switch to emacs but when I press Tab it messes up
the stuff more.  Is there some kind of configuration for guile indentation?

Also, every time I open gnu/build/linux-modules.scm I get

"The local variables list .. may not be safe

* bug-reference-bug-repexp
* eval: (modify-syntax-entry 126 "'")
* eval: (modify-syntax-entry 36 "'")
* eval: (modify-syntax-entry 43 "'")"

???

[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.