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 660593 - Should apply color profile from colord
Should apply color profile from colord
Status: RESOLVED OBSOLETE
Product: gnome-shell
Classification: Core
Component: general
3.15.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on: 667134
Blocks:
 
 
Reported: 2011-09-30 20:25 UTC by Baptiste Mille-Mathias
Modified: 2018-05-14 08:56 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Baptiste Mille-Mathias 2011-09-30 20:25:02 UTC
[baptiste@oak gnome-utils]$ gnome-screenshot -d 4 -i
** (gnome-screenshot:32259): DEBUG: The GetProfileForWindow request failed: GDBus.Error:org.freedesktop.DBus.Error.Spawn.ExecFailed: Failed to execute program /usr/bin/gcm-session: Success
Comment 1 Matthias Clasen 2011-10-01 14:15:51 UTC
Its a debug message; gnome-screenshot just lacks the log handler setup to not show these by default.
Comment 2 Baptiste Mille-Mathias 2011-10-01 15:50:32 UTC
Hi Matthias,

I'm using F16, my concern is also for the missing /usr/bin/gcm-session which should be installed by package gnome-color-manager but which is not.
Comment 3 Baptiste Mille-Mathias 2012-01-04 10:35:20 UTC
Actually this issue is caused by bug 667134.
so gnome-screenshot should use the new API provided by
gnome-settings-daemon
Comment 4 Cosimo Cecchi 2012-01-26 22:45:35 UTC
gnome-settings-daemon has no such API. I believe we would need to ask colord directly.
Richard, what API should we use instead of GetProfileForWindow?
Comment 5 Jean-François Fortin Tam 2014-11-19 04:14:55 UTC
I don't see this warning here (probably because bug #667134 is fixed).

But there's something I don't understand here, fundamentally: since my screens are now calibrated and managed with fullscreen color correction, the screenshots I take with gnome-screenshot or by simply pressing the printscreen key (optionally with the Alt modifier etc.) show up with the wrong colors when opening them with Eye of GNOME, as if they had an embedded color profile (unlike the premises of this bug report?) that gets applied twice (by the app and then with fullscreen color correction on top), which would seem exactly like what one would want it *not* to do.

Shouldn't screenshots be taken with the raw colors, without fullscreen color management affecting them? Does gnome-screenshot or gnome-shell take color correction into account and embed such information in the resulting images nowadays, or that's not the case?

I'm worried about those screenshot files being "tainted" with a profile when they shouldn't, which would result in them being misrendered on other computers as far as I can imagine - am I missing something? It's a bit confusing.


See also: https://bugzilla.gnome.org/buglist.cgi?query_format=advanced;short_desc=color%20colour;short_desc_type=anywordssubstr;resolution=---;product=eog
Comment 6 Cosimo Cecchi 2014-11-19 06:49:28 UTC
-> gnome-shell

This is a tricky issue; the shell/compositor does not currently know about the color profile in use, so the screenshot (taken by the shell) will be color corrected.
The ICC color profile should be attached to the picture though, so that your viewer can convert the colorspace. This used to work, see bug 613926 that describes the same use case, but then the colord API was removed and the code never ported to another one.

These days it's arguably the shell that should embed the color profile information into the image, since it also first saves the image to disk.
Comment 7 imyxhuang 2018-05-14 01:45:57 UTC
Okay, seeing as this hasn't had any activity for almost four years, I re-submitted this on GitLab here: https://gitlab.gnome.org/GNOME/gnome-shell/issues/272.

Hopefully that's allowed?
Comment 8 Florian Müllner 2018-05-14 08:56:57 UTC
Of course it's allowed, but creating additional work that has to be spend on bug triaging instead of fixing issues isn't the most productive. But now that it's done, let's close this one to prevent the bug from being migrated to gitlab.