On Mon, 27 Jul 2009, Mauro Carvalho Chehab wrote: > Em Tue, 21 Jul 2009 09:14:36 +0200 > Matthias Schwarzott <zzam@xxxxxxxxxx> escreveu: > It is not a good idea to run as root. Most people compile everything > with a normal user and then use "sudo" command to install/remove/insert > modules. Unfortunately, depending on the distribution, sudo inherits PATH from > the normal user, instead of root. Due to that, if you replace it for just > lsmod, it will fail for people that don't use gentoo. > > Maybe good solution is to test if lsmod (and other similar tools) are at /sbin > or /usr/sbin. > > Alternatively, we can try to replace lsmod by something like (untested): > > v4l_modules := $(shell PATH=$PATH:/usr/local/sbin:/usr/sbin:/sbin lsmod|cut -d' ' -f1 ) $(patsubst %.ko,%,$(inst-m)) Check my patch again, we can just delete the v4l_modules line as nothing uses it. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html