[PATCH] depmod -r dummy

[Natanael Copa]

On Tue, 2009-02-17 at 14:08 +0100, Bernhard Reutner-Fischer wrote:
> On Tue, Feb 17, 2009 at 12:00:12PM +0100, Natanael Copa wrote:
> >Hi,
> >
> >The attatched patch makes it possible to run 'make modules_install' with
> >busybox depmod.
> >
> >The linux kernel makefile runs depmod -r which is a compatibility option
> >that does nothing.
> 
> Did module-init-tools ever support any depmod -r ?
> Perhaps fix the kernel instead since it needs (IIRC) module-init-tools anyway
> which supposedly never did anything real with -r?


from depmod.c (around row 1232):
                case 'u':
                case 'q':
                case 'r':
                        break;
                case 'C':
                        config = optarg;
                        break;

it supports -r. Its not in the man page and it does nothing. I assume
linux kernel Makefiles uses it for compatibility reasons of some sort.

Thought it would be easier to get a patch into bb than in linux.

-nc


> 
> >--- busybox-1.13.2/modutils/depmod.c.orig	2009-02-12 12:53:11.000000000 +0000
> >+++ busybox-1.13.2/modutils/depmod.c	2009-02-12 12:57:29.000000000 +0000
> >@@ -37,6 +37,7 @@
> > 	ARG_e = (1<<3), /* with -F, print unresolved symbols */
> > 	ARG_F = (1<<4), /* System.map that contains the symbols */
> > 	ARG_n = (1<<5)  /* dry-run, print to stdout only */
> >+	ARG_r = (1<<6)  /* Compat dummy. Linux Makefile uses it */
> > };
> > 
> > static int FAST_FUNC parse_module(const char *fname, struct stat *sb,
> >@@ -138,7 +139,7 @@
> > 	struct utsname uts;
> > 	int tmp;
> > 
> >-	getopt32(argv, "aAb:eF:n", &moddir_base, NULL);
> >+	getopt32(argv, "aAb:eF:nr", &moddir_base, NULL);
> > 	argv += optind;
> > 
> > 	/* goto modules location */
> 
> >_______________________________________________
> >busybox mailing list
> >busybox at busybox.net
> >http://lists.busybox.net/mailman/listinfo/busybox