Skip to content

Commit

Permalink
depmod: handle the case of /sbin/depmod without /sbin in PATH
Browse files Browse the repository at this point in the history
[ Upstream commit cedd186 ]

Commit 436e980 ("kbuild: don't hardcode depmod path") stopped
hard-coding the path of depmod, but in the process caused trouble for
distributions that had that /sbin location, but didn't have it in the
PATH (generally because /sbin is limited to the super-user path).

Work around it for now by just adding /sbin to the end of PATH in the
depmod.sh script.

Reported-and-tested-by: Sedat Dilek <sedat.dilek@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
torvalds authored and gregkh committed Jan 12, 2021
1 parent 8532d3e commit 43dee88
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions scripts/depmod.sh
Expand Up @@ -15,6 +15,8 @@ if ! test -r System.map ; then
exit 0
fi

# legacy behavior: "depmod" in /sbin, no /sbin in PATH
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
Expand Down

0 comments on commit 43dee88

Please sign in to comment.