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 745749 - [fdonotificationbackend] Check server's spec version and respond accordingly
[fdonotificationbackend] Check server's spec version and respond accordingly
Status: RESOLVED WONTFIX
Product: glib
Classification: Platform
Component: gapplication
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2015-03-06 15:03 UTC by ria.freelander
Modified: 2015-04-08 10:32 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description ria.freelander 2015-03-06 15:03:22 UTC
This is more complex case of https://bugzilla.gnome.org/show_bug.cgi?id=745634
I think that fdonotificationbackend should not always assume than daemon supports 1.2 version of the spec, but maintain backward compatibility.
I think fdonotificationbackend should:
1. check spec_version from GetServerInformation.
2. use only tags from returned version.
For example use image-path only if spec_version >= 1.2, if 1.2 <= spec_version <= 1.1, use image_path, else use app_icon.
Comment 1 Lars Karlitski 2015-03-09 08:38:53 UTC
The fdo backend is fully backwards compatible with older versions of the spec, as hints that a server doesn't understand must be ignored. It just means that you don't get an icon in some cases.

In general, I'd like to avoid the extra round-trip on the bus if that's possible. Are there really still servers out there that haven't been patched to recognoize 'image-path' as well as 'image_path'? Should we maybe just always use 'image_path', then?

As I said in the other bug, we will not support sending the notification icon as 'app_icon', as that is supposed to be the application's icon.
Comment 2 ria.freelander 2015-03-09 08:41:45 UTC
yes, KDE one understand image_path, but not image-path for now.
(there is https://bugs.kde.org/show_bug.cgi?id=344885 for understanding image-path)
XFCE one does not understand both.
Comment 3 Lars Karlitski 2015-03-09 09:12:05 UTC
We'll try to push the spec to xdg soon. Closing this bug here.

I don't care much about supporting older notification servers. If XFCE's daemon doesn't get patched, it won't have icons. That's an ok regression and saves the majority of users from the extra dbus call.
Comment 4 David Edmundson 2015-04-08 10:32:49 UTC
I've not seen any progress on pushing this to freedesktop.org.

It was referenced in the relevant KDE bug that you're not putting this on fd.o because of slow sys admins. 

You don't need a sysadmin to create a wiki editing account.

Anyone with fd.o git access for any repo can create a new wiki account for anyone else see http://wiki.freedesktop.org/sitewranglers/wiki/401/.