netdev
[Top] [All Lists]

Re: [patch 2.6.9 0/11] Add MODULE_VERSION to several network drivers

To: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx>
Subject: Re: [patch 2.6.9 0/11] Add MODULE_VERSION to several network drivers
From: "John W. Linville" <linville@xxxxxxxxxxxxx>
Date: Thu, 21 Oct 2004 08:22:05 -0400
Cc: netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, jgarzik@xxxxxxxxx, davem@xxxxxxxxxxxxx, john.ronciak@xxxxxxxxx, ganesh.venkatesan@xxxxxxxxx, akpm@xxxxxxxx, romieu@xxxxxxxxxxxxx, ctindel@xxxxxxxxxxxxxxxxxxxxx, fubar@xxxxxxxxxx, greearb@xxxxxxxxxxxxxxx
In-reply-to: <1098350269.2810.17.camel@laptop.fenrus.com>; from arjan@fenrus.demon.nl on Thu, Oct 21, 2004 at 11:17:49AM +0200
Mail-followup-to: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, jgarzik@xxxxxxxxx, davem@xxxxxxxxxxxxx, john.ronciak@xxxxxxxxx, ganesh.venkatesan@xxxxxxxxx, akpm@xxxxxxxx, romieu@xxxxxxxxxxxxx, ctindel@xxxxxxxxxxxxxxxxxxxxx, fubar@xxxxxxxxxx, greearb@xxxxxxxxxxxxxxx
References: <20041020141146.C8775@tuxdriver.com> <1098350269.2810.17.camel@laptop.fenrus.com>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.2.5.1i
On Thu, Oct 21, 2004 at 11:17:49AM +0200, Arjan van de Ven wrote:
> On Wed, 2004-10-20 at 20:11, John W. Linville wrote:
> > Patches to add MODULE_VERSION lines to several network drivers...
> > 
> > Here is the list:
> 
> have you checked if the version of these drivers is actually useful? (eg
> updated when the driver changes) If it's not I'd say adding a
> MODULE_VERSION to it makes no sense whatsoever.

Why do I feel like I'm being baited...? :-)

I would have to suspect that if a version string exists, that it has at
least some meaning to the primary developers/maintainters.  It certainly
is beyond my control to force the maintainers to give meaning to their
version strings.

Is this a political statement against the MODULE_VERSION macro and/or
its purpose?  I'm not overly interested in debating that one...

John
-- 
John W. Linville
linville@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>