[FFmpeg-devel] [PATCH] libavdevice: v4l2: use libv4l2 when requested.

[Luca Abeni]

Hi Ronald,

On 13/05/10 21:23, Ronald S. Bultje wrote:
> Hi,
>
> On Thu, May 13, 2010 at 3:17 PM, Konstantin Pavlov<thresh at altlinux.ru>  wrote:
>> This adds support for numerous pixfmts used in web cameras through
>> libv4l2.
>
> I'm not particularly in favour of random external libXYZ thingies,
> this goes for any XYZ that will soon be irrelevant, unmaintained and
> will take a SoC student to properly integrate into FFmpeg after that.

As far as I can see, this libv4l dependency is optional (if the library 
is not installed, we can still compile the good old code). So, I think 
this should not be a problem...

I agree that the correct solution would be to add support for bayer (or 
whatever it is) in libswscale or libavcodec, and if/when I get a webcam 
which requires it I'll implement the right thing.
But for now there are people who seem to want to use libv4l (and do not 
want to use LD_PRELOAD, for some reason...). Since the patch is not 
intrusive, I think it can be ok (modulo the fact that someone has to 
review the configure part). No?


			Thanks,
				Luca