2000-11-22 09:28:18

by Christian Gennerat

[permalink] [raw]
Subject: Re: [Cooker] Re: [CHRPM] modutils-2.3.20-1mdk

Chmouel Boudjnah a ?crit :

> i have no idea what's happen here, but gonna to look at this.

The problem is in modules.conf
Most users do not use "depfile" and "path" commands
So they have not seen this problem.
Other have not read the man.
As I have tested the new 2.4 kernel, and I wanted to be able
to load and run modules, I have a big modules.conf using
theese commands.

The man for modules.conf says:

DEFAULT CONFIGURATION
depfile=/lib/modules/`uname -r`/modules.dep
pcimapfile=/lib/modules/`uname -r`/modules.pcimap
isapnpmapfile=/lib/modules/`uname -r`/modules.isapnpmap
usbmapfile=/lib/modules/`uname -r`/modules.usbmap

path[boot]=/lib/modules/boot
path[toplevel]=/lib/modules/`uname -r`
path[toplevel]=/lib/modules/`kernelversion`
path[toplevel]=/lib/modules/default

This was true for modutils-2.3.19, and older versions.
With modutils-2.3.20 the syntax have changed.
Anything in ChangeLog?
`uname -r` has to be changed in $(uname -r)




2000-11-22 10:13:29

by Keith Owens

[permalink] [raw]
Subject: Re: [Cooker] Re: [CHRPM] modutils-2.3.20-1mdk

On Wed, 22 Nov 2000 09:57:02 +0100,
Christian Gennerat <[email protected]> wrote:
>The problem is in modules.conf
>Most users do not use "depfile" and "path" commands
>So they have not seen this problem.
>
>This was true for modutils-2.3.19, and older versions.
>With modutils-2.3.20 the syntax have changed.
>Anything in ChangeLog?
>`uname -r` has to be changed in $(uname -r)

Try this patch against modutils 2.3.20. If it does not fix the problem
then I will need a copy of your modules.conf. `uname -r` should work
even without this patch, it does for me. You must have something
unusual in your modules.conf.

Index: 21.4/util/meta_expand.c
--- 21.4/util/meta_expand.c Tue, 21 Nov 2000 19:38:04 +1100 kaos (modutils-2.3/10_meta_expan 1.4.1.3 644)
+++ 21.4(w)/util/meta_expand.c Wed, 22 Nov 2000 11:14:54 +1100 kaos (modutils-2.3/10_meta_expan 1.4.1.3 644)
@@ -328,10 +328,10 @@ int meta_expand(char *pt, GLOB_LIST *g,
pclose(fin);

if (line) {
- /* Ignore result if no expansion occurred */
- xstrcat(tmpline, "\n", sizeof(tmpline));
- if (strcmp(tmpline, line))
- split_line(g, line, 0);
+ /* shell used to strip one set of quotes. Paranoia code in
+ * 2.3.20 stops that strip so we do it ourselves.
+ */
+ split_line(g, line, 1);
free(line);
}