GNOME Bugzilla – Bug 753529
screenshot: Use gnome-screenshot
Last modified: 2015-09-13 11:40:39 UTC
Created attachment 309089 [details] [review] libscreenshot: Use gnome-screenshot This makes it functional. It restores the functionality dropped by this commit: https://git.gnome.org/browse/gnome-settings-daemon/commit/?id=f33cd1921b4c32eecf54d37d15f9bec0232aa9e3
I'm working on an improved patch.
Created attachment 309128 [details] [review] libscreenshot: Use gnome-screenshot v2 More complete patch. Now it supports filename and include_cursor parameters.
Thanks for helping... :) I will test your patch when I will have free time and most likely I will accept it - this will be 100% better then current warning. :)
It sounded too good to be true... gnome-screenshot first tries to get screenshot from shell (in our case - from gnome-flashback) and fallbacks to X11 if there was problems. Your code worked (partially) because we never tried to own org.gnome.Shell.Screenshot dbus name that allowed gnome-screenshot directly fallback to X11 as dbus name was not available. So in short - I think we can not use gnome-screenshot to take screenshots. :( I pushed few commits. Now we own dbus name, also SelectArea method is implemented.
Done! Please test if you have time.
Thanks! The new code works, but it has a major problem. I opened a new bug: https://bugzilla.gnome.org/show_bug.cgi?id=754952