[patch] depmod -b

Timo Teräs timo.teras at iki.fi
Sat Jan 3 15:36:04 UTC 2009


Hi,

I somehow missed that the depmod -b option does not do at all what it is supposed
to do. It just did a chdir() to the basedir which is wrong. Not sure why it was
done like that.

The module-init-tools manpage explains:

-b basedir --basedir basedir
   If  your  modules  are  not  currently  in  the (normal) directory /lib/mod-
   ules/version, but in a staging area, you can  specify  a  basedir  which  is
   prepended  to the directory name.  This basedir is stripped from the result-
   ing modules.dep file, so it is ready to be moved into the normal location.

So the basedir is prepended when *looking* for the files, but it's not included
in the output dependency file.

Attached is a patch to fix it. The patch also fixes the freopen() not
complaining on error.

It's +163 bytes here. But absolutely necessary to make the -b useful at all.

- Timo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: depmod.diff
Type: text/x-diff
Size: 3737 bytes
Desc: not available
Url : http://lists.busybox.net/pipermail/busybox/attachments/20090103/49b9bf07/attachment.bin 


More information about the busybox mailing list