Rafael, thanks again for the patch. The last version sounds reasonable. I upload test builds in the PPA with it. Regarding the preloading of libv4lcompat, I handle that directly within the package (by requiring installation of the proper library).
It'd be great if people with v4l-needing hardware could test it, once it's available. Thanks again, Nicola > Subject: Re: [CinCV] v4l1 patch, try2 > From: [email protected] > To: [email protected] > Date: Mon, 30 May 2011 10:06:44 -0700 > > People, > Thinking a little, after Einar email, I reached a patch that: > > - keeps v4l support as always for people using kernel <= 2.6.37 > - keeps v4l support for people using kernel >= 2.6.38, needs > LD_PRELOAD'ing of libv4l1compat.so > - cinelerra compiles fine with and without libv4l > - adds videodev.h to our tree > > ; ) > > > > > Hi Einar, > > I must disagree with you. > > > > If you take a look at the old videodev.h, there is this statement: > > > > * Video for Linux version 1 - OBSOLETE > > * > > * Header file for v4l1 drivers and applications, for > > * Linux kernels 2.2.x or 2.4.x. > > * > > * Provides header for legacy drivers and applications > > > > > > And now it's gone. > > There is no such device you're saying that have only v4l1 driver. V4L1 is > > not present in kernel for a long long time (only the V4L1 compatibility > > layer was there). > > Anyone using kernel 2.4? > > IMHO, the correct way to support V4L1 is using libv4l. > > > > Of course we could commit videodev.h inside our cinelerra tree in order to > > get rid of the libv4l dependency for those still using kernel <= 2.6.37 or > > that don't need V4L support at all. > > > > Best regards, > > Rafael Diniz > > > > > >> Hi. > >> > >> You must not unconditionally remove v4l1 support. I think that > >> configure must detect v4l1 and if it does not exist use libv4l1. There > >> may be some older devices, that have only v4l1 drivers. We have to > >> leave user an option to use older kernel with v4l1 support. Option to > >> fully remove v4l1 would be nice too. > >> > >> Einar > > > > > > > > _______________________________________________ > > Cinelerra mailing list > > [email protected] > > https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra > >
