[gstreamer-bugs] [Bug 583640] [v4l2src] add support for better device detection with libudev

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=583640

  GStreamer | gst-plugins-good | Ver: git




------- Comment #7 from Filippo Argiolas  2009-06-01 13:09 UTC -------
(In reply to comment #5)
> I think our current device probing API (GstPropertyProbe and selected
> properties) is complete rubbish and not fit for the job, and we shouldn't put
> too much effort into improving it, let alone add new dependencies for that,
> even if they're optional. I think we should rather try and come up with a new
> API (GstDeviceProbe interface or somesuch) that doesn't completely suck.

I agree, GstPropertyProbe sucks, but that's what we have now. And, IMHO, it's
not so good to reject an improvement of the present state in the name of a
future, probably better but still not even sketched, interface that will
replace it.
I wrote this patch because my mail in gstreamer-devel didn't get any reply. So
I supposed (and still believe) PropertyProbe rework it's not exactly an high
priority task, and there isn't anyone willing/able to spend some time in
designing a good interface for device detection.

If propertyprobe sucks, its v4l?src implementation sucks even more. This patch
just tries to fix the current implementation (with this patch I can easily do
device detection, while still being a bit ugly it works pretty fine. As it is
now I just cannot use gstreamer for that job).

About the new dependency argument, let alone it's optional, please note that
libudev is part of udev, it's currently used at least in pulseaudio, and will
be used soon (if not yet) in DeviceKit-power, DeviceKit-disks, NetworkManager,
gnome-bluetooth, maybe X and probably many more
(http://lists.freedesktop.org/archives/devkit-devel/2009-April/000140.html). So
it's not an exotic dependency, it's or will be soon part of the core stuff in
every linux distribution.

> I realise that's not going to help you immediately, but then getting this code
> out of cheese and into gstreamer is not exactly mission critical, so maybe we
> can afford taking a bit longer to get this right?

Sure I have no hurry, unless a bit longer does mean never ;)
Anyway, while getting this code out of cheese isn't mission critical I consider
a gstreamer fail having to do a low level task like device detection in cheese.

@Stefan: I meant udev 128, sorry.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=583640.