After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 709910 - Add GStreamer-based resolve plugin
Add GStreamer-based resolve plugin
Status: RESOLVED DUPLICATE of bug 749458
Product: grilo
Classification: Other
Component: plugins
git master
Other Linux
: Normal enhancement
: ---
Assigned To: grilo-maint
grilo-maint
Depends on:
Blocks:
 
 
Reported: 2013-10-11 12:56 UTC by Jack Leigh
Modified: 2015-05-21 13:01 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Jack Leigh 2013-10-11 12:56:25 UTC
Probably the best way to do this will be to use GstDiscoverer.
The additional keys can be exposed under slow_keys so that the user can determine whether to take the performance hit.
Comment 1 Bastien Nocera 2014-02-21 13:12:17 UTC
What additional metadata would be useful here and for which sort of application?

I'd personally rather this was implemented in a separate plugin that'd depend on GStreamer, rather than making grilo's core libraries (which are now responsible for doing GFile -> GrlMedia with metadata) depend on GStreamer.
Comment 2 Juan A. Suarez Romero 2014-02-21 15:18:13 UTC
I think it could implemented on filesystem conditionally. So downstream could enable gstreamer or not depending if they want this extended capability.
Comment 3 Bastien Nocera 2014-02-21 15:31:42 UTC
How is that better than a separate plugin?

Conditional support in a library like this is a recipe for more problems when bugs are reported.
Comment 4 Juan A. Suarez Romero 2014-02-21 19:01:51 UTC
This new plugin is mostly like the Filesystem one, except it provides more information thanks to the use of GStreamer.

So probably any new feature we add to Filesystem would be great to have it in this new one.

I don't know, maybe I think this extra information is really useful, that distros will package it instead of our current filesystem plugin. If so, I don't see why we do not want to include that extended plugin instead of current Filesystem one.

And even in this case, there could be some use case where gstreamer is not required/available, so that's why the conditional compilation proposal.

Anyway, other approach is do what do you suggest, and just move the new plugin to our core set if it is really very used.
Comment 5 Bastien Nocera 2014-08-08 08:33:18 UTC
(In reply to comment #4)
> This new plugin is mostly like the Filesystem one, except it provides more
> information thanks to the use of GStreamer.

No, not really. You'd only implement resolve in the GStreamer plugin. The tracker plugin also exports a whole lot of metadata gathered using GStreamer.
Comment 6 Bastien Nocera 2014-08-08 08:36:53 UTC
The plugin could gather things like:
- bitrate
- duration
- framerate
- width, height
- orientation

Based on a URI being provided in the GrlMedia element.
Comment 7 Bastien Nocera 2015-05-21 13:01:56 UTC
Thanks for taking the time to report this.
This particular bug has already been reported into our bug tracking system, but please feel free to report any further bugs you find.

*** This bug has been marked as a duplicate of bug 749458 ***