[PATCH] kbuild: do not check for ancient modutils tools

4 views
Skip to first unread message

Lucas De Marchi

unread,
Jan 17, 2012, 11:50:51 AM1/17/12
to linux-...@vger.kernel.org, Kay Sievers, Michal Marek, linux-...@vger.kernel.org, Lucas De Marchi
scripts/depmod.sh checks for the output of '-V' expecting that it has
module-init-tools in it. It's a hack to prevent users from using
modutils instead of module-init-tools, that only works with 2.4.x
kernels. This however prints an annoying warning for kmod tool, that is
currently replacing module-init-tools.

Rather than putting another check for kmod's version, just remove it
since users of 2.4.x kernel are unlikely to upgrade to 3.x, and if they
do, let depmod fail in that case because they should know what they are
doing.

Signed-off-by: Lucas De Marchi <lucas.d...@profusion.mobi>
---
scripts/depmod.sh | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/scripts/depmod.sh b/scripts/depmod.sh
index a272356..2ae4817 100755
--- a/scripts/depmod.sh
+++ b/scripts/depmod.sh
@@ -9,12 +9,6 @@ fi
DEPMOD=$1
KERNELRELEASE=$2

-if ! "$DEPMOD" -V 2>/dev/null | grep -q module-init-tools; then
- echo "Warning: you may need to install module-init-tools" >&2
- echo "See http://www.codemonkey.org.uk/docs/post-halloween-2.6.txt" >&2
- sleep 1
-fi
-
if ! test -r System.map -a -x "$DEPMOD"; then
exit 0
fi
--
1.7.8.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Cong Wang

unread,
Jan 17, 2012, 11:33:39 PM1/17/12
to Lucas De Marchi, linux-...@vger.kernel.org, Kay Sievers, Michal Marek, linux-...@vger.kernel.org
On Tue, 2012-01-17 at 14:50 -0200, Lucas De Marchi wrote:
> scripts/depmod.sh checks for the output of '-V' expecting that it has
> module-init-tools in it. It's a hack to prevent users from using
> modutils instead of module-init-tools, that only works with 2.4.x
> kernels. This however prints an annoying warning for kmod tool, that is
> currently replacing module-init-tools.
>
> Rather than putting another check for kmod's version, just remove it
> since users of 2.4.x kernel are unlikely to upgrade to 3.x, and if they
> do, let depmod fail in that case because they should know what they are
> doing.
>
> Signed-off-by: Lucas De Marchi <lucas.d...@profusion.mobi>

Acked-by: WANG Cong <amw...@redhat.com>

Thanks.

Reply all
Reply to author
Forward
0 new messages