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 651187 - Correctly register thumbnailers (without gconf schemas)
Correctly register thumbnailers (without gconf schemas)
Status: RESOLVED FIXED
Product: libgsf
Classification: Core
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2011-05-26 20:06 UTC by Vincent Untz
Modified: 2011-12-07 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Register thumbnailer the new way (39.37 KB, patch)
2011-05-26 20:20 UTC, Vincent Untz
none Details | Review
Script to automatically convert the old GConf file into a (1.33 KB, text/plain)
2011-11-08 23:42 UTC, Wouter Bolsterlee (uws)
  Details

Description Vincent Untz 2011-05-26 20:06:49 UTC
Registering thumbnailers now doesn't require a gconf schema anymore, but just a .thumbnailer file in /usr/share/thumbnailers. This file is similar to a .desktop file. See for instance http://git.gnome.org/browse/evince/tree/thumbnailer/evince.thumbnailer.in
Comment 1 Vincent Untz 2011-05-26 20:20:56 UTC
Created attachment 188704 [details] [review]
Register thumbnailer the new way

We now need a .thumbnailer file in $(datadir)/thumbnailers. This also
means we don't need the gconf dependency anymore.
Comment 2 Morten Welinder 2011-07-03 01:51:42 UTC
Interesting.  Do you have a reference to documentation for this?

Note to self: this patch conflicts with the recent doc/Makefile.am
commit.  Fix is trivial, no need to update patch.
Comment 3 Vincent Untz 2011-07-03 07:58:02 UTC
Hrm, I'm not sure there's documentation outside https://bugzilla.gnome.org/show_bug.cgi?id=638172

I see we have an old integration guide mentioning the old way [1], but I'm not sure it's still maintained. I filed bug 653889 to get it fixed.

[1] http://developer.gnome.org/integration-guide/stable/thumbnailer.html
Comment 4 Wouter Bolsterlee (uws) 2011-11-08 23:42:34 UTC
Created attachment 201037 [details]
Script to automatically convert the old GConf file into a 

I missed this functionality as well, so some time ago I wrote a conversion script for the registration file, the result of which I manually install in /usr (hacky, I know). FYI, this simple script is attached to this report. Unfortunately I discovered this bug too late so I did some duplicate work it seems... :s

Any reason why this isn't merged yet?
Comment 5 Morten Welinder 2011-11-09 12:50:07 UTC
> Any reason why this isn't merged yet?

I am still hoping someone will actually document that this is the correct
way to do things.
Comment 6 Morten Welinder 2011-12-07 21:47:17 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.