Messages in this thread |  | | | From | Marcel Siegert <> | | Subject | Re: [v4l-dvb-maintainer] [PATCH] DVB: Delete unused header file linux/dvb/version.h. | | Date | Fri, 23 Mar 2007 17:18:27 +0100 |
| |
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/
|  |