lkml.org 
[lkml]   [2007]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [v4l-dvb-maintainer] [PATCH] DVB: Delete unused header file linux/dvb/version.h.
Date
On Friday 23 March 2007, Arjan van de Ven wrote:
> On Fri, 2007-03-23 at 16:47 +0100, Marcel Siegert wrote:
> > On Friday 23 March 2007, Robert P. J. Day wrote:
> > >
> > > Delete the unreferenced header file include/linux/dvb/version.h.
> > >
> > > Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
> > >
> > NACK
> > this header is unreferenced in kernel, but used for applications that compile for userspace
> > to determine the api version.
>
>
> btw that is something extremely evil to do. Either you have varying apis
> between kernels (which is naughty, but I suppose you can go up in
> capabilities) or it's fully static.
>
> If it's dynamic, doing it compiletime is BAD. The same binaries should
> run on multiple kernels after all. An ioctl or a sysfs thing or
> something is a lot better than a compile time constant.
>
> If it's entirely static exposing a "version" doesn't mean anything and
> it might as well be killed...
>
> Greetings,
> Arjan van de Ven

hi arjan,

we discussed that topic - an IOCTL to determine the api version - on dvb ml in nov.2005 and
got no result. so this is still there and will stay imho, until we will find a solution.

regards
marcel

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

\
 
 \ /
  Last update: 2007-03-23 17:23    [from the cache]
©2003-2011 Jasper Spaans