[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAMOw1v7+Hb8RgBdA3ZPzEFM0Mt5y=_6=RF12CodciOw0Yq2vmg@mail.gmail.com>
Date: Mon, 23 Jan 2012 12:32:22 -0200
From: Lucas De Marchi <lucas.demarchi@...fusion.mobi>
To: Michal Marek <mmarek@...e.cz>
Cc: Kay Sievers <kay.sievers@...y.org>, Cong Wang <amwang@...hat.com>,
linux-kernel@...r.kernel.org, linux-modules@...r.kernel.org
Subject: Re: [PATCH] kbuild: do not check for ancient modutils tools
On Mon, Jan 23, 2012 at 12:17 PM, Michal Marek <mmarek@...e.cz> wrote:
> On Fri, Jan 20, 2012 at 05:40:58PM +0100, Kay Sievers wrote:
>> On Wed, Jan 18, 2012 at 05:33, Cong Wang <amwang@...hat.com> wrote:
>> > 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.
>
> You could call kmod "module-init-tools 3.13 (compatible; kmod 3)" ;)
>
> Just kidding.
I even wrote a patch to fake kmod as module-init-tools 4.0. Not worth
applying though :-)
>
>> >> 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.demarchi@...fusion.mobi>
>> >
>> > Acked-by: WANG Cong <amwang@...hat.com>
>>
>> Michal, mind picking this up? The warning it produces with kmod is
>> really misleading, and the check seems unnecessary these days.
>
> I applied it to kbuild.git#rc-fixes and will send it to Linus for 3.3.
Thanks
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists