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 585853 - video file previews
video file previews
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: gphoto backend
git master
Other Linux
: Normal normal
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2009-06-15 13:44 UTC by Michael Chudobiak
Modified: 2009-06-16 13:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Small test program (1.80 KB, text/plain)
2009-06-16 00:08 UTC, David Zeuthen (not reading bugmail)
Details

Description Michael Chudobiak 2009-06-15 13:44:09 UTC
gThumb trunk now uses this backend for its camera importer. gThumb imports any image, video, and audio file it finds on a camera.

It would be VERY much appreciated if video previews were added to the backend. libgphoto2 does provide previews for videos, so it shouldn't be a big deal. The old importer used this functionality.

Otherwise, using other methods to thumbnail videos on a camera card turn out to be brutally slow, and we have to fallback to a generic mime-type icon (which is what gThumb trunk is doing at the moment).

The backend code has this comment:

/* assume that all JPG files, and only all JPG files, has a preview file */

- Mike
Comment 1 David Zeuthen (not reading bugmail) 2009-06-16 00:08:44 UTC
Created attachment 136680 [details]
Small test program

Fixed in master:

http://git.gnome.org/cgit/gvfs/commit/?id=3431510c410c4c97c372ac797c0507d1c7eb69bf

Since Nautilus won't use libgnome-desktop for thumbnailing video, I wrote a small test program that I'm attaching here. This is the output

$ ./main gphoto2://[usb:004,006]/DCIM/100NIKON/DSCN0143.AVI
Read preview::icon thumbnail of 3882 bytes into preview.bin
for URI gphoto2://[usb:004,006]/DCIM/100NIKON/DSCN0143.AVI of 15057696 bytes.
Comment 2 Bastien Nocera 2009-06-16 09:27:11 UTC
Nitpick, but wouldn't something like that work better:
g_content_type_is_a (mimetype, "video/*")
as it would do subclass checking as well.
Comment 3 Michael Chudobiak 2009-06-16 13:03:17 UTC
Thanks, David!

- Mike