On Mon, Dec 28, 2020 at 8:40 PM Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > > On Mon, Dec 28, 2020 at 12:04 AM Sedat Dilek <sedat.dilek@xxxxxxxxx> wrote: > > > > > $ dpkg -L kmod | grep bin | grep depmod > > > /sbin/depmod > > > > > > $ which depmod > > > [ empty ] > > > > > > $ echo $PATH > > > /opt/proxychains-ng/bin:/home/dileks/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games > > Ok, I think this is a broken setup that has a separate /sbin but does > not have it in the PATH. > > As you noticed, you can fix it with > > DEPMOD=/sbin/depmod > > or you could just make /sbin part of your PATH. > > It looks like on your distro, /sbin is restricted to just the > super-user PATH, which is odd, but I guess there's at least _some_ > logic to it. > > I guess we could have some compatibility thing in scripts/depmod.sh, > something like > > diff --git a/scripts/depmod.sh b/scripts/depmod.sh > index e083bcae343f..a93261207453 100755 > --- a/scripts/depmod.sh > +++ b/scripts/depmod.sh > @@ -15,6 +15,8 @@ if ! test -r System.map ; then > exit 0 > fi > > +# legacy behavior: "depmod" in /sbin > +PATH="$PATH:/sbin" > if [ -z $(command -v $DEPMOD) ]; then > echo "Warning: 'make modules_install' requires $DEPMOD. Please > install it." >&2 > echo "This is probably in the kmod package." >&2 > > or similar. Does that work for you? > Looks like it works. I see: sh ./scripts/depmod.sh depmod 5.11.0-rc1-2-amd64-clang-ias But no more a depmod warning. Will you send a patch or apply this directly? - Sedat -
